Scrutinizer php fixes

This commit is contained in:
Dennis Eichhorn 2017-11-08 22:00:34 +01:00
parent da5ede0f9b
commit 433b3b4d66
2 changed files with 7 additions and 10 deletions

View File

@ -242,6 +242,7 @@ class ModuleManager
*/ */
public function getAvailableModules() : array public function getAvailableModules() : array
{ {
return [];
} }
/** /**
@ -410,13 +411,13 @@ class ModuleManager
* *
* @param string $module Module name * @param string $module Module name
* *
* @return bool * @return void
* *
* @throws InvalidModuleException Throws this exception in case the installer doesn't exist * @throws InvalidModuleException Throws this exception in case the installer doesn't exist
* *
* @since 1.0.0 * @since 1.0.0
*/ */
public function reInit(string $module) : bool public function reInit(string $module) /* : void */
{ {
$info = $this->loadInfo($module); $info = $this->loadInfo($module);
$class = '\\Modules\\' . $info->getDirectory() . '\\Admin\\Installer'; $class = '\\Modules\\' . $info->getDirectory() . '\\Admin\\Installer';

View File

@ -290,24 +290,20 @@ class MultiMap implements \Countable
* *
* @param mixed $key Key used to identify value * @param mixed $key Key used to identify value
* *
* @return bool * @return void
* *
* @since 1.0.0 * @since 1.0.0
*/ */
private function removeMultiple($key) : bool private function removeMultiple($key) /* : void */
{ {
if ($this->orderType === OrderType::LOOSE) { if ($this->orderType === OrderType::LOOSE) {
$keys = Permutation::permut($key); $keys = Permutation::permut($key);
$removed = false;
foreach ($keys as $key => $value) { foreach ($keys as $key => $value) {
$removed |= $this->remove(implode(':', $value)); $this->remove(implode(':', $value));
} }
return $removed;
} else { } else {
return $this->remove(implode(':', $key)); $this->remove(implode(':', $key));
} }
} }