| Conditions | 3 |
| Paths | 3 |
| Total Lines | 26 |
| Code Lines | 12 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 10 |
| CRAP Score | 3.3332 |
| Changes | 1 | ||
| Bugs | 0 | Features | 0 |
| 1 | <?php |
||
| 18 | 460 | public static function resolve( |
|
| 19 | string $alias, |
||
| 20 | \Stringable|string|null $context = null, |
||
| 21 | ?ContainerInterface $c = null |
||
| 22 | ): object { |
||
| 23 | 460 | $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.'); |
|
| 24 | |||
| 25 | try { |
||
| 26 | /** @psalm-suppress TooManyArguments */ |
||
| 27 | 457 | $result = $c->get($alias, $context) ?? throw new ContainerException( |
|
|
|
|||
| 28 | 457 | 'Resolved `null` from the container.', |
|
| 29 | 457 | ); |
|
| 30 | } catch (\Throwable $e) { |
||
| 31 | throw new ContainerException( |
||
| 32 | \sprintf('Unable to resolve `%s` in a Proxy.', $alias), |
||
| 33 | previous: $e, |
||
| 34 | ); |
||
| 35 | } |
||
| 36 | |||
| 37 | 457 | if (Proxy::isProxy($result)) { |
|
| 38 | 1 | throw new RecursiveProxyException( |
|
| 39 | 1 | \sprintf('Recursive proxy detected for `%s`.', $alias), |
|
| 40 | 1 | ); |
|
| 41 | } |
||
| 42 | |||
| 43 | 456 | return $result; |
|
| 44 | } |
||
| 46 |
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.