Skip to content

Commit 93e808d

Browse files
ruudkruudk
committed
Use named argument in error for variadic types
When a type is variadic, and multiple named arguments are used, it's often hard to tell for which argument the error is coming. Since we have the named argument, we could use it in this situation.
1 parent e819f58 commit 93e808d

13 files changed

+48
-35
lines changed

‎src/Rules/AttributesCheck.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -144,14 +144,14 @@ public function check(
144144
'Attribute class ' . $attributeClassName . ' constructor invoked with %d parameters, at least %d required.',
145145
'Attribute class ' . $attributeClassName . ' constructor invoked with %d parameter, %d-%d required.',
146146
'Attribute class ' . $attributeClassName . ' constructor invoked with %d parameters, %d-%d required.',
147-
'Parameter %s of attribute class ' . $attributeClassName . ' constructor expects %s, %s given.',
147+
'%s of attribute class ' . $attributeClassName . ' constructor expects %s, %s given.',
148148
'', // constructor does not have a return type
149-
'Parameter %s of attribute class ' . $attributeClassName . ' constructor is passed by reference, so it expects variables only',
149+
'%s of attribute class ' . $attributeClassName . ' constructor is passed by reference, so it expects variables only',
150150
'Unable to resolve the template type %s in instantiation of attribute class ' . $attributeClassName,
151151
'Missing parameter $%s in call to ' . $attributeClassName . ' constructor.',
152152
'Unknown parameter $%s in call to ' . $attributeClassName . ' constructor.',
153153
'Return type of call to ' . $attributeClassName . ' constructor contains unresolvable type.',
154-
'Parameter %s of attribute class ' . $attributeClassName . ' constructor contains unresolvable type.',
154+
'%s of attribute class ' . $attributeClassName . ' constructor contains unresolvable type.',
155155
'Attribute class ' . $attributeClassName . ' constructorinvoked with %s, but it\'s not allowed because of @no-named-arguments.',
156156
);
157157

‎src/Rules/Classes/InstantiationRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -209,14 +209,14 @@ private function checkClassName(string $class, bool $isName, Node $node, Scope $
209209
'Class ' . $classDisplayName . ' constructor invoked with %d parameters, at least %d required.',
210210
'Class ' . $classDisplayName . ' constructor invoked with %d parameter, %d-%d required.',
211211
'Class ' . $classDisplayName . ' constructor invoked with %d parameters, %d-%d required.',
212-
'Parameter %s of class ' . $classDisplayName . ' constructor expects %s, %s given.',
212+
'%s of class ' . $classDisplayName . ' constructor expects %s, %s given.',
213213
'', // constructor does not have a return type
214-
'Parameter %s of class ' . $classDisplayName . ' constructor is passed by reference, so it expects variables only',
214+
' %s of class ' . $classDisplayName . ' constructor is passed by reference, so it expects variables only',
215215
'Unable to resolve the template type %s in instantiation of class ' . $classDisplayName,
216216
'Missing parameter $%s in call to ' . $classDisplayName . ' constructor.',
217217
'Unknown parameter $%s in call to ' . $classDisplayName . ' constructor.',
218218
'Return type of call to ' . $classDisplayName . ' constructor contains unresolvable type.',
219-
'Parameter %s of class ' . $classDisplayName . ' constructor contains unresolvable type.',
219+
'%s of class ' . $classDisplayName . ' constructor contains unresolvable type.',
220220
'Class ' . $classDisplayName . ' constructor invoked with %s, but it\'s not allowed because of @no-named-arguments.',
221221
));
222222
}

