Completed
Pull Request — master (#31175)
by Victor
10:22
created
lib/private/App/AppManager.php 1 patch
Doc Comments   +8 added lines, -5 removed lines patch added patch discarded remove patch
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 	/**
133 133
 	 * List all installed apps
134 134
 	 *
135
-	 * @return string[]
135
+	 * @return integer[]
136 136
 	 */
137 137
 	public function getInstalledApps() {
138 138
 		return array_keys($this->getInstalledAppsValues());
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
 	/**
242 242
 	 * Do not allow more than one active app-theme
243 243
 	 *
244
-	 * @param $appId
244
+	 * @param string $appId
245 245
 	 * @throws AppManagerException
246 246
 	 */
247 247
 	protected function canEnableTheme($appId) {
@@ -400,6 +400,9 @@  discard block
 block discarded – undo
400 400
 		return in_array($appId, $this->shippedApps);
401 401
 	}
402 402
 
403
+	/**
404
+	 * @param string $appId
405
+	 */
403 406
 	private function isAlwaysEnabled($appId) {
404 407
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
405 408
 		return in_array($appId, $alwaysEnabled);
@@ -428,7 +431,7 @@  discard block
 block discarded – undo
428 431
 	/**
429 432
 	 * @param string $package package path
430 433
 	 * @param bool $skipMigrations whether to skip migrations, which would only install the code
431
-	 * @return string|false app id or false in case of error
434
+	 * @return integer app id or false in case of error
432 435
 	 * @since 10.0
433 436
 	 */
434 437
 	public function installApp($package, $skipMigrations = false) {
@@ -441,7 +444,7 @@  discard block
 block discarded – undo
441 444
 
442 445
 	/**
443 446
 	 * @param string $package
444
-	 * @return mixed
447
+	 * @return boolean
445 448
 	 * @since 10.0
446 449
 	 */
447 450
 	public function updateApp($package) {
@@ -548,7 +551,7 @@  discard block
 block discarded – undo
548 551
 	 * both are relative to OC root directory and webroot
549 552
 	 *
550 553
 	 * @param string $appId
551
-	 * @return false|string[]
554
+	 * @return string
552 555
 	 */
553 556
 	protected function findAppInDirectories($appId) {
554 557
 		$sanitizedAppId = \OC_App::cleanAppId($appId);
Please login to merge, or discard this patch.