ConfigureCommand::hasValidateUserInput() - Code Metrics - Inspection of "Merge pull request #77 from msiebeneicher/feature/..." - trivago/chapi - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 151e35...1ba630 )

by Marc

created 2017-03-07 13:28 UTC

ConfigureCommand::hasValidateUserInput() A

↳ Parent: ConfigureCommand

Complexity

Conditions	4
Paths	3

Size

Total Lines	13
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	6
cts	6
cp	1
rs	9.2
c	0
b	0
f	0
cc	4
eloc	6
nc	3
nop	1
crap	4

<?php
/**
 * @package: chapi
 *
 * @author:  msiebeneicher
 * @since:   2015-07-28
 *
 */

namespace Chapi\Commands;

use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Question\Question;
use Symfony\Component\Filesystem\Filesystem;
use Symfony\Component\Yaml\Dumper;
use Symfony\Component\Yaml\Parser;

class ConfigureCommand extends AbstractCommand
{
    /**
     * Configures the current command.
     */
    protected function configure()
    {
        $this->setName('configure')
            ->setDescription('Configure application and add necessary configs')
            ->addOption('cache_dir', 'd', InputOption::VALUE_OPTIONAL, 'Path to cache directory')

            ->addOption('chronos_url', 'u', InputOption::VALUE_OPTIONAL, 'The chronos url (inclusive port)', '')
            ->addOption('chronos_http_username', 'un', InputOption::VALUE_OPTIONAL, 'The chronos username (HTTP credentials)', '')
            ->addOption('chronos_http_password', 'p', InputOption::VALUE_OPTIONAL, 'The chronos password (HTTP credentials)', '')
            ->addOption('repository_dir', 'r', InputOption::VALUE_OPTIONAL, 'Root path to your job files', '')

            ->addOption('marathon_url', 'mu', InputOption::VALUE_OPTIONAL, 'The marathon url (inclusive port)', '')
            ->addOption('marathon_http_username', 'mun', InputOption::VALUE_OPTIONAL, 'The marathon username (HTTP credentials)', '')
            ->addOption('marathon_http_password', 'mp', InputOption::VALUE_OPTIONAL, 'The marathon password (HTTP credentials)', '')
            ->addOption('repository_dir_marathon', 'mr', InputOption::VALUE_OPTIONAL, 'Root path to the app files', '')
        ;
    }

    /**
     * @param InputInterface $oInput
     * @param OutputInterface $oOutput
     * @return int
     */
    protected function execute(InputInterface $oInput, OutputInterface $oOutput)
    {
        $this->oInput = $oInput;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->oOutput = $oOutput;

        return $this->process();
    }

    /**
     * @return int
     */
    protected function process()
    {
        $_aParams = $this->getInputValues();

        if ($this->hasValidateUserInput($_aParams))
        {
            $this->saveParameters($_aParams);
            return 0;
        }

        return 1;
    }

    /**
     * @return array<string,array<string,string|boolean>>
     */
    private function getInputValues()
    {
        $_aResult = [];

        $_aResult['cache_dir'] = [
            'value' => $this->getInputValue('cache_dir', '[GLOBAL] Please enter a cache directory'),
            'required' => true
        ];

        $_aResult['chronos_url'] = [
            'value' => $this->getInputValue('chronos_url', '[CHRONOS] Please enter the chronos url (inclusive port)'),
            'required' => false
        ];

        $_aResult['chronos_http_username'] = [
            'value' => $this->getInputValue('chronos_http_username', '[CHRONOS] Please enter the username to access your chronos instance'),
            'required' => false
        ];

        $_aResult['chronos_http_password'] = [
            'value' => $this->getInputValue('chronos_http_password', '[CHRONOS] Please enter the password to access your chronos instance', true),
            'required' => false
        ];

        $_aResult['repository_dir'] = [
            'value' => $this->getInputValue('repository_dir', '[CHRONOS] Please enter absolute path to your local chronos jobs configurations'),
            'required' => false
        ];

        $_aResult['marathon_url'] = [
            'value' => $this->getInputValue('marathon_url', '[MARATHON] Please enter the marathon url (inclusive port)'),
            'required' => false
        ];

        $_aResult['marathon_http_username'] = [
            'value' => $this->getInputValue('marathon_http_username', '[MARATHON] Please enter the username to access marathon instance'),
            'required' => false
        ];

        $_aResult['marathon_http_password'] = [
            'value' => $this->getInputValue('marathon_http_password', '[MARATHON] Please enter the password to access marathon instance', true),
            'required' => false
        ];

        $_aResult['repository_dir_marathon'] = [
            'value' => $this->getInputValue('repository_dir_marathon', '[MARATHON] Please enter absolute path to your local marathon tasks configurations'),
            'required' => false
        ];

        return $_aResult;
    }

