-
Notifications
You must be signed in to change notification settings - Fork 1
Expand file tree
/
Copy pathForbidRawSqlRule.php
More file actions
60 lines (47 loc) · 1.54 KB
/
ForbidRawSqlRule.php
File metadata and controls
60 lines (47 loc) · 1.54 KB
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
<?php
declare(strict_types=1);
namespace PhpStanMigrationRules\Rules\Phinx;
use PhpParser\Node;
use PhpParser\Node\Expr\MethodCall;
use PhpParser\Node\Identifier;
use PHPStan\Analyser\Scope;
use PHPStan\Rules\RuleErrorBuilder;
/**
* @extends PhinxRule<MethodCall>
*/
final class ForbidRawSqlRule extends PhinxRule
{
private const string RULE_IDENTIFIER = 'phinx.schema.rawSqlForbidden';
private const string MESSAGE =
'Forbidden: raw SQL via %s(). '
. 'Why: raw SQL bypasses the schema builder, making migrations harder to review, less portable, and prone to errors. '
. 'Fix: use Phinx schema builder methods instead.';
private const array FORBIDDEN_METHODS = ['execute', 'query'];
public function getNodeType(): string
{
return MethodCall::class;
}
public function processNode(Node $node, Scope $scope): array
{
if (!$this->isPhinxMigration($scope)) {
return [];
}
$methodName = $this->getForbiddenMethodName($node);
if ($methodName === null) {
return [];
}
return [
RuleErrorBuilder::message(sprintf(self::MESSAGE, $methodName))
->identifier(self::RULE_IDENTIFIER)
->build(),
];
}
private function getForbiddenMethodName(MethodCall $node): ?string
{
if (!$node->name instanceof Identifier) {
return null;
}
$name = $node->name->toString();
return in_array($name, self::FORBIDDEN_METHODS, true) ? $name : null;
}
}