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