Passed
Pull Request — master (#1140)
by Aleksei
20:44
created

Resolver::resolve()   A

Complexity

Conditions 3
Paths 3

Size

Total Lines 26
Code Lines 12

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 10
CRAP Score 3.3332

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 12
c 1
b 0
f 0
dl 0
loc 26
ccs 10
cts 15
cp 0.6667
rs 9.8666
cc 3
nc 3
nop 3
crap 3.3332
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Spiral\Core\Internal\Proxy;
6
7
use Psr\Container\ContainerInterface;
8
use Spiral\Core\ContainerScope;
9
use Spiral\Core\Exception\Container\ContainerException;
10
use Spiral\Core\Exception\Container\RecursiveProxyException;
11
use Spiral\Core\Internal\Proxy;
12
13
/**
14
 * @internal
15
 */
16
final class Resolver
17
{
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(
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

27
            $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...
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
    }
45
}
46