ListCommand - Code Metrics - Inspection of "Testing model generation" - nWidart/laravel-modules - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 15e02a...758c52 )

by Nicolas

created 2016-06-29 13:02 UTC

ListCommand A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

14.29%

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	9
c	1
b	0
f	0
lcom	1
cbo	1
dl	0
loc	81
ccs	4
cts	28
cp	0.1429
rs	10

4 Methods

Rating	Name	Size	Complexity
A	fire()	4	1
A	getRows()	15	3
A	getModules()	20	4
A	getOptions()	7	1

<?php

namespace Nwidart\Modules\Commands;

use Illuminate\Console\Command;
use Symfony\Component\Console\Input\InputOption;

class ListCommand extends Command
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'module:list';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Show list of all modules.';

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function fire()
    {
        $this->table(['Name', 'Status', 'Order', 'Path'], $this->getRows());
    }

    /**
     * Get table rows.
     *
     * @return array
     */
    public function getRows()
    {
        $rows = [];

        foreach ($this->getModules() as $module) {
            $rows[] = [
                $module->getStudlyName(),
                $module->enabled() ? 'Enabled' : 'Disabled',
                $module->get('order'),
                $module->getPath(),
            ];
        }

        return $rows;
    }

    public function getModules()
    {
        switch ($this->option('only')) {
            case 'enabled':
                return $this->laravel['modules']->getByStatus(1);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            case 'disabled':
                return $this->laravel['modules']->getByStatus(0);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            case 'ordered':
                return $this->laravel['modules']->getOrdered($this->option('direction'));
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            default:
                return $this->laravel['modules']->all();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }

    /**
     * Get the console command options.
     *
     * @return array
     */
    protected function getOptions()
    {
        return array(
            array('only', null, InputOption::VALUE_OPTIONAL, 'Types of modules will be displayed.', null),
            array('direction', 'd', InputOption::VALUE_OPTIONAL, 'The direction of ordering.', 'asc'),
        );
    }
}


1		<?php
2
3		namespace Nwidart\Modules\Commands;
4
5		use Illuminate\Console\Command;
6		use Symfony\Component\Console\Input\InputOption;
7
8		class ListCommand extends Command
9		{
10		/**
11		* The console command name.
12		*
13		* @var string
14		*/
15		protected $name = 'module:list';
16
17		/**
18		* The console command description.
19		*
20		* @var string
21		*/
22		protected $description = 'Show list of all modules.';
23
24		/**
25		* Execute the console command.
26		*
27		* @return mixed
28		*/
29		public function fire()
30		{
31		$this->table(['Name', 'Status', 'Order', 'Path'], $this->getRows());
32		}
33
34		/**
35		* Get table rows.
36		*
37		* @return array
38		*/
39		public function getRows()
40		{
41		$rows = [];
42
43		foreach ($this->getModules() as $module) {
44		$rows[] = [
45		$module->getStudlyName(),
46		$module->enabled() ? 'Enabled' : 'Disabled',
47		$module->get('order'),
48		$module->getPath(),
49		];
50		}
51
52		return $rows;
53		}
54
55		public function getModules()
56		{
57		switch ($this->option('only')) {
58		case 'enabled':
59		return $this->laravel['modules']->getByStatus(1);
60		break;
		0 ignored issues – show Unused Code introduced 2016-06-28 11:31 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
61
62		case 'disabled':
63		return $this->laravel['modules']->getByStatus(0);
64		break;
		0 ignored issues – show Unused Code introduced 2016-06-28 11:31 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
65
66		case 'ordered':
67		return $this->laravel['modules']->getOrdered($this->option('direction'));
68		break;
		0 ignored issues – show Unused Code introduced 2016-06-28 11:31 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
69
70		default:
71		return $this->laravel['modules']->all();
72		break;
		0 ignored issues – show Unused Code introduced 2016-06-28 11:31 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
73		}
74		}
75
76		/**
77		* Get the console command options.
78		*
79		* @return array
80		*/
81	16	protected function getOptions()
82		{
83		return array(
84	16	array('only', null, InputOption::VALUE_OPTIONAL, 'Types of modules will be displayed.', null),
85	16	array('direction', 'd', InputOption::VALUE_OPTIONAL, 'The direction of ordering.', 'asc'),
86	16	);
87		}
88		}
89

nWidart / laravel-modules

Push — master ( 15e02a...758c52 )

ListCommand A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

4 Methods

Duplication Side-by-Side

Filter issues like