Conditions | 2 |
Paths | 2 |
Total Lines | 20 |
Code Lines | 9 |
Lines | 0 |
Ratio | 0 % |
Tests | 6 |
CRAP Score | 2.3755 |
Changes | 1 | ||
Bugs | 0 | Features | 0 |
1 | <?php |
||
16 | 13 | public static function resolve( |
|
17 | string $alias, |
||
18 | \Stringable|string|null $context = null, |
||
19 | ?ContainerInterface $c = null |
||
20 | ): object { |
||
21 | 13 | $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.'); |
|
22 | |||
23 | try { |
||
24 | /** @psalm-suppress TooManyArguments */ |
||
25 | 11 | $result = $c->get($alias, $context) ?? throw new ContainerException( |
|
|
|||
26 | 11 | 'Resolved `null` from the container.', |
|
27 | 11 | ); |
|
28 | } catch (\Throwable $e) { |
||
29 | throw new ContainerException( |
||
30 | \sprintf('Unable to resolve `%s` in a Proxy.', $alias), |
||
31 | previous: $e, |
||
32 | ); |
||
33 | } |
||
34 | |||
35 | 11 | return $result; |
|
36 | } |
||
38 |
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.