Only do the description kung-fu on strings - fixes #17028

1 files changed, 19 insertions, 15 deletions

diff --git a/lib/private/app.php b/lib/private/app.php
index 1a32fcfcf77..d5e07ffdbcc 100644
--- a/lib/private/app.php
+++ b/lib/private/app.php
@@ -1223,23 +1223,27 @@ class OC_App {
 		// just modify the description if it is available
 		// otherwise this will create a $data element with an empty 'description'
 		if (isset($data['description'])) {
-			// sometimes the description contains line breaks and they are then also
-			// shown in this way in the app management which isn't wanted as HTML
-			// manages line breaks itself
-
-			// first of all we split on empty lines
-			$paragraphs = preg_split("!\n[[:space:]]*\n!mu", $data['description']);
-
-			$result = [];
-			foreach ($paragraphs as $value) {
-				// replace multiple whitespace (tabs, space, newlines) inside a paragraph
-				// with a single space - also trims whitespace
-				$result[] = trim(preg_replace('![[:space:]]+!mu', ' ', $value));
-			}
+			if (is_string($data['description'])) {
+				// sometimes the description contains line breaks and they are then also
+				// shown in this way in the app management which isn't wanted as HTML
+				// manages line breaks itself
+
+				// first of all we split on empty lines
+				$paragraphs = preg_split("!\n[[:space:]]*\n!mu", $data['description']);
+
+				$result = [];
+				foreach ($paragraphs as $value) {
+					// replace multiple whitespace (tabs, space, newlines) inside a paragraph
+					// with a single space - also trims whitespace
+					$result[] = trim(preg_replace('![[:space:]]+!mu', ' ', $value));
+				}
 
-			// join the single paragraphs with a empty line in between
-			$data['description'] = implode("\n\n", $result);
+				// join the single paragraphs with a empty line in between
+				$data['description'] = implode("\n\n", $result);
 
+			} else {
+				$data['description'] = '';
+			}
 		}
 
 		return $data;