ConfigureActionCommand::processInputs() - Code Metrics - Inspection of "Add targetPath for config output" - Niktux/karma - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#96)

by Sébastien

created 2016-06-01 20:23 UTC

ConfigureActionCommand::processInputs() C

↳ Parent: ConfigureActionCommand

Complexity

Conditions	9
Paths	68

Size

Total Lines	64
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	1	Features	1

Metric	Value
c	3
b	1
f	1
dl	0
loc	64
rs	6.5449
cc	9
eloc	34
nc	68
nop	1

How to fix Long Method

<?php

namespace Karma\Command;

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Input\InputOption;
use Karma\Application;
use Karma\Command;
use Karma\Configuration\FilterInputVariable;
use Karma\ConfigurableProcessor;

abstract class ConfigureActionCommand extends Command
{
    use FilterInputVariable;

    const
        ENV_DEV = 'dev',
        OPTION_ASSIGNMENT = '=';

    private
        $dryRun,

        $isBackupEnabled,
        $systemEnvironment,
        $name,

        $description,

        $outputTitle;

    protected
        $environment;

    public function __construct(Application $app, $name, $description, $outputTitle)
    {
        $this->name = $name;
        $this->description = $description;
        $this->outputTitle = $outputTitle;

        parent::__construct($app);

        $this->dryRun = false;
        $this->isBackupEnabled = false;

        $this->environment = self::ENV_DEV;
        $this->systemEnvironment = null;
    }

    protected function configure()
    {
        parent::configure();

        $this
            ->setName($this->name)
            ->setDescription($this->description)

            ->addArgument('sourcePath', InputArgument::OPTIONAL | InputArgument::IS_ARRAY, 'source path to hydrate/generate')

            ->addOption('targetPath', 't', InputOption::VALUE_REQUIRED, 'target path to hydrate/generate', null)
            ->addOption('env', 'e', InputOption::VALUE_REQUIRED, 'Target environment', self::ENV_DEV)
            ->addOption('system', 's', InputOption::VALUE_REQUIRED, 'Target environment for system variables', null)
            ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Simulation mode')
            ->addOption('backup', 'b', InputOption::VALUE_NONE, 'Backup overwritten files')
            ->addOption('override', 'o', InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED, 'Override variable values', array())
            ->addOption('data', 'd', InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED, 'Custom data values', array())
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        parent::execute($input, $output);

        $this->processInputs($input);

        $processor = $this->getProcessor();
        $this->configureProcessor($processor);
        $this->launchConfigurationAction($processor);
    }

    private function processInputs(InputInterface $input)
    {
        $this->environment = $input->getOption('env');
        $this->systemEnvironment = $input->getOption('system');

        if($input->getOption('dry-run'))
        {
            $this->dryRun = true;
            $this->output->writeln("<fg=cyan>Run in dry-run mode</fg=cyan>");
        }

        if($input->getOption('backup'))
        {
            $this->isBackupEnabled = true;
            $this->output->writeln("<fg=cyan>Backup enabled</fg=cyan>");
        }

        $profile = $this->app['profile'];
        $sourcePath = $input->getArgument('sourcePath');
        if(empty($sourcePath))
        {
            if($profile->hasSourcePath() !== true)
            {
                throw new \RuntimeException('Missing argument sourcePath');
            }

            $sourcePath = $profile->getSourcePath();
        }

        if(! is_array($sourcePath))
        {
            $sourcePath = array($sourcePath);
        }

        $targetPath = $input->getOption('targetPath');

        if(empty($targetPath) && $profile->hasTargetPath() === true)
        {
            $targetPath = $profile->getTargetPath();
        }

        $this->output->writeln(sprintf(
            "<info>%s <comment>%s</comment> with <comment>%s</comment> values</info>",
            $this->outputTitle,
            implode(' ', $sourcePath),
            $this->environment
        ));
        $this->output->writeln('');

        $this->app['sources.path'] = $sourcePath;

        $this->app['target.path'] = $targetPath;
        if(empty($targetPath))
        {
            $this->app['target.path'] = $sourcePath;
        }

        $this->processOverridenVariables(
            $this->parseOptionWithAssignments($input, 'override')
        );
        $this->processCustomData(
            $this->parseOptionWithAssignments($input, 'data')
        );
    }

    abstract protected function launchConfigurationAction(ConfigurableProcessor $processor);
    abstract protected function getProcessor();

    private function configureProcessor(ConfigurableProcessor $processor)
    {
        if($this->dryRun === true)
        {
            $processor->setDryRun();
        }

        if($this->isBackupEnabled === true)
        {
            $processor->enableBackup();
        }

        if($this->systemEnvironment !== null)
        {
            $processor->setSystemEnvironment($this->systemEnvironment);

             $this->app['logger']->info(sprintf(
                'Hydrate <important>system</important> variables with <important>%s</important> values',
                $this->systemEnvironment
            ));
        }
    }

