Passed
Pull Request — master (#1045)
by Aleksei
09:49
created

Resolver   A

Complexity

Total Complexity 2

Size/Duplication

Total Lines 19
Duplicated Lines 0 %

Test Coverage

Coverage 54.55%

Importance

Changes 1
Bugs 0 Features 0
Metric Value
wmc 2
eloc 10
c 1
b 0
f 0
dl 0
loc 19
ccs 6
cts 11
cp 0.5455
rs 10

1 Method

Rating   Name   Duplication   Size   Complexity  
A resolve() 0 17 2
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Spiral\Core\Internal\Proxy;
6
7
use Spiral\Core\ContainerScope;
8
use Spiral\Core\Exception\Container\ContainerException;
9
10
/**
11
 * @internal
12
 */
13
final class Resolver
14
{
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(
0 ignored issues
show
Unused Code introduced by
The call to Psr\Container\ContainerInterface::get() has too many arguments starting with $context. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

20
            $result = $c->/** @scrutinizer ignore-call */ get($alias, $context) ?? throw new ContainerException(

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.

Loading history...
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
    }
33
}
34