‎src/Rules/FunctionCallParametersCheck.php‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ public function check(
331331
$verbosityLevel = VerbosityLevel::getRecommendedLevelByType($parameterType, $argumentValueType);
332332
$errors[] = RuleErrorBuilder::message(sprintf(
333333
$wrongArgumentTypeMessage,
334-
$this->describeParameter($parameter, $argumentName === null ? $i + 1 : null),
334+
$this->describeParameter($parameter, $argumentName === null ? $i + 1 : $argumentName),
335335
$parameterType->describe($verbosityLevel),
336336
$argumentValueType->describe($verbosityLevel),
337337
))
@@ -409,7 +409,7 @@ public function check(
409409
}
410410

411411
$errors[] = RuleErrorBuilder::message(sprintf(
412-
'Parameter %s is passed by reference so it does not accept %s.',
412+
'%s is passed by reference so it does not accept %s.',
413413
$this->describeParameter($parameter, $argumentName === null ? $i + 1 : null),
414414
$propertyDescription,
415415
))->identifier('argument.byRef')->line($argumentLine)->build();
@@ -625,11 +625,15 @@ private function processArguments(
625625
return [$errors, $newArguments];
626626
}
627627

628-
private function describeParameter(ParameterReflection $parameter, ?int $position): string
628+
private function describeParameter(ParameterReflection $parameter, null|int|string $positionOrNamed): string
629629
{
630630
$parts = [];
631-
if ($position !== null) {
632-
$parts[] = '#' . $position;
631+
if (is_int($positionOrNamed)) {
632+
$parts[] = 'Parameter #' . $positionOrNamed;
633+
} elseif (is_string($positionOrNamed)) {
634+
$parts[] = 'Named argument ' . $positionOrNamed . ' for variadic parameter';
635+
} else {
636+
$parts[] = 'Parameter';
633637
}
634638

635639
$name = $parameter->getName();

‎src/Rules/Functions/CallCallablesRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,14 +126,14 @@ public function processNode(
126126
ucfirst($callableDescription) . ' invoked with %d parameters, at least %d required.',
127127
ucfirst($callableDescription) . ' invoked with %d parameter, %d-%d required.',
128128
ucfirst($callableDescription) . ' invoked with %d parameters, %d-%d required.',
129-
'Parameter %s of ' . $callableDescription . ' expects %s, %s given.',
129+
'%s of ' . $callableDescription . ' expects %s, %s given.',
130130
'Result of ' . $callableDescription . ' (void) is used.',
131-
'Parameter %s of ' . $callableDescription . ' is passed by reference, so it expects variables only.',
131+
'%s of ' . $callableDescription . ' is passed by reference, so it expects variables only.',
132132
'Unable to resolve the template type %s in call to ' . $callableDescription,
133133
'Missing parameter $%s in call to ' . $callableDescription . '.',
134134
'Unknown parameter $%s in call to ' . $callableDescription . '.',
135135
'Return type of call to ' . $callableDescription . ' contains unresolvable type.',
136-
'Parameter %s of ' . $callableDescription . ' contains unresolvable type.',
136+
'%s of ' . $callableDescription . ' contains unresolvable type.',
137137
ucfirst($callableDescription) . ' invoked with %s, but it\'s not allowed because of @no-named-arguments.',
138138
),
139139
);

‎src/Rules/Functions/CallToFunctionParametersRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,14 @@ public function processNode(Node $node, Scope $scope): array
5757
'Function ' . $functionName . ' invoked with %d parameters, at least %d required.',
5858
'Function ' . $functionName . ' invoked with %d parameter, %d-%d required.',
5959
'Function ' . $functionName . ' invoked with %d parameters, %d-%d required.',
60-
'Parameter %s of function ' . $functionName . ' expects %s, %s given.',
60+
'%s of function ' . $functionName . ' expects %s, %s given.',
6161
'Result of function ' . $functionName . ' (void) is used.',
62-
'Parameter %s of function ' . $functionName . ' is passed by reference, so it expects variables only.',
62+
'%s of function ' . $functionName . ' is passed by reference, so it expects variables only.',
6363
'Unable to resolve the template type %s in call to function ' . $functionName,
6464
'Missing parameter $%s in call to function ' . $functionName . '.',
6565
'Unknown parameter $%s in call to function ' . $functionName . '.',
6666
'Return type of call to function ' . $functionName . ' contains unresolvable type.',
67-
'Parameter %s of function ' . $functionName . ' contains unresolvable type.',
67+
'%s of function ' . $functionName . ' contains unresolvable type.',
6868
'Function ' . $functionName . ' invoked with %s, but it\'s not allowed because of @no-named-arguments.',
6969
);
7070
}

‎src/Rules/Functions/CallUserFuncRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -73,14 +73,14 @@ public function processNode(Node $node, Scope $scope): array
7373
ucfirst($callableDescription) . ' invoked with %d parameters, at least %d required.',
7474
ucfirst($callableDescription) . ' invoked with %d parameter, %d-%d required.',
7575
ucfirst($callableDescription) . ' invoked with %d parameters, %d-%d required.',
76-
'Parameter %s of ' . $callableDescription . ' expects %s, %s given.',
76+
'%s of ' . $callableDescription . ' expects %s, %s given.',
7777
'Result of ' . $callableDescription . ' (void) is used.',
78-
'Parameter %s of ' . $callableDescription . ' is passed by reference, so it expects variables only.',
78+
'%s of ' . $callableDescription . ' is passed by reference, so it expects variables only.',
7979
'Unable to resolve the template type %s in call to ' . $callableDescription,
8080
'Missing parameter $%s in call to ' . $callableDescription . '.',
8181
'Unknown parameter $%s in call to ' . $callableDescription . '.',
8282
'Return type of call to ' . $callableDescription . ' contains unresolvable type.',
83-
'Parameter %s of ' . $callableDescription . ' contains unresolvable type.',
83+
'%s of ' . $callableDescription . ' contains unresolvable type.',
8484
ucfirst($callableDescription) . ' invoked with %s, but it\'s not allowed because of @no-named-arguments.',
8585
);
8686
}