    private function parseOptionWithAssignments(InputInterface $input, $optionName)
    {
        $strings = $input->getOption($optionName);

        if(! is_array($strings))
        {
            $strings = array($strings);
        }

        $data = array();

        foreach($strings as $string)
        {
            if(stripos($string, self::OPTION_ASSIGNMENT) === false)
            {
                throw new \InvalidArgumentException(sprintf(
                    '%s option must contain %c : --%s <variable>=<value>',
                    $optionName,
                    self::OPTION_ASSIGNMENT,
                    $optionName
                ));
            }

            list($variable, $value) = explode(self::OPTION_ASSIGNMENT, $string, 2);

            if(array_key_exists($variable, $data))
            {
                throw new \InvalidArgumentException("Duplicated %s option value : $variable");
            }

            $data[$variable] = $value;
        }

        return $data;
    }

    private function processOverridenVariables(array $overrides)
    {
        $reader = $this->app['configuration'];
        $logger = $this->app['logger'];

        foreach($overrides as $variable => $value)
        {
            $logger->info(sprintf(
               'Override <important>%s</important> with value <important>%s</important>',
               $variable,
               $value
            ));

            $value = $this->parseList($value);

            $reader->overrideVariable($variable, $this->filterValue($value));
        }
    }

    private function processCustomData(array $data)
    {
        $reader = $this->app['configuration'];
        $logger = $this->app['logger'];

        foreach($data as $variable => $value)
        {
            $logger->info(sprintf(
               'Set custom data <important>%s</important> with value <important>%s</important>',
               $variable,
               $value
            ));

            $reader->setCustomData($variable, $this->filterValue($value));
        }
    }
}