    /**
     * @param string $sValueKey
     * @param string $sQuestion
     * @param boolean $bHiddenAnswer
     * @return string
     */
    private function getInputValue($sValueKey, $sQuestion, $bHiddenAnswer = false)
    {
        $_sValue = $this->oInput->getOption($sValueKey);
        if (empty($_sValue))
        {
            $_sValue = $this->printQuestion(
                $sQuestion,
                $this->getParameterValue($sValueKey),
                $bHiddenAnswer
            );
        }

        return $_sValue;
    }

    /**
     * @param array $aUserInput
     */
    private function saveParameters(array $aUserInput)
    {
        // We implemented an additional level of information
        // into the user input array: Is this field required or not?
        // To be backwards compatible we only store the value of
        // the question in the dump file.
        // With this loop we get rid of the "required" information
        // from getInputValues().
        $aToStore = [];
        foreach ($aUserInput as $key => $value)
        {
            $aToStore[$key] = $value['value'];
        }

        $_oDumper = new Dumper();
        $_sYaml = $_oDumper->dump(array('parameters' => $aToStore), 2);
$a = "a";
$ab = "ab";
$abc = "abc";

        $_oFileSystem = new Filesystem();
        $_oFileSystem->dumpFile(
            $this->getHomeDir() . DIRECTORY_SEPARATOR . $this->getParameterFileName(),
            $_sYaml
        );
    }

    /**
     * @param array $aUserInput
     * @return bool
     */
    private function hasValidateUserInput(array $aUserInput)
    {
        foreach ($aUserInput as $_sKey => $_sValue)
        {
            if ($_sValue['required'] == true && empty($_sValue['value']))
            {
                $this->oOutput->writeln(sprintf('<error>Please add a valid value for parameter "%s"</error>', $_sKey));
                return false;
            }
        }

        return true;
    }

    /**
     * @param string $sKey
     * @param mixed $mDefaultValue
     * @return mixed
     */
    private function getParameterValue($sKey, $mDefaultValue = null)
    {
        $_oParser = new Parser();
$a = "a";
$ab = "ab";
$abc = "abc";
        $_sParameterFile = $this->getHomeDir() . DIRECTORY_SEPARATOR . $this->getParameterFileName();

        if (file_exists($_sParameterFile))
        {
            $_aParameters = $_oParser->parse(
                file_get_contents($_sParameterFile)
            );

            if (isset($_aParameters['parameters']) && isset($_aParameters['parameters'][$sKey]))
            {
                return $_aParameters['parameters'][$sKey];
            }
        }

        return $mDefaultValue;
    }

    /**
     * @param string $sQuestion
     * @param null|mixed $mDefaultValue
     * @param boolean $bHiddenAnswer
     * @return mixed
     */
    private function printQuestion($sQuestion, $mDefaultValue = null, $bHiddenAnswer = false)
    {
        $_oHelper = $this->getHelper('question');

        // If we have a hidden answer and the default value is not empty
        // the we will set it as empty, because we don`t want to show
        // the default value on the terminal.
        // We know that the user has to enter the password again
        // if he / she want to reconfigure something. But this
        // is an acceptable tradeoff.
        if ($bHiddenAnswer === true && !empty($mDefaultValue))
        {
            $mDefaultValue = null;

        }

        $_sFormat = (!empty($mDefaultValue)) ? '<comment>%s (default: %s):</comment>' : '<comment>%s:</comment>';
$a = "a";
$ab = "ab";
$abc = "abc";
        $_oQuestion = new Question(sprintf($_sFormat, $sQuestion, $mDefaultValue), $mDefaultValue);

        // Sensitive information (like passwords) should not be
        // visible during the configuration wizard
        if ($bHiddenAnswer === true)
        {
            $_oQuestion->setHidden(true);
            $_oQuestion->setHiddenFallback(false);
        }

        return $_oHelper->ask($this->oInput, $this->oOutput, $_oQuestion);
    }
}

