enh(NavigationManager): Use ID as fallback for app property of entries

Signed-off-by: Ferdinand Thiessen <[email protected]>

Author: susnux

lib/private/NavigationManager.php

@@ -95,7 +95,10 @@ public function add($entry) {
 			return;
 		}
 
+		$id = $entry['id'];
+
 		$entry['active'] = false;
+		$entry['unread'] = $this->unreadCounters[$id] ?? 0;
 		if (!isset($entry['icon'])) {
 			$entry['icon'] = '';
 		}
@@ -105,9 +108,11 @@ public function add($entry) {
 		if (!isset($entry['type'])) {
 			$entry['type'] = 'link';
 		}
+		// app might not be set when using closures, in this case try to fallback to ID
+		if (!isset($entry['app']) && $this->appManager->isEnabledForUser($id)) {
+			$entry['app'] = $id;
+		}
 
-		$id = $entry['id'];
-		$entry['unread'] = $this->unreadCounters[$id] ?? 0;
 		if ($entry['type'] === 'link') {
 			// This is the default app that will always be shown first
 			$entry['default'] = ($entry['app'] ?? false) === $this->defaultApp;