‎src/Rules/Methods/CallMethodsRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,14 @@ public function processNode(Node $node, Scope $scope): array
6363
'Method ' . $messagesMethodName . ' invoked with %d parameters, at least %d required.',
6464
'Method ' . $messagesMethodName . ' invoked with %d parameter, %d-%d required.',
6565
'Method ' . $messagesMethodName . ' invoked with %d parameters, %d-%d required.',
66-
'Parameter %s of method ' . $messagesMethodName . ' expects %s, %s given.',
66+
'%s of method ' . $messagesMethodName . ' expects %s, %s given.',
6767
'Result of method ' . $messagesMethodName . ' (void) is used.',
68-
'Parameter %s of method ' . $messagesMethodName . ' is passed by reference, so it expects variables only.',
68+
'%s of method ' . $messagesMethodName . ' is passed by reference, so it expects variables only.',
6969
'Unable to resolve the template type %s in call to method ' . $messagesMethodName,
7070
'Missing parameter $%s in call to method ' . $messagesMethodName . '.',
7171
'Unknown parameter $%s in call to method ' . $messagesMethodName . '.',
7272
'Return type of call to method ' . $messagesMethodName . ' contains unresolvable type.',
73-
'Parameter %s of method ' . $messagesMethodName . ' contains unresolvable type.',
73+
'%s of method ' . $messagesMethodName . ' contains unresolvable type.',
7474
'Method ' . $messagesMethodName . ' invoked with %s, but it\'s not allowed because of @no-named-arguments.',
7575
));
7676
}

‎src/Rules/Methods/CallStaticMethodsRule.php‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,14 +71,14 @@ public function processNode(Node $node, Scope $scope): array
7171
$displayMethodName . ' invoked with %d parameters, at least %d required.',
7272
$displayMethodName . ' invoked with %d parameter, %d-%d required.',
7373
$displayMethodName . ' invoked with %d parameters, %d-%d required.',
74-
'Parameter %s of ' . $lowercasedMethodName . ' expects %s, %s given.',
74+
'%s of ' . $lowercasedMethodName . ' expects %s, %s given.',
7575
'Result of ' . $lowercasedMethodName . ' (void) is used.',
76-
'Parameter %s of ' . $lowercasedMethodName . ' is passed by reference, so it expects variables only.',
76+
'%s of ' . $lowercasedMethodName . ' is passed by reference, so it expects variables only.',
7777
'Unable to resolve the template type %s in call to method ' . $lowercasedMethodName,
7878
'Missing parameter $%s in call to ' . $lowercasedMethodName . '.',
7979
'Unknown parameter $%s in call to ' . $lowercasedMethodName . '.',
8080
'Return type of call to ' . $lowercasedMethodName . ' contains unresolvable type.',
81-
'Parameter %s of ' . $lowercasedMethodName . ' contains unresolvable type.',
81+
'%s of ' . $lowercasedMethodName . ' contains unresolvable type.',
8282
$displayMethodName . ' invoked with %s, but it\'s not allowed because of @no-named-arguments.',
8383
));
8484

‎tests/PHPStan/Rules/Classes/ClassAttributesRuleTest.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ public function testBug7171(): void
136136
{
137137
$this->analyse([__DIR__ . '/data/bug-7171.php'], [
138138
[
139-
'Parameter $repositoryClass of attribute class Bug7171\Entity constructor expects class-string<Bug7171\EntityRepository<T of object>>|null, \'stdClass\' given.',
139+
'Named argument repositoryClass for variadic parameter $repositoryClass of attribute class Bug7171\Entity constructor expects class-string<Bug7171\EntityRepository<T of object>>|null, \'stdClass\' given.',
140140
66,
141141
],
142142
]);

‎tests/PHPStan/Rules/Classes/InstantiationRuleTest.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -425,7 +425,7 @@ public function testBug7048(): void
425425
41,
426426
],
427427
[
428-
'Parameter $end of class DatePeriod constructor expects DateTimeInterface|int, string given.',
428+
'Named argument end for variadic parameter $end of class DatePeriod constructor expects DateTimeInterface|int, string given.',
429429
49,
430430
],
431431
]);

0 commit comments

Comments
 (0)