fix(NavigationManager): Remove break to return all defaultEntryIds

SimJoSt · backportbot[bot] · commit 8d87dfa1a779 · 2026-01-12T18:30:14.000Z
Otherwise only the first entry id is returned, breaking loading the value for the "Global Default App" option

Signed-off-by: Joda Stößer &lt;git@simjo.st&gt;
diff --git a/lib/private/NavigationManager.php b/lib/private/NavigationManager.php
@@ -474,7 +474,6 @@ public function getDefaultEntryIds(bool $withFallbacks = true): array {
 		foreach ($storedIds as $id) {
 			if (in_array($id, $entryIds, true)) {
 				$ids[] = $id;
-				break;
 			}
 		}
 		return array_filter($ids);

Original file line number	Diff line number	Diff line change
`@@ -474,7 +474,6 @@ public function getDefaultEntryIds(bool $withFallbacks = true): array {`
`474`	`474`	`foreach ($storedIds as $id) {`
`475`	`475`	`if (in_array($id, $entryIds, true)) {`
`476`	`476`	`$ids[] = $id;`
`477`		`- break;`
`478`	`477`	`}`
`479`	`478`	`}`
`480`	`479`	`return array_filter($ids);`