Issues in SignatureParser.php (master) - Issues in master - railt/console - Measure and Improve Code Quality continuously with Scrutinizer

Issues (26)

Security Analysis no request data

Cross-Site Scripting

Cross-Site Scripting enables an attacker to inject code into the response of a web-request that is viewed by other users. It can for example be used to bypass access controls, or even to take over other users' accounts.

File Exposure

File Exposure allows an attacker to gain access to local files that he should not be able to access. These files can for example include database credentials, or other configuration files.

File Manipulation

File Manipulation enables an attacker to write custom data to files. This potentially leads to injection of arbitrary code on the server.

Object Injection

Object Injection enables an attacker to inject an object into PHP code, and can lead to arbitrary code execution, file exposure, or file manipulation attacks.

Code Injection

Code Injection enables an attacker to execute arbitrary code on the server.

Response Splitting

Response Splitting can be used to send arbitrary responses.

File Inclusion

File Inclusion enables an attacker to inject custom files into PHP's file loading mechanism, either explicitly passed to include, or for example via PHP's auto-loading mechanism.

Command Injection

Command Injection enables an attacker to inject a shell command that is execute with the privileges of the web-server. This can be used to expose sensitive data, or gain access of your server.

SQL Injection

SQL Injection enables an attacker to execute arbitrary SQL code on your database server gaining access to user data, or manipulating user data.

XPath Injection

XPath Injection enables an attacker to modify the parts of XML document that are read. If that XML document is for example used for authentication, this can lead to further vulnerabilities similar to SQL Injection.

LDAP Injection

LDAP Injection enables an attacker to inject LDAP statements potentially granting permission to run unauthorized queries, or modify content inside the LDAP tree.

Header Injection

Other Vulnerability

This category comprises other attack vectors such as manipulating the PHP runtime, loading custom extensions, freezing the runtime, or similar.

Regex Injection

Regex Injection enables an attacker to execute arbitrary code in your PHP process.

XML Injection

XML Injection enables an attacker to read files on your local filesystem including configuration files, or can be abused to freeze your web-server process.

Variable Injection

Variable Injection enables an attacker to overwrite program variables with custom data, and can lead to further vulnerabilities.

Unfortunately, the security analysis is currently not available for your project. If you are a non-commercial open-source project, please contact support to gain access.

