-
Notifications
You must be signed in to change notification settings - Fork 563
Expand file tree
/
Copy pathMissingMethodParameterTypehintRule.php
More file actions
137 lines (119 loc) · 4.53 KB
/
MissingMethodParameterTypehintRule.php
File metadata and controls
137 lines (119 loc) · 4.53 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
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
<?php declare(strict_types = 1);
namespace PHPStan\Rules\Methods;
use PhpParser\Node;
use PHPStan\Analyser\Scope;
use PHPStan\DependencyInjection\RegisteredRule;
use PHPStan\DependencyInjection\ValidatesStubFiles;
use PHPStan\Node\InClassMethodNode;
use PHPStan\Reflection\MethodReflection;
use PHPStan\Rules\IdentifierRuleError;
use PHPStan\Rules\MissingTypehintCheck;
use PHPStan\Rules\Rule;
use PHPStan\Rules\RuleErrorBuilder;
use PHPStan\Type\MixedType;
use PHPStan\Type\Type;
use PHPStan\Type\VerbosityLevel;
use function sprintf;
/**
* @implements Rule<InClassMethodNode>
*/
#[RegisteredRule(level: 6)]
#[ValidatesStubFiles]
final class MissingMethodParameterTypehintRule implements Rule
{
public function __construct(
private MissingTypehintCheck $missingTypehintCheck,
)
{
}
public function getNodeType(): string
{
return InClassMethodNode::class;
}
public function processNode(Node $node, Scope $scope): array
{
$methodReflection = $node->getMethodReflection();
$messages = [];
foreach ($methodReflection->getParameters() as $parameterReflection) {
foreach ($this->checkMethodParameter($methodReflection, sprintf('parameter $%s', $parameterReflection->getName()), $parameterReflection->getType()) as $parameterMessage) {
$messages[] = $parameterMessage;
}
if ($parameterReflection->getClosureThisType() !== null) {
foreach ($this->checkMethodParameter($methodReflection, sprintf('@param-closure-this PHPDoc tag for parameter $%s', $parameterReflection->getName()), $parameterReflection->getClosureThisType()) as $parameterMessage) {
$messages[] = $parameterMessage;
}
}
if ($parameterReflection->getOutType() === null) {
continue;
}
foreach ($this->checkMethodParameter($methodReflection, sprintf('@param-out PHPDoc tag for parameter $%s', $parameterReflection->getName()), $parameterReflection->getOutType()) as $parameterMessage) {
$messages[] = $parameterMessage;
}
}
return $messages;
}
/**
* @return list<IdentifierRuleError>
*/
private function checkMethodParameter(MethodReflection $methodReflection, string $parameterMessage, Type $parameterType): array
{
if ($parameterType instanceof MixedType && !$parameterType->isExplicitMixed()) {
return [
RuleErrorBuilder::message(sprintf(
'Method %s::%s() has %s with no type specified.',
$methodReflection->getDeclaringClass()->getDisplayName(),
$methodReflection->getName(),
$parameterMessage,
))->identifier('missingType.parameter')->build(),
];
}
$messages = [];
foreach ($this->missingTypehintCheck->getIterableTypesWithMissingValueTypehint($parameterType) as $iterableType) {
$iterableTypeDescription = $iterableType->describe(VerbosityLevel::typeOnly());
$messages[] = RuleErrorBuilder::message(sprintf(
'Method %s::%s() has %s with no value type specified in iterable type %s.',
$methodReflection->getDeclaringClass()->getDisplayName(),
$methodReflection->getName(),
$parameterMessage,
$iterableTypeDescription,
))
->tip(MissingTypehintCheck::MISSING_ITERABLE_VALUE_TYPE_TIP)
->identifier('missingType.iterableValue')
->build();
}
foreach ($this->missingTypehintCheck->getNonGenericObjectTypesWithGenericClass($parameterType) as [$name, $genericTypeNames]) {
$messages[] = RuleErrorBuilder::message(sprintf(
'Method %s::%s() has %s with generic %s but does not specify its types: %s',
$methodReflection->getDeclaringClass()->getDisplayName(),
$methodReflection->getName(),
$parameterMessage,
$name,
$genericTypeNames,
))
->identifier('missingType.generics')
->build();
}
foreach ($this->missingTypehintCheck->getRawGenericTypeAliasesUsage($parameterType) as [$aliasName, $missingParams]) {
$messages[] = RuleErrorBuilder::message(sprintf(
'Method %s::%s() has %s with generic type alias %s but does not specify its types: %s',
$methodReflection->getDeclaringClass()->getDisplayName(),
$methodReflection->getName(),
$parameterMessage,
$aliasName,
$missingParams,
))
->identifier('missingType.generics')
->build();
}
foreach ($this->missingTypehintCheck->getCallablesWithMissingSignature($parameterType) as $callableType) {
$messages[] = RuleErrorBuilder::message(sprintf(
'Method %s::%s() has %s with no signature specified for %s.',
$methodReflection->getDeclaringClass()->getDisplayName(),
$methodReflection->getName(),
$parameterMessage,
$callableType->describe(VerbosityLevel::typeOnly()),
))->identifier('missingType.callable')->build();
}
return $messages;
}
}