XoopsConsole\Commands\DeactivateModuleCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	execute()	0	23	3

<?php

namespace XoopsConsole\Commands;

use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputOption;
use Xoops\Core\XoopsTpl;

class DeactivateModuleCommand extends Command
{
    protected function configure()
    {
        $this->setName("deactivate-module")
            ->setDescription("Deactivate an installed module")
            ->setDefinition(array(
                new InputArgument('module', InputArgument::REQUIRED, 'Module directory name'),
            ))->setHelp(<<<EOT
The <info>deactivate-module</info> command deactivates a currently installed module.
EOT
            );
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $module = $input->getArgument('module');
        $output->writeln(sprintf('Deactivating %s', $module));

It seems like $module can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

        $xoops = \Xoops::getInstance();
        $moduleHandler = $xoops->getHandlerModule();
        $moduleObject = $moduleHandler->getByDirname($module);

It seems like $module can also be of type string[]; however, parameter $dirname of Xoops\Core\Kernel\Handlers\XoopsModuleHandler::getByDirname() does only seem to accept string, maybe add an additional type check?

        if (false === $moduleObject) {
            $output->writeln(sprintf('<error>%s is not an installed module!</error>', $module));
            return;
        }
        $moduleObject->setVar('isactive', false);
        $moduleHandler->insert($moduleObject, true);

        $blockHandler = $xoops->getHandlerBlock();
        $blocks = $blockHandler->getByModule($moduleObject->getVar('mid'));
        foreach ($blocks as $block) {
            /* @var $block \Xoops\Core\Kernel\Handlers\XoopsBlock */
            $block->setVar('isactive', false);
            $blockHandler->insert($block);
        }
        $output->writeln(sprintf('<info>Set %s module inactive</info>', $module));
        $xoops->cache()->delete('system');
    }
}