ListCommand::getOptions() - Code Metrics - Inspection of "Load all the services providers attached to a modu..." - nWidart/laravel-modules - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#322)

by Mathieu

created 2017-09-04 13:17 UTC

ListCommand::getOptions() A

↳ Parent: ListCommand

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	7
rs	9.4285
c	0
b	0
f	0
ccs	2
cts	2
cp	1
cc	1
eloc	4
nc	1
nop	0
crap	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.
     */
    public function handle()
    {
        $this->table(['Name', 'Status', 'Order', 'Path'], $this->getRows());
    }

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

        foreach ($this->getModules() as $module) {
            $rows[] = [
                $module->getName(),
                $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		public function handle()
28		{
29		$this->table(['Name', 'Status', 'Order', 'Path'], $this->getRows());
30		}
31
32		/**
33		* Get table rows.
34		*
35		* @return array
36		*/
37		public function getRows()
38		{
39		$rows = [];
40
41		foreach ($this->getModules() as $module) {
42		$rows[] = [
43		$module->getName(),
44		$module->enabled() ? 'Enabled' : 'Disabled',
45		$module->get('order'),
46		$module->getPath(),
47		];
48		}
49
50		return $rows;
51		}
52
53		public function getModules()
54		{
55		switch ($this->option('only')) {
56		case 'enabled':
57		return $this->laravel['modules']->getByStatus(1);
58		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...
59
60		case 'disabled':
61		return $this->laravel['modules']->getByStatus(0);
62		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...
63
64		case 'ordered':
65		return $this->laravel['modules']->getOrdered($this->option('direction'));
66		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...
67
68		default:
69		return $this->laravel['modules']->all();
70		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...
71		}
72		}
73
74		/**
75		* Get the console command options.
76		*
77		* @return array
78		*/
79	56	protected function getOptions()
80		{
81		return array(
82	56	array('only', null, InputOption::VALUE_OPTIONAL, 'Types of modules will be displayed.', null),
83		array('direction', 'd', InputOption::VALUE_OPTIONAL, 'The direction of ordering.', 'asc'),
84		);
85		}
86		}
87

nWidart / laravel-modules

Pull Request — master (#322)

ListCommand::getOptions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like