Completed
Pull Request — master (#29747)
by Victor
09:49
created
lib/private/App/AppManager.php 1 patch
Doc Comments   +7 added lines, -4 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 	/**
122 122
 	 * List all installed apps
123 123
 	 *
124
-	 * @return string[]
124
+	 * @return integer[]
125 125
 	 */
126 126
 	public function getInstalledApps() {
127 127
 		return array_keys($this->getInstalledAppsValues());
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
 	/**
231 231
 	 * Do not allow more than one active app-theme
232 232
 	 *
233
-	 * @param $appId
233
+	 * @param string $appId
234 234
 	 * @throws \Exception
235 235
 	 */
236 236
 	protected function canEnableTheme($appId) {
@@ -389,6 +389,9 @@  discard block
 block discarded – undo
389 389
 		return in_array($appId, $this->shippedApps);
390 390
 	}
391 391
 
392
+	/**
393
+	 * @param string $appId
394
+	 */
392 395
 	private function isAlwaysEnabled($appId) {
393 396
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
394 397
 		return in_array($appId, $alwaysEnabled);
@@ -417,7 +420,7 @@  discard block
 block discarded – undo
417 420
 	/**
418 421
 	 * @param string $package package path
419 422
 	 * @param bool $skipMigrations whether to skip migrations, which would only install the code
420
-	 * @return string|false app id or false in case of error
423
+	 * @return integer app id or false in case of error
421 424
 	 * @since 10.0
422 425
 	 */
423 426
 	public function installApp($package, $skipMigrations = false) {
@@ -430,7 +433,7 @@  discard block
 block discarded – undo
430 433
 
431 434
 	/**
432 435
 	 * @param string $package
433
-	 * @return mixed
436
+	 * @return boolean
434 437
 	 * @since 10.0
435 438
 	 */
436 439
 	public function updateApp($package) {
Please login to merge, or discard this patch.