@@ -121,7 +121,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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) { |