From: Michael <heluecht@pirati.ca>
Date: Sun, 8 Jul 2018 04:55:45 +0000 (+0000)
Subject: Avoid undefined index
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=5ee2db8a164ae27e2a5821ff4036f3d686b0c40e;p=friendica.git

Avoid undefined index
---

diff --git a/src/Model/Item.php b/src/Model/Item.php
index 3086a869be..6fff08c715 100644
--- a/src/Model/Item.php
+++ b/src/Model/Item.php
@@ -181,7 +181,7 @@ class Item extends BaseObject
 			if (array_key_exists('object-type', $row)) {
 				$row['object-type'] = ACTIVITY_OBJ_NOTE;
 			}
-		} elseif (in_array($row['verb'], ['', ACTIVITY_POST, ACTIVITY_SHARE])) {
+		} elseif (array_key_exists('verb', $row) && in_array($row['verb'], ['', ACTIVITY_POST, ACTIVITY_SHARE])) {
 			// Posts don't have an object or target - but having tags or files.
 			// We safe some performance by building tag and file strings only here.
 			// We remove object and target since they aren't used for this type.
@@ -191,15 +191,16 @@ class Item extends BaseObject
 			if (array_key_exists('target', $row)) {
 				$row['target'] = '';
 			}
-			// Build the tag string out of the term entries
-			if (array_key_exists('tag', $row) && empty($row['tag'])) {
-				$row['tag'] = Term::tagTextFromItemId($row['internal-iid']);
-			}
+		}
 
-			// Build the file string out of the term entries
-			if (array_key_exists('file', $row) && empty($row['file'])) {
-				$row['file'] = Term::fileTextFromItemId($row['internal-iid']);
-			}
+		// Build the tag string out of the term entries
+		if (array_key_exists('tag', $row) && empty($row['tag'])) {
+			$row['tag'] = Term::tagTextFromItemId($row['internal-iid']);
+		}
+
+		// Build the file string out of the term entries
+		if (array_key_exists('file', $row) && empty($row['file'])) {
+			$row['file'] = Term::fileTextFromItemId($row['internal-iid']);
 		}
 
 		// Remove internal fields