src/SignatureParser.php (16 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
/**
 * This file is part of Railt package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace Railt\Console;

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputOption;

/**
 * Class SignatureParser
 */
class SignatureParser
{
    /**
     * @var string
     */
    private $name;

    /**
     * @var array
     */
    private $arguments = [];

    /**
     * @var array
     */
    private $options = [];

    /**
     * SignatureParser constructor.
     * @param string $signature
     * @throws \InvalidArgumentException
     */
    public function __construct(string $signature)
    {
        [$this->name, $this->arguments, $this->options] = $this->parse($signature);
    }

    /**
     * @return string
     */
    public function getName(): string
    {
        return $this->name;
    }

    /**
     * @return array
     */
    public function getArguments(): array
    {
        return $this->arguments;
    }

    /**
     * @return array
     */
    public function getOptions(): array
    {
        return $this->options;
    }

    /**
     * Parse the given console command definition into an array.
     *
     * @param string $expression
     * @return array
     * @throws \InvalidArgumentException
     */
    protected function parse($expression): array
    {
        $name = $this->name($expression);

        if (\preg_match_all('/\{\s*(.*?)\s*\}/', $expression, $matches) && \count($matches[1])) {
            return \array_merge([$name], $this->parameters($matches[1]));
        }

        return [$name, [], []];
    }

    /**
     * Extract the name of the command from the expression.
     *
     * @param string $expression
     * @return string
     * @throws \InvalidArgumentException
     */
    protected function name($expression): string
    {
        if (\trim($expression) === '') {
            throw new \InvalidArgumentException('Console command definition is empty.');
        }

        if (! \preg_match('/[\S]+/', $expression, $matches)) {
            throw new \InvalidArgumentException('Unable to determine command name from signature.');
        }

        return $matches[0];
    }

    /**
     * Extract all of the parameters from the tokens.
     *
     * @param array $tokens
     * @return array
     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException
     */
    protected function parameters(array $tokens): array
    {
        [$arguments, $options] = [[], []];
function someFunction() {
    $x = 5;
    echo $x;
}

        foreach ($tokens as $token) {
            if (\preg_match('/-{2,}(.*)/', $token, $matches)) {
                $options[] = $this->parseOption($matches[1]);
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            } else {
                $arguments[] = $this->parseArgument($token);
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            }
        }

        return [$arguments, $options];
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }

    /**
     * Parse an option expression.
     *
     * @param string $token
     * @return \Symfony\Component\Console\Input\InputOption
     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException
     */
    protected function parseOption($token): ?InputOption
    {
        [$token, $description] = $this->extractDescription($token);


        $matches = \preg_split('/\s*\|\s*/', $token, 2);

        if (isset($matches[1])) {
            [$shortcut, $token] = [$matches[0], $matches[1]];
function someFunction() {
    $x = 5;
    echo $x;
}
        } else {
            $shortcut = null;
        }

        switch (true) {

            case $this->endsWith($token, '='):
                return new InputOption(\trim($token, '='), $shortcut, InputOption::VALUE_OPTIONAL, $description);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

            case $this->endsWith($token, '=*'):
                return new InputOption(\trim($token, '=*'), $shortcut, InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY, $description);

            case \preg_match('/(.+)\=\*(.+)/', $token, $matches):
                return new InputOption($matches[1], $shortcut, InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY, $description, \preg_split('/,\s?/', $matches[2]));

            case \preg_match('/(.+)\=(.+)/', $token, $matches):
                return new InputOption($matches[1], $shortcut, InputOption::VALUE_OPTIONAL, $description, $matches[2]);

            default:
                return new InputOption($token, $shortcut, InputOption::VALUE_NONE, $description);
        }
    }

    /**
     * Parse the token into its token and description segments.
     *
     * @param  string $token
     * @return array
     */
    protected function extractDescription($token): array
    {
        $parts = \preg_split('/\s+:\s+/', \trim($token), 2);

        return \count($parts) === 2 ? $parts : [$token, ''];
    }

    /**
     * @param string $haystack
     * @param string $needle
     * @return bool
     */
    private function endsWith(string $haystack, string $needle): bool
    {
        return \substr($haystack, -\strlen($needle)) === $needle;
    }

    /**
     * Parse an argument expression.
     *
     * @param string $token
     * @return \Symfony\Component\Console\Input\InputArgument
     * @throws \Symfony\Component\Console\Exception\InvalidArgumentException
     */
    protected function parseArgument($token): ?InputArgument
    {
        [$token, $description] = $this->extractDescription($token);


        switch (true) {

            case $this->endsWith($token, '?*'):
                return new InputArgument(\trim($token, '?*'), InputArgument::IS_ARRAY, $description);

            case $this->endsWith($token, '*'):
                return new InputArgument(\trim($token, '*'), InputArgument::IS_ARRAY | InputArgument::REQUIRED, $description);

            case $this->endsWith($token, '?'):
                return new InputArgument(\trim($token, '?'), InputArgument::OPTIONAL, $description);

            case \preg_match('/(.+)\=\*(.+)/', $token, $matches):
                return new InputArgument($matches[1], InputArgument::IS_ARRAY, $description, \preg_split('/,\s?/', $matches[2]));
function someFunction() {
    $x = 5;
    echo $x;
}

            case \preg_match('/(.+)\=(.+)/', $token, $matches):
                return new InputArgument($matches[1], InputArgument::OPTIONAL, $description, $matches[2]);
function someFunction() {
    $x = 5;
    echo $x;
}

            default:
                return new InputArgument($token, InputArgument::REQUIRED, $description);
        }
    }
}


Issues (26)

Security Analysis no request data

src/SignatureParser.php (16 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

Available Fixes

Available Fixes

1			<?php
2			/**
3			* This file is part of Railt 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			declare(strict_types=1);
9
10			namespace Railt\Console;
11
12			use Symfony\Component\Console\Input\InputArgument;
13			use Symfony\Component\Console\Input\InputOption;
14
15			/**
16			* Class SignatureParser
17			*/
18			class SignatureParser
19			{
20			/**
21			* @var string
22			*/
23			private $name;
24
25			/**
26			* @var array
27			*/
28			private $arguments = [];
29
30			/**
31			* @var array
32			*/
33			private $options = [];
34
35			/**
36			* SignatureParser constructor.
37			* @param string $signature
38			* @throws \InvalidArgumentException
39			*/
40			public function __construct(string $signature)
41			{
42			[$this->name, $this->arguments, $this->options] = $this->parse($signature);
43			}
44
45			/**
46			* @return string
47			*/
48			public function getName(): string
49			{
50			return $this->name;
51			}
52
53			/**
54			* @return array
55			*/
56			public function getArguments(): array
57			{
58			return $this->arguments;
59			}
60
61			/**
62			* @return array
63			*/
64			public function getOptions(): array
65			{
66			return $this->options;
67			}
68
69			/**
70			* Parse the given console command definition into an array.
71			*
72			* @param string $expression
73			* @return array
74			* @throws \InvalidArgumentException
75			*/
76			protected function parse($expression): array
77			{
78			$name = $this->name($expression);
79
80			if (\preg_match_all('/\{\s(.?)\s*\}/', $expression, $matches) && \count($matches[1])) {
81			return \array_merge([$name], $this->parameters($matches[1]));
82			}
83
84			return [$name, [], []];
85			}
86
87			/**
88			* Extract the name of the command from the expression.
89			*
90			* @param string $expression
91			* @return string
92			* @throws \InvalidArgumentException
93			*/
94			protected function name($expression): string
95			{
96			if (\trim($expression) === '') {
97			throw new \InvalidArgumentException('Console command definition is empty.');
98			}
99
100			if (! \preg_match('/[\S]+/', $expression, $matches)) {
101			throw new \InvalidArgumentException('Unable to determine command name from signature.');
102			}
103
104			return $matches[0];
105			}
106
107			/**
108			* Extract all of the parameters from the tokens.
109			*
110			* @param array $tokens
111			* @return array
112			* @throws \Symfony\Component\Console\Exception\InvalidArgumentException
113			*/
114			protected function parameters(array $tokens): array
115			{
116			[$arguments, $options] = [[], []];
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$arguments` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history... Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$options` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
117
118			foreach ($tokens as $token) {
119			if (\preg_match('/-{2,}(.*)/', $token, $matches)) {
120			$options[] = $this->parseOption($matches[1]);
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$options` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$options = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
121			} else {
122			$arguments[] = $this->parseArgument($token);
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$arguments` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$arguments = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
123			}
124			}
125
126			return [$arguments, $options];
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$arguments` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history... Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$options` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
127			}
128
129			/**
130			* Parse an option expression.
131			*
132			* @param string $token
133			* @return \Symfony\Component\Console\Input\InputOption
134			* @throws \Symfony\Component\Console\Exception\InvalidArgumentException
135			*/
136			protected function parseOption($token): ?InputOption
137			{
138			[$token, $description] = $this->extractDescription($token);
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$description` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
139
140			$matches = \preg_split('/\s\\|\s/', $token, 2);
141
142			if (isset($matches[1])) {
143			[$shortcut, $token] = [$matches[0], $matches[1]];
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$shortcut` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
144			} else {
145			$shortcut = null;
146			}
147
148			switch (true) {
			0 ignored issues – show Bug Best Practice introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like you are loosely comparing `\preg_match('/(.+)\\=\\*(.+)/', $token, $matches)` of type `integer` to the boolean `true`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history... Bug Best Practice introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like you are loosely comparing `\preg_match('/(.+)\\=(.+)/', $token, $matches)` of type `integer` to the boolean `true`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history...
149			case $this->endsWith($token, '='):
150			return new InputOption(\trim($token, '='), $shortcut, InputOption::VALUE_OPTIONAL, $description);
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$shortcut` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
151
152			case $this->endsWith($token, '=*'):
153			return new InputOption(\trim($token, '=*'), $shortcut, InputOption::VALUE_OPTIONAL \| InputOption::VALUE_IS_ARRAY, $description);
154
155			case \preg_match('/(.+)\=\*(.+)/', $token, $matches):
156			return new InputOption($matches[1], $shortcut, InputOption::VALUE_OPTIONAL \| InputOption::VALUE_IS_ARRAY, $description, \preg_split('/,\s?/', $matches[2]));
157
158			case \preg_match('/(.+)\=(.+)/', $token, $matches):
159			return new InputOption($matches[1], $shortcut, InputOption::VALUE_OPTIONAL, $description, $matches[2]);
160
161			default:
162			return new InputOption($token, $shortcut, InputOption::VALUE_NONE, $description);
163			}
164			}
165
166			/**
167			* Parse the token into its token and description segments.
168			*
169			* @param string $token
170			* @return array
171			*/
172			protected function extractDescription($token): array
173			{
174			$parts = \preg_split('/\s+:\s+/', \trim($token), 2);
175
176			return \count($parts) === 2 ? $parts : [$token, ''];
177			}
178
179			/**
180			* @param string $haystack
181			* @param string $needle
182			* @return bool
183			*/
184			private function endsWith(string $haystack, string $needle): bool
185			{
186			return \substr($haystack, -\strlen($needle)) === $needle;
187			}
188
189			/**
190			* Parse an argument expression.
191			*
192			* @param string $token
193			* @return \Symfony\Component\Console\Input\InputArgument
194			* @throws \Symfony\Component\Console\Exception\InvalidArgumentException
195			*/
196			protected function parseArgument($token): ?InputArgument
197			{
198			[$token, $description] = $this->extractDescription($token);
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$description` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
199
200			switch (true) {
			0 ignored issues – show Bug Best Practice introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like you are loosely comparing `\preg_match('/(.+)\\=\\*(.+)/', $token, $matches)` of type `integer` to the boolean `true`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history... Bug Best Practice introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like you are loosely comparing `\preg_match('/(.+)\\=(.+)/', $token, $matches)` of type `integer` to the boolean `true`. If you are specifically checking for non-zero, consider using something more explicit like `> 0` or `!== 0` instead. Loading history...
201			case $this->endsWith($token, '?*'):
202			return new InputArgument(\trim($token, '?*'), InputArgument::IS_ARRAY, $description);
203
204			case $this->endsWith($token, '*'):
205			return new InputArgument(\trim($token, '*'), InputArgument::IS_ARRAY \| InputArgument::REQUIRED, $description);
206
207			case $this->endsWith($token, '?'):
208			return new InputArgument(\trim($token, '?'), InputArgument::OPTIONAL, $description);
209
210			case \preg_match('/(.+)\=\*(.+)/', $token, $matches):
211			return new InputArgument($matches[1], InputArgument::IS_ARRAY, $description, \preg_split('/,\s?/', $matches[2]));
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$matches` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
212
213			case \preg_match('/(.+)\=(.+)/', $token, $matches):
214			return new InputArgument($matches[1], InputArgument::OPTIONAL, $description, $matches[2]);
			0 ignored issues – show Bug introduced 2018-07-21 02:35 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$matches` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
215
216			default:
217			return new InputArgument($token, InputArgument::REQUIRED, $description);
218			}
219			}
220			}
221

railt / console

Issues (26)

Security Analysis no request data

src/SignatureParser.php (16 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like