Merge pull request #23542 from owncloud/catch-empty-app-path-oh-we-really-should-use-exceptions-in-the-future

getAppPath can return false

1 files changed, 6 insertions, 3 deletions

diff --git a/lib/private/route/router.php b/lib/private/route/router.php
index 03bee3d2037..ef02b5aba0e 100644
--- a/lib/private/route/router.php
+++ b/lib/private/route/router.php
@@ -95,9 +95,12 @@ class Router implements IRouter {
 		if (!isset($this->routingFiles)) {
 			$this->routingFiles = [];
 			foreach (\OC_APP::getEnabledApps() as $app) {
-				$file = \OC_App::getAppPath($app) . '/appinfo/routes.php';
-				if (file_exists($file)) {
-					$this->routingFiles[$app] = $file;
+				$appPath = \OC_App::getAppPath($app);
+				if($appPath !== false) {
+					$file = $appPath . '/appinfo/routes.php';
+					if (file_exists($file)) {
+						$this->routingFiles[$app] = $file;
+					}
 				}
 			}
 		}