DamienHarper /
NavigationBundle
| Conditions | 1 |
| Paths | 1 |
| Total Lines | 12 |
| Code Lines | 8 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 0 |
| 1 | <?php |
||
| 19 | protected static function configureOptionResolver(OptionsResolver $resolver): void |
||
| 20 | { |
||
| 21 | $resolver->setDefaults([ |
||
| 22 | 'http_client' => null, |
||
| 23 | 'region' => null, |
||
| 24 | 'api_key' => null, |
||
| 25 | ]); |
||
| 26 | |||
| 27 | $resolver->setAllowedTypes('http_client', ['object', 'null']); |
||
| 28 | $resolver->setAllowedTypes('api_key', ['string', 'null']); |
||
| 29 | $resolver->setAllowedTypes('region', ['string', 'null']); |
||
| 30 | $resolver->setRequired(['api_key']); |
||
| 31 | } |
||
| 33 |
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.