InstallSampleData::execute() - Code Metrics - Inspection of "Garden change log" - netz98/n98-magerun2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 569b94...a0c664 )

by Tom

created 2016-04-30 06:15 UTC

InstallSampleData::execute() A

↳ Parent: InstallSampleData

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	1	Features	0

Metric	Value
c	3
b	1
f	0
dl	0
loc	15
rs	9.4285
cc	3
eloc	8
nc	3
nop	0

<?php

namespace N98\Magento\Command\Installer\SubCommand;

use N98\Magento\Command\SubCommand\AbstractSubCommand;
use N98\Util\OperatingSystem;
use Symfony\Component\Process\ProcessBuilder;

class InstallSampleData extends AbstractSubCommand
{
    /**
     * @return bool

     */
    public function execute()
    {
        if ($this->input->getOption('noDownload')) {
            return false;
        }

        $installationFolder = $this->config->getString('installationFolder');
        chdir($installationFolder);

        $flag = $this->getOptionalBooleanOption('installSampleData', 'Install sample data?');

        if ($flag) {
            $this->runSampleDataInstaller();
        }
    }

    protected function runSampleDataInstaller()
    {
        $installationArgs = $this->config->getArray('installation_args');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $this->runMagentoCommand('sampledata:deploy');
        $this->runMagentoCommand('setup:upgrade');
    }

    /**
     * @return array

     */
    private function runMagentoCommand($command)
    {
        $processBuilder = new ProcessBuilder([
            'php',
            'bin/magento',
            $command,
        ]);

        if (!OperatingSystem::isWindows()) {
            $processBuilder->setPrefix('/usr/bin/env');
        }

        $process = $processBuilder->getProcess();
        $process->setTimeout(86400);
        $process->start();
        $process->wait(function ($type, $buffer) {
            $this->output->write('bin/magento > ' . $buffer, false);
        });
    }
}


1			<?php
2
3			namespace N98\Magento\Command\Installer\SubCommand;
4
5			use N98\Magento\Command\SubCommand\AbstractSubCommand;
6			use N98\Util\OperatingSystem;
7			use Symfony\Component\Process\ProcessBuilder;
8
9			class InstallSampleData extends AbstractSubCommand
10			{
11			/**
12			* @return bool
			0 ignored issues – show Documentation introduced 2015-11-23 11:00 UTC by Report Bug Copy Issue Report Should the return type not be `false\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
13			*/
14			public function execute()
15			{
16			if ($this->input->getOption('noDownload')) {
17			return false;
18			}
19
20			$installationFolder = $this->config->getString('installationFolder');
21			chdir($installationFolder);
22
23			$flag = $this->getOptionalBooleanOption('installSampleData', 'Install sample data?');
24
25			if ($flag) {
26			$this->runSampleDataInstaller();
27			}
28			}
29
30			protected function runSampleDataInstaller()
31			{
32			$installationArgs = $this->config->getArray('installation_args');
			0 ignored issues – show Unused Code introduced 2015-11-23 11:00 UTC by Report Bug Copy Issue Report `$installationArgs` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
33
34			$this->runMagentoCommand('sampledata:deploy');
35			$this->runMagentoCommand('setup:upgrade');
36			}
37
38			/**
39			* @return array
			0 ignored issues – show Documentation introduced 2016-04-30 06:19 UTC by Report Bug Copy Issue Report Should the return type not be `array\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
40			*/
41			private function runMagentoCommand($command)
42			{
43			$processBuilder = new ProcessBuilder([
44			'php',
45			'bin/magento',
46			$command,
47			]);
48
49			if (!OperatingSystem::isWindows()) {
50			$processBuilder->setPrefix('/usr/bin/env');
51			}
52
53			$process = $processBuilder->getProcess();
54			$process->setTimeout(86400);
55			$process->start();
56			$process->wait(function ($type, $buffer) {
57			$this->output->write('bin/magento > ' . $buffer, false);
58			});
59			}
60			}
61

netz98 / n98-magerun2

Push — develop ( 569b94...a0c664 )

InstallSampleData::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like