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.