fix(Navigation): ensure NavManager is inited when adding an entry

[blizzz]

Summary

otherwise custom order cannot be applied

Needed by nextcloud/tables#904

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[Pytal]

Would adding as a closure entry be better? ref: 9708466

[blizzz]

Would adding as a closure entry be better? ref: 9708466

Ah, probably, but it does not make this fix less useful.

Even better would be an Event one could listen to. Right now I react to BeforeTemplateRenderedEvent, but it does only feel so right.

[Pytal]

Ah, probably, but it does not make this fix less useful.

Agreed!

Even better would be an Event one could listen to. Right now I react to BeforeTemplateRenderedEvent, but it does only feel so right.

👍

[blizzz]

☑️ Changed my code to supply a Closure – thanks!