Merge pull request #2757 from Icinga/bugfix/icingacli-module-disable-fails-2714

CLI: don't fail to disable an already disabled module
This commit is contained in:
Eric Lippmann 2017-03-28 10:13:06 +02:00
commit c64c6ee0b9
1 changed files with 11 additions and 1 deletions

View File

@ -3,6 +3,8 @@
namespace Icinga\Clicommands;
use Icinga\Application\Logger;
use Icinga\Application\Modules\Manager;
use Icinga\Cli\Command;
/**
@ -14,6 +16,9 @@ use Icinga\Cli\Command;
*/
class ModuleCommand extends Command
{
/**
* @var Manager
*/
protected $modules;
public function init()
@ -109,7 +114,12 @@ class ModuleCommand extends Command
if (! $module || $this->hasRemainingParams()) {
return $this->showUsage();
}
$this->modules->disableModule($module);
if ($this->modules->hasEnabled($module)) {
$this->modules->disableModule($module);
} else {
Logger::info('Module "%s" is already disabled', $module);
}
}
/**