Merge pull request #30144 from owncloud/fix-30112

Do not cache app path if app is not found

lib/private/App/AppManager.php

@@ -561,11 +561,24 @@ protected function findAppInDirectories($appId) {
 					$versionToLoad['url'] .= '/' . $appId;
 				}
 			}
-			$this->appDirs[$appId] = empty($versionToLoad) ? false : $versionToLoad;
+
+			if (empty($versionToLoad)) {
+				return false;
+			}
+			$this->saveAppPath($appId, $versionToLoad);
 		}
 		return $this->appDirs[$appId];
 	}
 
+	/**
+	 * Save app path and webPath to internal cache
+	 * @param string $appId
+	 * @param string[] $appData
+	 */
+	protected function saveAppPath($appId, $appData) {
+		$this->appDirs[$appId] = $appData;
+	}
+
 	/**
 	 * Get apps roots as an array of path and url
 	 * Wrapper for easy mocking

tests/lib/App/ManagerTest.php

@@ -510,4 +510,23 @@ public function appInfoDataProvider() {
 			[ '2.2.3', '2.2.1', true ]
 		];
 	}
+
+	public function testPathIsNotCachedForNotFoundApp() {
+		$appId = 'notexistingapp';
+
+		$appManager = $this->getMockBuilder(AppManager::class)
+			->setMethods(['getAppVersionByPath', 'getAppRoots', 'saveAppPath'])
+			->disableOriginalConstructor()
+			->getMock();
+
+		$appManager->expects($this->any())
+			->method('getAppRoots')
+			->willReturn([]);
+
+		$appManager->expects($this->never())
+			->method('saveAppPath');
+
+		$appPath = $appManager->getAppPath($appId);
+		$this->assertFalse($appPath);
+	}
 }