GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Passed
Push — master ( a1835d...10e84a )
by
unknown
02:48
created

Commander::execute()   B

Complexity

Conditions 7
Paths 21

Size

Total Lines 101
Code Lines 78

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 7
eloc 78
nc 21
nop 0
dl 0
loc 101
rs 7.5466
c 0
b 0
f 0

How to fix   Long Method   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
/**
3
 * This file is part of the O2System Framework package.
4
 *
5
 * For the full copyright and license information, please view the LICENSE
6
 * file that was distributed with this source code.
7
 *
8
 * @author         Steeve Andrian Salim
9
 * @copyright      Copyright (c) Steeve Andrian Salim
10
 */
11
12
// ------------------------------------------------------------------------
13
14
namespace O2System\Framework\Cli\Commanders\Make;
15
16
// ------------------------------------------------------------------------
17
18
use O2System\Framework\Cli\Commanders\Make;
19
use O2System\Kernel\Cli\Writers\Format;
20
21
/**
22
 * Class Commander
23
 *
24
 * @package O2System\Framework\Cli\Commanders\Make
25
 */
26
class Commander extends Make
27
{
28
    /**
29
     * Commander::$commandDescription
30
     *
31
     * Command description.
32
     *
33
     * @var string
34
     */
35
    protected $commandDescription = 'CLI_MAKE_COMMANDER_DESC';
36
37
    // ------------------------------------------------------------------------
38
39
    /**
40
     * Commander::execute
41
     */
42
    public function execute()
43
    {
44
        parent::execute();
45
46
        if (empty($this->optionFilename)) {
47
            output()->write(
0 ignored issues
show
Bug introduced by
The method write() does not exist on O2System\Kernel\Http\Output. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

47
            output()->/** @scrutinizer ignore-call */ write(

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
48
                (new Format())
49
                    ->setContextualClass(Format::DANGER)
50
                    ->setString(language()->getLine('CLI_MAKE_COMMANDER_E_FILENAME'))
51
                    ->setNewLinesAfter(1)
52
            );
53
54
            exit(EXIT_ERROR);
0 ignored issues
show
Best Practice introduced by
Using exit here is not recommended.

In general, usage of exit should be done with care and only when running in a scripting context like a CLI script.

Loading history...
55
        }
56
57
        if (strpos($this->optionPath, 'Commanders') === false) {
58
            $filePath = $this->optionPath . 'Commanders' . DIRECTORY_SEPARATOR . $this->optionFilename;
59
        } else {
60
            $filePath = $this->optionPath . $this->optionFilename;
61
        }
62
63
        $fileDirectory = dirname($filePath) . DIRECTORY_SEPARATOR;
64
65
        if ( ! is_dir($fileDirectory)) {
66
            mkdir($fileDirectory, 0777, true);
67
        }
68
69
        if (is_file($filePath)) {
70
            output()->write(
71
                (new Format())
72
                    ->setContextualClass(Format::DANGER)
73
                    ->setString(language()->getLine('CLI_MAKE_COMMANDER_E_EXISTS', [$filePath]))
74
                    ->setNewLinesAfter(1)
75
            );
76
77
            exit(EXIT_ERROR);
0 ignored issues
show
Best Practice introduced by
Using exit here is not recommended.

In general, usage of exit should be done with care and only when running in a scripting context like a CLI script.

Loading history...
78
        }
79
80
        $className = prepare_class_name(pathinfo($filePath, PATHINFO_FILENAME));
81
        @list($namespaceDirectory, $subNamespace) = explode('Commanders', dirname($filePath));
82
83
        $classNamespace = loader()->getDirNamespace(
84
                $namespaceDirectory
85
            ) . 'Commanders' . (empty($subNamespace)
86
                ? null
87
                : str_replace(
88
                    '/',
89
                    '\\',
90
                    $subNamespace
91
                )) . '\\';
92
93
        $vars[ 'CREATE_DATETIME' ] = date('d/m/Y H:m');
0 ignored issues
show
Comprehensibility Best Practice introduced by
$vars was never initialized. Although not strictly required by PHP, it is generally a good practice to add $vars = array(); before regardless.
Loading history...
94
        $vars[ 'NAMESPACE' ] = trim($classNamespace, '\\');
95
        $vars[ 'PACKAGE' ] = '\\' . trim($classNamespace, '\\');
96
        $vars[ 'CLASS' ] = $className;
97
        $vars[ 'FILEPATH' ] = $filePath;
98
99
        $phpTemplate = <<<PHPTEMPLATE
100
<?php
101
/**
102
 * Created by O2System Framework File Generator.
103
 * DateTime: CREATE_DATETIME
104
 */
105
106
// ------------------------------------------------------------------------
107
108
namespace NAMESPACE;
109
110
// ------------------------------------------------------------------------
111
112
use O2System\Framework\Http\Commander;
113
114
/**
115
 * Class CLASS
116
 *
117
 * @package PACKAGE
118
 */
119
class CLASS extends Commander
120
{
121
    /**
122
     * CLASS::index
123
     */
124
    public function index()
125
    {
126
        // TODO: Change the autogenerated stub
127
    }
128
}
129
PHPTEMPLATE;
130
131
        $fileContent = str_replace(array_keys($vars), array_values($vars), $phpTemplate);
132
        file_put_contents($filePath, $fileContent);
133
134
        if (is_file($filePath)) {
135
            output()->write(
136
                (new Format())
137
                    ->setContextualClass(Format::SUCCESS)
138
                    ->setString(language()->getLine('CLI_MAKE_COMMANDER_S_MAKE', [$filePath]))
139
                    ->setNewLinesAfter(1)
140
            );
141
142
            exit(EXIT_SUCCESS);
0 ignored issues
show
Best Practice introduced by
Using exit here is not recommended.

In general, usage of exit should be done with care and only when running in a scripting context like a CLI script.

Loading history...
143
        }
144
    }
145
}