Controller - Code Metrics - Inspection of "added Directory component, not tested" - hiqdev/hidev - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fbb4c3...48aefa )

by Andrii

created 2017-05-04 13:08 UTC

Controller A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

23.33%

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
wmc	13
c	1
b	0
f	1
lcom	0
cbo	2
dl	0
loc	81
ccs	7
cts	30
cp	0.2333
rs	10

9 Methods

Rating	Name	Size	Complexity
A	behaviors()	6	1
A	setBefore()	4	1
A	getBefore()	4	1
A	setAfter()	4	1
A	getAfter()	4	1
A	runActions()	13	4
A	isResponseOk()	4	2
A	readline()	4	1
A	readpassword()	10	1

<?php
/**
 * Automation tool mixed with code generator for easier continuous development
 *
 * @link      https://github.com/hiqdev/hidev
 * @package   hidev
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2017, HiQDev (http://hiqdev.com/)
 */

namespace hidev\base;

use hidev\controllers\CommonBehavior;

/**
 * Basic controller.
 */
abstract class Controller extends \yii\console\Controller
{
    use GettersTrait;

    public $layout = false;

    protected $_before = [];
    protected $_after  = [];

    public function behaviors()
    {
        return [
            CommonBehavior::class,
        ];
    }

    public function setBefore($requests)
    {
        $this->_before = array_merge($this->getBefore(), $this->normalizeTasks($requests));
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    public function getBefore()
    {
        return $this->_before;
    }

    public function setAfter($requests)
    {
        $this->_after = array_merge($this->getAfter(), $this->normalizeTasks($requests));
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    public function getAfter()
    {
        return $this->_after;
    }

    public function readline($prompt)
    {
        return readline($prompt);
    }

    public function readpassword($prompt)
    {
        echo $prompt;
        system('stty -echo');
        $password = rtrim(fgets(STDIN), PHP_EOL);
        system('stty echo');
        echo "\n";

        return $password;
    }

    /**
     * Runs list of actions.
     * @param null|string|array $actions
     * @return int|Response exit code
     */
    public function runActions($actions)
    {
        foreach ($this->normalizeTasks($actions) as $action => $enabled) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            if ($enabled) {
                $res = $this->runAction($action);
                if (!static::isResponseOk($res)) {
                    return $res;
                }
            }
        }

        return 0;
    }

    /**
     * Is response Ok.
     * @param Response|int $response
     * @return bool
     */
    public static function isResponseOk($response)
    {
        return !(is_object($response) ? $response->exitStatus : $response);
    }
}


1		<?php
2		/**
3		* Automation tool mixed with code generator for easier continuous development
4		*
5		* @link https://github.com/hiqdev/hidev
6		* @package hidev
7		* @license BSD-3-Clause
8		* @copyright Copyright (c) 2015-2017, HiQDev (http://hiqdev.com/)
9		*/
10
11		namespace hidev\base;
12
13		use hidev\controllers\CommonBehavior;
14
15		/**
16		* Basic controller.
17		*/
18		abstract class Controller extends \yii\console\Controller
19		{
20		use GettersTrait;
21
22		public $layout = false;
23
24		protected $_before = [];
25		protected $_after = [];
26
27	1	public function behaviors()
28		{
29		return [
30	1	CommonBehavior::class,
31		];
32		}
33
34	1	public function setBefore($requests)
35		{
36	1	$this->_before = array_merge($this->getBefore(), $this->normalizeTasks($requests));
		0 ignored issues – show Documentation Bug introduced 2017-04-29 17:12 UTC by Report Bug Copy Issue Report The method `normalizeTasks` does not exist on `object<hidev\base\Controller>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
37	1	}
38
39	1	public function getBefore()
40		{
41	1	return $this->_before;
42		}
43
44		public function setAfter($requests)
45		{
46		$this->_after = array_merge($this->getAfter(), $this->normalizeTasks($requests));
		0 ignored issues – show Documentation Bug introduced 2017-04-29 17:12 UTC by Report Bug Copy Issue Report The method `normalizeTasks` does not exist on `object<hidev\base\Controller>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
47		}
48
49		public function getAfter()
50		{
51		return $this->_after;
52		}
53
54		public function readline($prompt)
55		{
56		return readline($prompt);
57		}
58
59		public function readpassword($prompt)
60		{
61		echo $prompt;
62		system('stty -echo');
63		$password = rtrim(fgets(STDIN), PHP_EOL);
64		system('stty echo');
65		echo "\n";
66
67		return $password;
68		}
69
70		/**
71		* Runs list of actions.
72		* @param null\|string\|array $actions
73		* @return int\|Response exit code
74		*/
75		public function runActions($actions)
76		{
77		foreach ($this->normalizeTasks($actions) as $action => $enabled) {
		0 ignored issues – show Documentation Bug introduced 2017-05-03 09:59 UTC by Report Bug Copy Issue Report The method `normalizeTasks` does not exist on `object<hidev\base\Controller>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
78		if ($enabled) {
79		$res = $this->runAction($action);
80		if (!static::isResponseOk($res)) {
81		return $res;
82		}
83		}
84		}
85
86		return 0;
87		}
88
89		/**
90		* Is response Ok.
91		* @param Response\|int $response
92		* @return bool
93		*/
94		public static function isResponseOk($response)
95		{
96		return !(is_object($response) ? $response->exitStatus : $response);
97		}
98		}
99

hiqdev / hidev

Push — master ( fbb4c3...48aefa )

Controller A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

9 Methods

Duplication Side-by-Side

Filter issues like