forked from symplify/phpstan-rules
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathNoInlineStringRegexRule.php
139 lines (118 loc) · 3.38 KB
/
NoInlineStringRegexRule.php
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
<?php
declare(strict_types=1);
namespace Symplify\PHPStanRules\Rules;
use Nette\Utils\Strings;
use PhpParser\Node;
use PhpParser\Node\Arg;
use PhpParser\Node\Expr\FuncCall;
use PhpParser\Node\Expr\StaticCall;
use PhpParser\Node\Scalar\String_;
use PHPStan\Analyser\Scope;
use Symplify\PHPStanRules\NodeAnalyzer\RegexFuncCallAnalyzer;
use Symplify\PHPStanRules\NodeAnalyzer\RegexStaticCallAnalyzer;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Symplify\PHPStanRules\Tests\Rules\NoInlineStringRegexRule\NoInlineStringRegexRuleTest
*/
final class NoInlineStringRegexRule extends AbstractSymplifyRule
{
/**
* @var string
*/
public const ERROR_MESSAGE = 'Use local named constant instead of inline string for regex to explain meaning by constant name';
public function __construct(
private RegexFuncCallAnalyzer $regexFuncCallAnalyzer,
private RegexStaticCallAnalyzer $regexStaticCallAnalyzer
) {
}
/**
* @return array<class-string<Node>>
*/
public function getNodeTypes(): array
{
return [StaticCall::class, FuncCall::class];
}
/**
* @param StaticCall|FuncCall $node
* @return mixed[]|string[]
*/
public function process(Node $node, Scope $scope): array
{
if ($node instanceof FuncCall) {
return $this->processRegexFuncCall($node);
}
return $this->processRegexStaticCall($node);
}
public function getRuleDefinition(): RuleDefinition
{
return new RuleDefinition(self::ERROR_MESSAGE, [
new CodeSample(
<<<'CODE_SAMPLE'
class SomeClass
{
public function run($value)
{
return preg_match('#some_stu|ff#', $value);
}
}
CODE_SAMPLE
,
<<<'CODE_SAMPLE'
class SomeClass
{
/**
* @var string
*/
public const SOME_STUFF_REGEX = '#some_stu|ff#';
public function run($value)
{
return preg_match(self::SOME_STUFF_REGEX, $value);
}
}
CODE_SAMPLE
),
]);
}
/**
* @return string[]
*/
private function processRegexFuncCall(FuncCall $funcCall): array
{
if (! $this->regexFuncCallAnalyzer->isRegexFuncCall($funcCall)) {
return [];
}
$firstArgOrVariadicPlaceholder = $funcCall->args[0];
if (! $firstArgOrVariadicPlaceholder instanceof Arg) {
return [];
}
// it's not string → good
if (! $firstArgOrVariadicPlaceholder->value instanceof String_) {
return [];
}
return [self::ERROR_MESSAGE];
}
/**
* @return string[]
*/
private function processRegexStaticCall(StaticCall $staticCall): array
{
if (! $this->regexStaticCallAnalyzer->isRegexStaticCall($staticCall)) {
return [];
}
$argOrVariadicPlaceholder = $staticCall->args[1];
if (! $argOrVariadicPlaceholder instanceof Arg) {
return [];
}
$secondArgValue = $argOrVariadicPlaceholder->value;
// it's not string → good
if (! $secondArgValue instanceof String_) {
return [];
}
$regexValue = $secondArgValue->value;
if (Strings::length($regexValue) <= 7) {
return [];
}
return [self::ERROR_MESSAGE];
}
}