1		<?php
2		/**
3		* @package: chapi
4		*
5		* @author: msiebeneicher
6		* @since: 2015-07-28
7		*
8		*/
9
10		namespace Chapi\Commands;
11
12		use Symfony\Component\Console\Input\InputInterface;
13		use Symfony\Component\Console\Input\InputOption;
14		use Symfony\Component\Console\Output\OutputInterface;
15		use Symfony\Component\Console\Question\Question;
16		use Symfony\Component\Filesystem\Filesystem;
17		use Symfony\Component\Yaml\Dumper;
18		use Symfony\Component\Yaml\Parser;
19
20		class ConfigureCommand extends AbstractCommand
21		{
22		/**
23		* Configures the current command.
24		*/
25	4	protected function configure()
26		{
27	4	$this->setName('configure')
28	4	->setDescription('Configure application and add necessary configs')
29	4	->addOption('cache_dir', 'd', InputOption::VALUE_OPTIONAL, 'Path to cache directory')
30
31	4	->addOption('chronos_url', 'u', InputOption::VALUE_OPTIONAL, 'The chronos url (inclusive port)', '')
32	4	->addOption('chronos_http_username', 'un', InputOption::VALUE_OPTIONAL, 'The chronos username (HTTP credentials)', '')
33	4	->addOption('chronos_http_password', 'p', InputOption::VALUE_OPTIONAL, 'The chronos password (HTTP credentials)', '')
34	4	->addOption('repository_dir', 'r', InputOption::VALUE_OPTIONAL, 'Root path to your job files', '')
35
36	4	->addOption('marathon_url', 'mu', InputOption::VALUE_OPTIONAL, 'The marathon url (inclusive port)', '')
37	4	->addOption('marathon_http_username', 'mun', InputOption::VALUE_OPTIONAL, 'The marathon username (HTTP credentials)', '')
38	4	->addOption('marathon_http_password', 'mp', InputOption::VALUE_OPTIONAL, 'The marathon password (HTTP credentials)', '')
39	4	->addOption('repository_dir_marathon', 'mr', InputOption::VALUE_OPTIONAL, 'Root path to the app files', '')
40		;
41	4	}
42
43		/**
44		* @param InputInterface $oInput
45		* @param OutputInterface $oOutput
46		* @return int
47		*/
48	4	protected function execute(InputInterface $oInput, OutputInterface $oOutput)
49		{
50	4	$this->oInput = $oInput;
		0 ignored issues – show Coding Style introduced 2017-03-02 10:41 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
51	4	$this->oOutput = $oOutput;
52
53	4	return $this->process();
54		}
55
56		/**
57		* @return int
58		*/
59	4	protected function process()
60		{
61	4	$_aParams = $this->getInputValues();
62
63	4	if ($this->hasValidateUserInput($_aParams))
64		{
65	3	$this->saveParameters($_aParams);
66	3	return 0;
67		}
68
69	1	return 1;
70		}
71
72		/**
73		* @return array<string,array<string,string\|boolean>>
74		*/
75	4	private function getInputValues()
76		{
77	4	$_aResult = [];
78
79	4	$_aResult['cache_dir'] = [
80	4	'value' => $this->getInputValue('cache_dir', '[GLOBAL] Please enter a cache directory'),
81		'required' => true
82		];
83
84	4	$_aResult['chronos_url'] = [
85	4	'value' => $this->getInputValue('chronos_url', '[CHRONOS] Please enter the chronos url (inclusive port)'),
86		'required' => false
87		];
88
89	4	$_aResult['chronos_http_username'] = [
90	4	'value' => $this->getInputValue('chronos_http_username', '[CHRONOS] Please enter the username to access your chronos instance'),
91		'required' => false
92		];
93
94	4	$_aResult['chronos_http_password'] = [
95	4	'value' => $this->getInputValue('chronos_http_password', '[CHRONOS] Please enter the password to access your chronos instance', true),
96		'required' => false
97		];
98
99	4	$_aResult['repository_dir'] = [
100	4	'value' => $this->getInputValue('repository_dir', '[CHRONOS] Please enter absolute path to your local chronos jobs configurations'),
101		'required' => false
102		];
103
104	4	$_aResult['marathon_url'] = [
105	4	'value' => $this->getInputValue('marathon_url', '[MARATHON] Please enter the marathon url (inclusive port)'),
106		'required' => false
107		];
108
109	4	$_aResult['marathon_http_username'] = [
110	4	'value' => $this->getInputValue('marathon_http_username', '[MARATHON] Please enter the username to access marathon instance'),
111		'required' => false
112		];
113
114	4	$_aResult['marathon_http_password'] = [
115	4	'value' => $this->getInputValue('marathon_http_password', '[MARATHON] Please enter the password to access marathon instance', true),
116		'required' => false
117		];
118
119	4	$_aResult['repository_dir_marathon'] = [
120	4	'value' => $this->getInputValue('repository_dir_marathon', '[MARATHON] Please enter absolute path to your local marathon tasks configurations'),
121		'required' => false
122		];
123
124	4	return $_aResult;
125		}
126
127		/**
128		* @param string $sValueKey
129		* @param string $sQuestion
130		* @param boolean $bHiddenAnswer
131		* @return string
132		*/
133	4	private function getInputValue($sValueKey, $sQuestion, $bHiddenAnswer = false)
134		{
135	4	$_sValue = $this->oInput->getOption($sValueKey);
136	4	if (empty($_sValue))
137		{
138	3	$_sValue = $this->printQuestion(
139		$sQuestion,
140	3	$this->getParameterValue($sValueKey),
141		$bHiddenAnswer
142		);
143		}
144
145	4	return $_sValue;
146		}
147
148		/**
149		* @param array $aUserInput
150		*/
151	3	private function saveParameters(array $aUserInput)
152		{
153		// We implemented an additional level of information
154		// into the user input array: Is this field required or not?
155		// To be backwards compatible we only store the value of
156		// the question in the dump file.
157		// With this loop we get rid of the "required" information
158		// from getInputValues().
159	3	$aToStore = [];
160	3	foreach ($aUserInput as $key => $value)
161		{
162	3	$aToStore[$key] = $value['value'];
163		}
164
165	3	$_oDumper = new Dumper();
166	3	$_sYaml = $_oDumper->dump(array('parameters' => $aToStore), 2);
		0 ignored issues – show Coding Style introduced 2017-03-02 10:41 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
167
168	3	$_oFileSystem = new Filesystem();
169	3	$_oFileSystem->dumpFile(
170	3	$this->getHomeDir() . DIRECTORY_SEPARATOR . $this->getParameterFileName(),
171		$_sYaml
172		);
173	3	}
174
175		/**
176		* @param array $aUserInput
177		* @return bool
178		*/
179	4	private function hasValidateUserInput(array $aUserInput)
180		{
181	4	foreach ($aUserInput as $_sKey => $_sValue)
182		{
183	4	if ($_sValue['required'] == true && empty($_sValue['value']))
184		{
185	1	$this->oOutput->writeln(sprintf('<error>Please add a valid value for parameter "%s"</error>', $_sKey));
186	4	return false;
187		}
188		}
189
190	3	return true;
191		}
192
193		/**
194		* @param string $sKey
195		* @param mixed $mDefaultValue
196		* @return mixed
197		*/
198	3	private function getParameterValue($sKey, $mDefaultValue = null)
199		{
200	3	$_oParser = new Parser();
		0 ignored issues – show Coding Style introduced 2017-03-02 10:41 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
201	3	$_sParameterFile = $this->getHomeDir() . DIRECTORY_SEPARATOR . $this->getParameterFileName();
202
203	3	if (file_exists($_sParameterFile))
204		{
205		$_aParameters = $_oParser->parse(
206		file_get_contents($_sParameterFile)
207		);
208
209		if (isset($_aParameters['parameters']) && isset($_aParameters['parameters'][$sKey]))
210		{
211		return $_aParameters['parameters'][$sKey];
212		}
213		}
214
215	3	return $mDefaultValue;
216		}
217
218		/**
219		* @param string $sQuestion
220		* @param null\|mixed $mDefaultValue
221		* @param boolean $bHiddenAnswer
222		* @return mixed
223		*/
224	3	private function printQuestion($sQuestion, $mDefaultValue = null, $bHiddenAnswer = false)
225		{
226	3	$_oHelper = $this->getHelper('question');
227
228		// If we have a hidden answer and the default value is not empty
229		// the we will set it as empty, because we don`t want to show
230		// the default value on the terminal.
231		// We know that the user has to enter the password again
232		// if he / she want to reconfigure something. But this
233		// is an acceptable tradeoff.
234	3	if ($bHiddenAnswer === true && !empty($mDefaultValue))
235		{
236		$mDefaultValue = null;
		0 ignored issues – show Coding Style introduced 2017-03-02 10:41 UTC by Report Bug Copy Issue Report Consider using a different name than the parameter `$mDefaultValue`. This often makes code more readable. Loading history...
237		}
238
239	3	$_sFormat = (!empty($mDefaultValue)) ? '<comment>%s (default: %s):</comment>' : '<comment>%s:</comment>';
		0 ignored issues – show Coding Style introduced 2017-03-02 10:41 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
240	3	$_oQuestion = new Question(sprintf($_sFormat, $sQuestion, $mDefaultValue), $mDefaultValue);
241
242		// Sensitive information (like passwords) should not be
243		// visible during the configuration wizard
244	3	if ($bHiddenAnswer === true)
245		{
246	3	$_oQuestion->setHidden(true);
247	3	$_oQuestion->setHiddenFallback(false);
248		}
249
250	3	return $_oHelper->ask($this->oInput, $this->oOutput, $_oQuestion);
251		}
252		}

trivago / chapi

GitHub Access Token became invalid

Push — master ( 151e35...1ba630 )

ConfigureCommand::hasValidateUserInput() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like