1			<?php
2
3			namespace Karma\Command;
4
5			use Symfony\Component\Console\Input\InputArgument;
6			use Symfony\Component\Console\Input\InputInterface;
7			use Symfony\Component\Console\Output\OutputInterface;
8			use Symfony\Component\Console\Input\InputOption;
9			use Karma\Application;
10			use Karma\Command;
11			use Karma\Configuration\FilterInputVariable;
12			use Karma\ConfigurableProcessor;
13
14			abstract class ConfigureActionCommand extends Command
15			{
16			use FilterInputVariable;
17
18			const
19			ENV_DEV = 'dev',
20			OPTION_ASSIGNMENT = '=';
21
22			private
23			$dryRun,
			0 ignored issues – show Coding Style introduced 2015-12-04 15:18 UTC by Report Bug Copy Issue Report It is generally advisable to only define one property per statement. Only declaring a single property per statement allows you to later on add doc comments more easily. It is also recommended by PSR2, so it is a common style that many people expect. Loading history...
24			$isBackupEnabled,
25			$systemEnvironment,
26			$name,
			0 ignored issues – show Comprehensibility introduced 2015-12-04 15:18 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
27			$description,
			0 ignored issues – show Comprehensibility introduced 2015-12-04 15:18 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
28			$outputTitle;
29
30			protected
31			$environment;
32
33			public function __construct(Application $app, $name, $description, $outputTitle)
34			{
35			$this->name = $name;
36			$this->description = $description;
37			$this->outputTitle = $outputTitle;
38
39			parent::__construct($app);
40
41			$this->dryRun = false;
42			$this->isBackupEnabled = false;
43
44			$this->environment = self::ENV_DEV;
45			$this->systemEnvironment = null;
46			}
47
48			protected function configure()
49			{
50			parent::configure();
51
52			$this
53			->setName($this->name)
54			->setDescription($this->description)
55
56			->addArgument('sourcePath', InputArgument::OPTIONAL \| InputArgument::IS_ARRAY, 'source path to hydrate/generate')
57
58			->addOption('targetPath', 't', InputOption::VALUE_REQUIRED, 'target path to hydrate/generate', null)
59			->addOption('env', 'e', InputOption::VALUE_REQUIRED, 'Target environment', self::ENV_DEV)
60			->addOption('system', 's', InputOption::VALUE_REQUIRED, 'Target environment for system variables', null)
61			->addOption('dry-run', null, InputOption::VALUE_NONE, 'Simulation mode')
62			->addOption('backup', 'b', InputOption::VALUE_NONE, 'Backup overwritten files')
63			->addOption('override', 'o', InputOption::VALUE_IS_ARRAY \| InputOption::VALUE_REQUIRED, 'Override variable values', array())
64			->addOption('data', 'd', InputOption::VALUE_IS_ARRAY \| InputOption::VALUE_REQUIRED, 'Custom data values', array())
65			;
66			}
67
68			protected function execute(InputInterface $input, OutputInterface $output)
69			{
70			parent::execute($input, $output);
71
72			$this->processInputs($input);
73
74			$processor = $this->getProcessor();
75			$this->configureProcessor($processor);
76			$this->launchConfigurationAction($processor);
77			}
78
79			private function processInputs(InputInterface $input)
80			{
81			$this->environment = $input->getOption('env');
82			$this->systemEnvironment = $input->getOption('system');
83
84			if($input->getOption('dry-run'))
85			{
86			$this->dryRun = true;
87			$this->output->writeln("<fg=cyan>Run in dry-run mode</fg=cyan>");
88			}
89
90			if($input->getOption('backup'))
91			{
92			$this->isBackupEnabled = true;
93			$this->output->writeln("<fg=cyan>Backup enabled</fg=cyan>");
94			}
95
96			$profile = $this->app['profile'];
97			$sourcePath = $input->getArgument('sourcePath');
98			if(empty($sourcePath))
99			{
100			if($profile->hasSourcePath() !== true)
101			{
102			throw new \RuntimeException('Missing argument sourcePath');
103			}
104
105			$sourcePath = $profile->getSourcePath();
106			}
107
108			if(! is_array($sourcePath))
109			{
110			$sourcePath = array($sourcePath);
111			}
112
113			$targetPath = $input->getOption('targetPath');
114
115			if(empty($targetPath) && $profile->hasTargetPath() === true)
116			{
117			$targetPath = $profile->getTargetPath();
118			}
119
120			$this->output->writeln(sprintf(
121			"<info>%s <comment>%s</comment> with <comment>%s</comment> values</info>",
122			$this->outputTitle,
123			implode(' ', $sourcePath),
124			$this->environment
125			));
126			$this->output->writeln('');
127
128			$this->app['sources.path'] = $sourcePath;
129
130			$this->app['target.path'] = $targetPath;
131			if(empty($targetPath))
132			{
133			$this->app['target.path'] = $sourcePath;
134			}
135
136			$this->processOverridenVariables(
137			$this->parseOptionWithAssignments($input, 'override')
138			);
139			$this->processCustomData(
140			$this->parseOptionWithAssignments($input, 'data')
141			);
142			}
143
144			abstract protected function launchConfigurationAction(ConfigurableProcessor $processor);
145			abstract protected function getProcessor();
146
147			private function configureProcessor(ConfigurableProcessor $processor)
148			{
149			if($this->dryRun === true)
150			{
151			$processor->setDryRun();
152			}
153
154			if($this->isBackupEnabled === true)
155			{
156			$processor->enableBackup();
157			}
158
159			if($this->systemEnvironment !== null)
160			{
161			$processor->setSystemEnvironment($this->systemEnvironment);
162
163			$this->app['logger']->info(sprintf(
164			'Hydrate <important>system</important> variables with <important>%s</important> values',
165			$this->systemEnvironment
166			));
167			}
168			}
169
170			private function parseOptionWithAssignments(InputInterface $input, $optionName)
171			{
172			$strings = $input->getOption($optionName);
173
174			if(! is_array($strings))
175			{
176			$strings = array($strings);
177			}
178
179			$data = array();
180
181			foreach($strings as $string)
182			{
183			if(stripos($string, self::OPTION_ASSIGNMENT) === false)
184			{
185			throw new \InvalidArgumentException(sprintf(
186			'%s option must contain %c : --%s <variable>=<value>',
187			$optionName,
188			self::OPTION_ASSIGNMENT,
189			$optionName
190			));
191			}
192
193			list($variable, $value) = explode(self::OPTION_ASSIGNMENT, $string, 2);
194
195			if(array_key_exists($variable, $data))
196			{
197			throw new \InvalidArgumentException("Duplicated %s option value : $variable");
198			}
199
200			$data[$variable] = $value;
201			}
202
203			return $data;
204			}
205
206			private function processOverridenVariables(array $overrides)
207			{
208			$reader = $this->app['configuration'];
209			$logger = $this->app['logger'];
210
211			foreach($overrides as $variable => $value)
212			{
213			$logger->info(sprintf(
214			'Override <important>%s</important> with value <important>%s</important>',
215			$variable,
216			$value
217			));
218
219			$value = $this->parseList($value);
220
221			$reader->overrideVariable($variable, $this->filterValue($value));
222			}
223			}
224
225			private function processCustomData(array $data)
226			{
227			$reader = $this->app['configuration'];
228			$logger = $this->app['logger'];
229
230			foreach($data as $variable => $value)
231			{
232			$logger->info(sprintf(
233			'Set custom data <important>%s</important> with value <important>%s</important>',
234			$variable,
235			$value
236			));
237
238			$reader->setCustomData($variable, $this->filterValue($value));
239			}
240			}
241			}
242

Niktux / karma

Pull Request — master (#96)

ConfigureActionCommand::processInputs() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like