AbstractCommand::prepareDependencies() - Code Metrics - Inspection of "Allow checking bc between 2 project forks using si..." - console-helpers/code-insight - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cbb650...b6a2ec )

by Alexander

created 2017-02-05 17:10 UTC

AbstractCommand::prepareDependencies() A

↳ Parent: AbstractCommand

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	4
cp	0
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0
crap	2

<?php
/**
 * This file is part of the Code-Insight library.
 * For the full copyright and license information, please view
 * the LICENSE file that was distributed with this source code.
 *
 * @copyright Alexander Obuhovich <[email protected]>
 * @link      https://github.com/console-helpers/code-insight
 */

namespace ConsoleHelpers\CodeInsight\Command;


use ConsoleHelpers\ConsoleKit\Command\AbstractCommand as BaseCommand;

/**
 * Base command class.
 */
abstract class AbstractCommand extends BaseCommand
{

	/**
	 * Prepare dependencies.
	 *
	 * @return void
	 */
	protected function prepareDependencies()
	{
		$container = $this->getContainer();
$myVar = 'Value';
$higher = false;

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

	/**
	 * Returns and validates path.
	 *
	 * @param string $argument_name Argument name, that contains path.
	 *
	 * @return string
	 * @throws \InvalidArgumentException When path isn't valid.
	 */
	protected function getPath($argument_name)
	{
		$raw_path = $this->io->getArgument($argument_name);

		if ( !$raw_path ) {
			return '';
		}

		$path = realpath($raw_path);

		if ( !$path || !file_exists($path) || !is_dir($path) ) {
			throw new \InvalidArgumentException('The "' . $raw_path . '" path is invalid.');
		}

		return $path;
	}

}


1			<?php
2			/**
3			* This file is part of the Code-Insight library.
4			* For the full copyright and license information, please view
5			* the LICENSE file that was distributed with this source code.
6			*
7			* @copyright Alexander Obuhovich <[email protected]>
8			* @link https://github.com/console-helpers/code-insight
9			*/
10
11			namespace ConsoleHelpers\CodeInsight\Command;
12
13
14			use ConsoleHelpers\ConsoleKit\Command\AbstractCommand as BaseCommand;
15
16			/**
17			* Base command class.
18			*/
19			abstract class AbstractCommand extends BaseCommand
20			{
21
22			/**
23			* Prepare dependencies.
24			*
25			* @return void
26			*/
27			protected function prepareDependencies()
28			{
29			$container = $this->getContainer();
			0 ignored issues – show Unused Code introduced 2016-05-07 12:36 UTC by Report Bug Copy Issue Report `$container` 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...
30			}
31
32			/**
33			* Returns and validates path.
34			*
35			* @param string $argument_name Argument name, that contains path.
36			*
37			* @return string
38			* @throws \InvalidArgumentException When path isn't valid.
39			*/
40			protected function getPath($argument_name)
41			{
42			$raw_path = $this->io->getArgument($argument_name);
43
44			if ( !$raw_path ) {
45			return '';
46			}
47
48			$path = realpath($raw_path);
49
50			if ( !$path \|\| !file_exists($path) \|\| !is_dir($path) ) {
51			throw new \InvalidArgumentException('The "' . $raw_path . '" path is invalid.');
52			}
53
54			return $path;
55			}
56
57			}
58

console-helpers / code-insight

Push — master ( cbb650...b6a2ec )

AbstractCommand::prepareDependencies() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like