1
|
|
|
<?php |
2
|
|
|
declare(strict_types=1); |
3
|
|
|
|
4
|
|
|
namespace League\Tactician\PHPStan; |
5
|
|
|
|
6
|
|
|
use League\Tactician\Handler\Mapping\CommandToHandlerMapping; |
7
|
|
|
use PhpParser\Node; |
8
|
|
|
use PhpParser\Node\Expr\MethodCall; |
9
|
|
|
use PHPStan\Analyser\Scope; |
10
|
|
|
use PHPStan\Broker\Broker; |
11
|
|
|
use PHPStan\Broker\ClassNotFoundException; |
12
|
|
|
use PHPStan\Reflection\MissingMethodFromReflectionException; |
13
|
|
|
use PHPStan\Reflection\ParametersAcceptorSelector; |
14
|
|
|
use PHPStan\Rules\Rule; |
15
|
|
|
use PHPStan\Type\ObjectType; |
16
|
|
|
use PHPStan\Type\Type; |
17
|
|
|
use PHPStan\Type\TypeWithClassName; |
18
|
|
|
use PHPStan\Type\UnionType; |
19
|
|
|
use function array_filter; |
20
|
|
|
use function array_merge; |
21
|
|
|
|
22
|
|
|
final class TacticianRuleSet implements Rule |
23
|
|
|
{ |
24
|
|
|
/** |
25
|
|
|
* @var CommandToHandlerMapping |
26
|
|
|
*/ |
27
|
|
|
private $mapping; |
28
|
|
|
/** |
29
|
|
|
* @var Broker |
30
|
|
|
*/ |
31
|
|
|
private $broker; |
32
|
|
|
/** |
33
|
|
|
* @var string |
34
|
|
|
*/ |
35
|
|
|
private $commandBusClass; |
36
|
|
|
|
37
|
13 |
|
public function __construct(CommandToHandlerMapping $mapping, Broker $broker, string $commandBusClass) |
38
|
|
|
{ |
39
|
13 |
|
$this->mapping = $mapping; |
40
|
13 |
|
$this->broker = $broker; |
41
|
13 |
|
$this->commandBusClass = $commandBusClass; |
42
|
13 |
|
} |
43
|
|
|
|
44
|
13 |
|
public function getNodeType(): string |
45
|
|
|
{ |
46
|
13 |
|
return MethodCall::class; |
47
|
|
|
} |
48
|
|
|
|
49
|
13 |
|
public function processNode(Node $methodCall, Scope $scope): array |
50
|
|
|
{ |
51
|
13 |
|
if (! $methodCall instanceof MethodCall) { |
52
|
|
|
return []; |
53
|
|
|
} |
54
|
|
|
|
55
|
13 |
|
$type = $scope->getType($methodCall->var); |
56
|
|
|
|
57
|
13 |
|
if (! (new ObjectType($this->commandBusClass))->isSuperTypeOf($type)->yes()) { |
58
|
1 |
|
return []; |
59
|
|
|
} |
60
|
|
|
|
61
|
|
|
// Wrong number of arguments passed to handle? Delegate to other PHPStan rules |
62
|
12 |
|
if (count($methodCall->args) !== 1) { |
63
|
1 |
|
return []; // |
64
|
|
|
} |
65
|
|
|
|
66
|
11 |
|
$commandType = $scope->getType($methodCall->args[0]->value); |
67
|
|
|
|
68
|
11 |
|
$errors = []; |
69
|
11 |
|
foreach ($this->getInspectableCommandTypes($commandType) as $commandType) { |
70
|
11 |
|
$errors = array_merge( |
71
|
11 |
|
$errors, |
72
|
11 |
|
$this->inspectCommandType($methodCall, $scope, $commandType) |
73
|
|
|
); |
74
|
|
|
} |
75
|
|
|
|
76
|
11 |
|
return $errors; |
77
|
|
|
} |
78
|
|
|
|
79
|
|
|
/** |
80
|
|
|
* @return array<string> |
81
|
|
|
*/ |
82
|
11 |
|
private function inspectCommandType( |
83
|
|
|
MethodCall $methodCallOnBus, |
84
|
|
|
Scope $scope, |
85
|
|
|
TypeWithClassName $commandType |
86
|
|
|
): array { |
87
|
11 |
|
$handlerClassName = $this->mapping->getClassName($commandType->getClassName()); |
88
|
|
|
|
89
|
|
|
try { |
90
|
11 |
|
$handlerClass = $this->broker->getClass($handlerClassName); |
91
|
2 |
|
} catch (ClassNotFoundException $e) { |
92
|
|
|
return [ |
93
|
2 |
|
"Tactician tried to route the command {$commandType->getClassName()} but could not find the matching " . |
94
|
2 |
|
"handler {$handlerClassName}.", |
95
|
|
|
]; |
96
|
|
|
} |
97
|
|
|
|
98
|
10 |
|
$handlerMethodName = $this->mapping->getMethodName($commandType->getClassName()); |
99
|
|
|
|
100
|
|
|
try { |
101
|
10 |
|
$handlerMethod = $handlerClass->getMethod($handlerMethodName, $scope); |
102
|
1 |
|
} catch (MissingMethodFromReflectionException $e) { |
103
|
|
|
return [ |
104
|
1 |
|
"Tactician tried to route the command {$commandType->getClassName()} to " . |
105
|
1 |
|
"{$handlerClass->getName()}::{$handlerMethodName} but while the class could be loaded, the method " . |
106
|
1 |
|
"'{$handlerMethodName}' could not be found on the class.", |
107
|
|
|
]; |
108
|
|
|
} |
109
|
|
|
|
110
|
|
|
/** @var \PHPStan\Reflection\ParameterReflection[] $parameters */ |
111
|
9 |
|
$parameters = ParametersAcceptorSelector::selectFromArgs( |
112
|
9 |
|
$scope, |
113
|
9 |
|
$methodCallOnBus->args, |
114
|
9 |
|
$handlerMethod->getVariants() |
115
|
9 |
|
)->getParameters(); |
116
|
|
|
|
117
|
9 |
|
if (count($parameters) === 0) { |
118
|
|
|
return [ |
119
|
2 |
|
"Tactician tried to route the command {$commandType->getClassName()} to " . |
120
|
2 |
|
"{$handlerClass->getName()}::{$handlerMethodName} but the method '{$handlerMethodName}' does not " . |
121
|
2 |
|
"accept any parameters.", |
122
|
|
|
]; |
123
|
|
|
} |
124
|
|
|
|
125
|
7 |
|
if (count($parameters) > 1) { |
126
|
|
|
return [ |
127
|
1 |
|
"Tactician tried to route the command {$commandType->getClassName()} to " . |
128
|
1 |
|
"{$handlerClass->getName()}::{$handlerMethodName} but the method '{$handlerMethodName}' accepts " . |
129
|
1 |
|
"too many parameters.", |
130
|
|
|
]; |
131
|
|
|
} |
132
|
|
|
|
133
|
6 |
|
if ($parameters[0]->getType()->accepts($commandType, true)->no()) { |
134
|
|
|
return [ |
135
|
2 |
|
"Tactician tried to route the command {$commandType->getClassName()} to " . |
136
|
2 |
|
"{$handlerClass->getName()}::{$handlerMethodName} but the method '{$handlerMethodName}' has a " . |
137
|
2 |
|
"typehint that does not allow this command.", |
138
|
|
|
]; |
139
|
|
|
} |
140
|
|
|
|
141
|
4 |
|
return []; |
142
|
|
|
} |
143
|
|
|
|
144
|
|
|
/** @return TypeWithClassName[] */ |
145
|
11 |
|
private function getInspectableCommandTypes(Type $type): array |
146
|
|
|
{ |
147
|
11 |
|
if ($type instanceof TypeWithClassName) { |
148
|
10 |
|
return [$type]; |
149
|
|
|
} |
150
|
|
|
|
151
|
1 |
|
if ($type instanceof UnionType) { |
152
|
1 |
|
return array_filter( |
153
|
1 |
|
$type->getTypes(), |
154
|
|
|
function (Type $type) { |
155
|
1 |
|
return $type instanceof TypeWithClassName; |
156
|
1 |
|
} |
157
|
|
|
); |
158
|
|
|
} |
159
|
|
|
|
160
|
|
|
return []; |
161
|
|
|
} |
162
|
|
|
} |
163
|
|
|
|