From 2d56664e3587214254c1fd35370b6854c0ad878b Mon Sep 17 00:00:00 2001
From: Daniel Rudolf <github.com@daniel-rudolf.de>
Date: Thu, 5 Sep 2019 18:35:40 +0200
Subject: Improve usage of IAppManager::getAppWebPath()

Deprecate \OC_App::getAppWebPath() and \OC_App::getAppPath()

Signed-off-by: Daniel Rudolf <github.com@daniel-rudolf.de>
---
 lib/private/App/AppManager.php | 2 +-
 lib/private/legacy/app.php     | 2 ++
 lib/public/App/IAppManager.php | 4 ++--
 3 files changed, 5 insertions(+), 3 deletions(-)

(limited to 'lib')

diff --git a/lib/private/App/AppManager.php b/lib/private/App/AppManager.php
index 6571386f5cf..19242245600 100644
--- a/lib/private/App/AppManager.php
+++ b/lib/private/App/AppManager.php
@@ -411,7 +411,7 @@ class AppManager implements IAppManager {
 	 * @return string
 	 * @throws AppPathNotFoundException if app path can't be found
 	 */
-	public function getAppWebPath($appId) {
+	public function getAppWebPath(string $appId): string {
 		$appWebPath = \OC_App::getAppWebPath($appId);
 		if($appWebPath === false) {
 			throw new AppPathNotFoundException('Could not find web path for ' . $appId);
diff --git a/lib/private/legacy/app.php b/lib/private/legacy/app.php
index 58b617aae45..9e47f9064d9 100644
--- a/lib/private/legacy/app.php
+++ b/lib/private/legacy/app.php
@@ -485,6 +485,7 @@ class OC_App {
 	 *
 	 * @param string $appId
 	 * @return string|false
+	 * @deprecated 11.0.0 use \OC::$server->getAppManager()->getAppPath()
 	 */
 	public static function getAppPath(string $appId) {
 		if ($appId === null || trim($appId) === '') {
@@ -503,6 +504,7 @@ class OC_App {
 	 *
 	 * @param string $appId
 	 * @return string|false
+	 * @deprecated 18.0.0 use \OC::$server->getAppManager()->getAppWebPath()
 	 */
 	public static function getAppWebPath(string $appId) {
 		if (($dir = self::findAppInDirectories($appId)) != false) {
diff --git a/lib/public/App/IAppManager.php b/lib/public/App/IAppManager.php
index c57b94382a7..391e3f0dd64 100644
--- a/lib/public/App/IAppManager.php
+++ b/lib/public/App/IAppManager.php
@@ -131,10 +131,10 @@ interface IAppManager {
 	 *
 	 * @param string $appId
 	 * @return string
-	 * @since 17.0.0
+	 * @since 18.0.0
 	 * @throws AppPathNotFoundException
 	 */
-	public function getAppWebPath($appId);
+	public function getAppWebPath(string $appId): string;
 
 	/**
 	 * List all apps enabled for a user
-- 
cgit v1.2.3