Use IAppManager instead of private API

Signed-off-by: Julius Härtl <jus@bitgrid.net>

1 files changed, 17 insertions, 7 deletions

diff --git a/apps/theming/lib/ThemingDefaults.php b/apps/theming/lib/ThemingDefaults.php
index 0a9d5a6cc1f..5dd22fb6326 100644
--- a/apps/theming/lib/ThemingDefaults.php
+++ b/apps/theming/lib/ThemingDefaults.php
@@ -23,6 +23,8 @@
 namespace OCA\Theming;
 
 
+use OCP\App\AppPathNotFoundException;
+use OCP\App\IAppManager;
 use OCP\Files\IAppData;
 use OCP\ICacheFactory;
 use OCP\IConfig;
@@ -41,6 +43,10 @@ class ThemingDefaults extends \OC_Defaults {
 	private $appData;
 	/** @var ICacheFactory */
 	private $cacheFactory;
+	/** @var Util */
+	private $util;
+	/** @var IAppManager */
+	private $appManager;
 	/** @var string */
 	private $name;
 	/** @var string */
@@ -49,8 +55,7 @@ class ThemingDefaults extends \OC_Defaults {
 	private $slogan;
 	/** @var string */
 	private $color;
-	/** @var Util */
-	private $util;
+
 	/** @var string */
 	private $iTunesAppId;
 	/** @var string */
@@ -68,13 +73,15 @@ class ThemingDefaults extends \OC_Defaults {
 	 * @param IAppData $appData
 	 * @param ICacheFactory $cacheFactory
 	 * @param Util $util
+	 * @param IAppManager $appManager
 	 */
 	public function __construct(IConfig $config,
 								IL10N $l,
 								IURLGenerator $urlGenerator,
 								IAppData $appData,
 								ICacheFactory $cacheFactory,
-								Util $util
+								Util $util,
+								IAppManager $appManager
 	) {
 		parent::__construct();
 		$this->config = $config;
@@ -83,6 +90,7 @@ class ThemingDefaults extends \OC_Defaults {
 		$this->appData = $appData;
 		$this->cacheFactory = $cacheFactory;
 		$this->util = $util;
+		$this->appManager = $appManager;
 
 		$this->name = parent::getName();
 		$this->url = parent::getBaseUrl();
@@ -269,10 +277,12 @@ class ThemingDefaults extends \OC_Defaults {
 			return $this->urlGenerator->linkToRoute('theming.Icon.getTouchIcon', ['app' => $app]) . '?v=' . $cacheBusterValue;
 		}
 		if ($image === 'manifest.json') {
-			$appPath = \OC_App::getAppPath($app);
-			if ($appPath && file_exists($appPath . '/img/manifest.json')) {
-				return false;
-			}
+			try {
+				$appPath = $this->appManager->getAppPath($app);
+				if (file_exists($appPath . '/img/manifest.json')) {
+					return false;
+				}
+			} catch (AppPathNotFoundException $e) {}
 			return $this->urlGenerator->linkToRoute('theming.Theming.getManifest') . '?v=' . $cacheBusterValue;
 		}
 		return false;