fix(Router): Load attribute routes of all apps when not app is specified

Signed-off-by: provokateurin <kate@provokateurin.de>

diff --git a/lib/private/Route/Router.php b/lib/private/Route/Router.php
index 91bb733bebb2e7b9bfa2942a541d705f38a7aad1..67ce930ce3e9047b6e48810e4ed1774ed70a07e1 100644 (file)
--- a/lib/private/Route/Router.php
+++ b/lib/private/Route/Router.php
@@ -140,9 +140,14 @@ class Router implements IRouter {
                if ($this->loaded) {
                        return;
                }
+               $this->eventLogger->start('route:load:' . $requestedApp, 'Loading Routes for ' . $requestedApp);
                if (is_null($app)) {
                        $this->loaded = true;
                        $routingFiles = $this->getRoutingFiles();
+
+                       foreach (\OC_App::getEnabledApps() as $enabledApp) {
+                               $this->loadAttributeRoutes($enabledApp);
+                       }
                } else {
                        if (isset($this->loadedApps[$app])) {
                                return;
@@ -154,21 +159,9 @@ class Router implements IRouter {
                        } else {
                                $routingFiles = [];
                        }
-               }
-               $this->eventLogger->start('route:load:' . $requestedApp, 'Loading Routes for ' . $requestedApp);
-
-               if ($requestedApp !== null && in_array($requestedApp, \OC_App::getEnabledApps())) {
-                       $routes = $this->getAttributeRoutes($requestedApp);
-                       if (count($routes) > 0) {
-                               $this->useCollection($requestedApp);
-                               $this->setupRoutes($routes, $requestedApp);
-                               $collection = $this->getCollection($requestedApp);
-                               $this->root->addCollection($collection);
 
-                               // Also add the OCS collection
-                               $collection = $this->getCollection($requestedApp . '.ocs');
-                               $collection->addPrefix('/ocsapp');
-                               $this->root->addCollection($collection);
+                       if (in_array($app, \OC_App::getEnabledApps())) {
+                               $this->loadAttributeRoutes($app);
                        }
                }
 
@@ -442,6 +435,23 @@ class Router implements IRouter {
                return $routeName;
        }
 
+       private function loadAttributeRoutes(string $app): void {
+               $routes = $this->getAttributeRoutes($app);
+               if (count($routes) === 0) {
+                       return;
+               }
+
+               $this->useCollection($app);
+               $this->setupRoutes($routes, $app);
+               $collection = $this->getCollection($app);
+               $this->root->addCollection($collection);
+
+               // Also add the OCS collection
+               $collection = $this->getCollection($app . '.ocs');
+               $collection->addPrefix('/ocsapp');
+               $this->root->addCollection($collection);
+       }
+
        /**
         * @throws ReflectionException
         */