Magestead\Command\Index\ReindexCommand

Complexity

Total Complexity

6

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	3
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	getCommand()	0	15	4
A	execute()	0	9	1

<?php namespace Magestead\Command\Index;

use Magestead\Command\ProcessCommand;
use Magestead\Helper\Config;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class ReindexCommand
 * @package Magestead\Command\Index
 */
class ReindexCommand extends Command
{
    protected $_config;
    protected $_projectPath;

    protected function configure()
    {
        $this->_projectPath = getcwd();
        $this->setName("index:reindex");
        $this->setDescription("Reindex data");
        $this->addArgument('index', InputArgument::OPTIONAL, '[indexer]');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return ProcessCommand
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $output->writeln('<info>Reindexing data</info>');
        $index = $input->getArgument('index');

        $command  = $this->getCommand(new Config($output), $index);
        $pCommand = "vagrant ssh -c '". $command ."'";
        return new ProcessCommand($pCommand, $this->_projectPath, $output);
    }

    /**
     * @param Config $config
     * @param $index
     * @return bool|string
     */
    protected function getCommand(Config $config, $index)
    {
        $type = $config->type;

The property type does not seem to exist in Magestead\Helper\Config.

        switch ($type) {
            case 'magento':
                $index = (!is_null($index)) ? ' '.$index : ':all';
                return "cd /var/www/public;../bin/n98-magerun.phar index:reindex$index";
                break;

break is not strictly necessary here and could be removed.

            case 'magento2':
                return "cd /var/www/public;bin/magento indexer:reindex $index";
                break;

break is not strictly necessary here and could be removed.

        }

        return false;
    }
}