mirror of
https://github.com/danog/psalm.git
synced 2024-11-27 12:55:26 +01:00
43d91665c6
The text mentions `@no-named-arguments`, but the example shows `@no-named-params`. `@no-named-params` can not be found in the psalm source. Updated example to `@no-named-arguments`. [https://github.com/vimeo/psalm/search?q=no-named-params&unscoped_q=no-named-params](search for no-named-params) 1 result (this example) [https://github.com/vimeo/psalm/search?q=no-named-arguments&unscoped_q=no-named-arguments](search for no-named-arguments) 2 results (this example and in CommentAnalyzer.php)
96 lines
2.3 KiB
Markdown
96 lines
2.3 KiB
Markdown
# ParamNameMismatch
|
|
|
|
Emitted when method overrides a parent method but renames a param.
|
|
|
|
```php
|
|
<?php
|
|
|
|
class A {
|
|
public function foo(string $str, bool $b = false) : void {}
|
|
}
|
|
|
|
class AChild extends A {
|
|
public function foo(string $string, bool $b = false) : void {}
|
|
}
|
|
```
|
|
|
|
## Why is this bad?
|
|
|
|
PHP 8 introduces [named parameters](https://wiki.php.net/rfc/named_params) which allow developers to call methods with explicitly-named parameters;
|
|
|
|
```php
|
|
<?php
|
|
|
|
function callFoo(A $a) {
|
|
$a->foo(str: "hello");
|
|
}
|
|
```
|
|
|
|
In the first example passing `new AChild()` to `callFoo()` results in a fatal error, as AChild's definition of the method `foo()` doesn't have a parameter named `$str`.
|
|
|
|
## How to fix
|
|
|
|
You can change the child method param name to match:
|
|
|
|
```php
|
|
<?php
|
|
|
|
class A {
|
|
public function foo(string $str, bool $b = false) : void {}
|
|
}
|
|
|
|
class AChild extends A {
|
|
public function foo(string $str, bool $b = false) : void {}
|
|
}
|
|
```
|
|
|
|
This fix [can be applied automatically by Psalter](https://psalm.dev/docs/manipulating_code/fixing/#paramnamemismatch).
|
|
|
|
## Workarounds
|
|
|
|
### @no-named-arguments
|
|
|
|
Alternatively you can ignore this issue by adding a `@no-named-arguments` annotation to the parent method:
|
|
|
|
```php
|
|
<?php
|
|
|
|
class A {
|
|
/** @no-named-arguments */
|
|
public function foo(string $str, bool $b = false) : void {}
|
|
}
|
|
|
|
class AChild extends A {
|
|
public function foo(string $string, bool $b = false) : void {}
|
|
}
|
|
```
|
|
|
|
Any method with this annotation will be prevented (by Psalm) from being called with named parameters, so the original issue does not matter.
|
|
|
|
### Config allowNamedArgumentCalls="false"
|
|
|
|
This prevents any use of named params in your codebase. Ideal for self-contained projects, but less ideal for libraries.
|
|
|
|
It means the original code above will not emit any errors as long as the class `A` is defined in a directory that Psalm can scan.
|
|
|
|
### Config allowInternalNamedArgumentCalls="false"
|
|
|
|
For library authors Psalm supports a more nuanced flag that tells Psalm to prohibit any named parameter calls on `@internal` classes or methods.
|
|
|
|
With that config value, this is now allowed:
|
|
|
|
```php
|
|
<?php
|
|
|
|
/**
|
|
* @internal
|
|
*/
|
|
class A {
|
|
public function foo(string $str, bool $b = false) : void {}
|
|
}
|
|
|
|
class AChild extends A {
|
|
public function foo(string $string, bool $b = false) : void {}
|
|
}
|
|
```
|