CommandCollector::addCommand() - Code Metrics - Inspection of "ContainerAware trait deleted." - venta/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( a1b0a5...d00cde )

by Alexey

created 2016-12-03 21:56 UTC

CommandCollector::addCommand() A

↳ Parent: CommandCollector

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2.1481

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	6
cts	9
cp	0.6667
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	1
crap	2.1481

<?php declare(strict_types = 1);

namespace Venta\Console\Command;

use Venta\Console\Command;
use Venta\Contracts\Console\CommandCollector as CommandCollectorContract;
use Venta\Contracts\Container\Container;

/**
 * Class Collector
 *
 * @package Venta\Console\Command
 */
class CommandCollector implements CommandCollectorContract
{

    /**
     * Commands holder
     *
     * @var Command[]
     */
    protected $commands = [];

    /**
     * @var Container
     */
    protected $container;

    /**
     * CommandCollector constructor.
     *
     * @param Container $container
     */
    public function __construct(Container $container)
    {
        $this->container = $container;
    }

    /**
     * @inheritDoc
     */
    public function addCommand(string $commandClassName)
    {
        if (!is_subclass_of($commandClassName, Command::class)) {
            throw new \InvalidArgumentException(
                sprintf('Provided command "%s" doesn\'t extend Venta\Console\Command class.', $commandClassName)
            );
        }
        $this->commands[] = $commandClassName;
    }

    /**
     * @inheritDoc
     */
    public function getCommands(): array
    {
        return array_map([$this->container, 'get'], $this->commands);
    }

}

1		<?php declare(strict_types = 1);
2
3		namespace Venta\Console\Command;
4
5		use Venta\Console\Command;
6		use Venta\Contracts\Console\CommandCollector as CommandCollectorContract;
7		use Venta\Contracts\Container\Container;
8
9		/**
10		* Class Collector
11		*
12		* @package Venta\Console\Command
13		*/
14		class CommandCollector implements CommandCollectorContract
15		{
16
17		/**
18		* Commands holder
19		*
20		* @var Command[]
21		*/
22		protected $commands = [];
23
24		/**
25		* @var Container
26		*/
27		protected $container;
28
29		/**
30		* CommandCollector constructor.
31		*
32		* @param Container $container
33		*/
34	2	public function __construct(Container $container)
35		{
36	2	$this->container = $container;
37	2	}
38
39		/**
40		* @inheritDoc
41		*/
42	2	public function addCommand(string $commandClassName)
43		{
44	2	if (!is_subclass_of($commandClassName, Command::class)) {
45	1	throw new \InvalidArgumentException(
46	1	sprintf('Provided command "%s" doesn\'t extend Venta\Console\Command class.', $commandClassName)
47		);
48		}
49	1	$this->commands[] = $commandClassName;
50	1	}
51
52		/**
53		* @inheritDoc
54		*/
55	1	public function getCommands(): array
56		{
57	1	return array_map([$this->container, 'get'], $this->commands);
58		}
59
60		}

venta / framework

Push — master ( a1b0a5...d00cde )

CommandCollector::addCommand() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like