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

getAppPath can return false
10 years ago · 789df4d630
parent a355e3abe3 117b3e0a12
commit 789df4d630
1 changed files with 6 additions and 3 deletions
--- 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;
+					}
 				}
 			}
 		}