InvalidRegexPatternException - Code Metrics - Inspection of "allow php 8 (#14)" - everlutionsk/navigation - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( fba5ec...ac8f64 )

by Ivan

created 2021-05-14 17:05 UTC

InvalidRegexPatternException A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	6
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	1
eloc	3
c	1
b	0
f	0
dl	0
loc	6
rs	10

<?php

declare(strict_types=1);

namespace Everlution\Navigation\Match\Voter;


/**
 * Class InvalidRegexPatternException.
 *
 * @author Ivan Barlog <[email protected]>
 */
class InvalidRegexPatternException extends \InvalidArgumentException
{
    public function __construct(string $pattern)
    {
        parent::__construct(
            "Pattern '$pattern' is not valid regex pattern. Correct pattern would be '/regex expression/modifiers'"
// Instead of
$x = "foo $bar $baz";

// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);
        );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Everlution\Navigation\Match\Voter;
			0 ignored issues – show Bug introduced 2021-05-14 15:31 UTC by Report Bug Copy Issue Report A parse error occurred: Syntax error, unexpected T_MATCH, expecting T_STRING on line 5 at column 32 Loading history...
6
7			/**
8			* Class InvalidRegexPatternException.
9			*
10			* @author Ivan Barlog <[email protected]>
11			*/
12			class InvalidRegexPatternException extends \InvalidArgumentException
13			{
14			public function __construct(string $pattern)
15			{
16			parent::__construct(
17			"Pattern '$pattern' is not valid regex pattern. Correct pattern would be '/regex expression/modifiers'"
			0 ignored issues – show Coding Style Best Practice introduced 2018-01-05 21:14 UTC by Report Bug Copy Issue Report As per coding-style, please use concatenation or `sprintf` for the variable `$pattern` instead of interpolation. It is generally a best practice as it is often more readable to use concatenation instead of interpolation for variables inside strings. // Instead of $x = "foo $bar $baz"; // Better use either $x = "foo " . $bar . " " . $baz; $x = sprintf("foo %s %s", $bar, $baz); Loading history...
18			);
19			}
20			}
21

everlutionsk / navigation

Push — master ( fba5ec...ac8f64 )

InvalidRegexPatternException A

Complexity

Size/Duplication

Importance

Duplication Side-by-Side

Filter issues like