nextcloud/lib
Ferdinand Thiessen 1b4243f5a3
fix(NavigationManager): resolve entries only when needed
The `init` method previously contained two different logics:
1. It set up the internal state of default apps and app order
2. It resolved the app navigation entries

The 1. is needed before `add` can be called, so it was always called by
the `add` method, but this also resolved all appinfo.xml entries on the
first `add` call even if never used.
The 2. is only needed when the navigations are actually fetched.

This splits the logic into two functions:
- `init` for the bare initialization
- `resolveAppNavigationEntries` for resolving the entries when
  requesting to output them.

This should give a small performance improvement for API calls and fixes
a problem when navigations are added before all apps are registered.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2026-06-07 16:54:21 +02:00
..
composer chore: Remove unused and long deprecated Remote components 2026-06-03 10:36:41 +02:00
l10n fix(l10n): Update translations from Transifex 2026-06-06 00:22:55 +00:00
private fix(NavigationManager): resolve entries only when needed 2026-06-07 16:54:21 +02:00
public Merge pull request #60167 from nextcloud/enh/noid/allow-http-progressive-stream-response 2026-06-03 16:24:48 +02:00
unstable chore: Apply new coding standard to all files 2026-06-01 13:46:39 +02:00
base.php fix: make sure the core app is loaded 2026-06-02 22:51:05 +02:00
versioncheck.php ci(PHP): Test against 8.5 on CI 2025-12-19 15:34:34 +01:00