Passed
Pull Request — 2.2 (#1908)
by Ben
03:10
created

CachedRouteNameResolver   A

Complexity

Total Complexity 3

Size/Duplication

Total Lines 25
Duplicated Lines 0 %

Importance

Changes 0
Metric Value
wmc 3
dl 0
loc 25
rs 10
c 0
b 0
f 0

2 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 4 1
A getRouteName() 0 7 2
1
<?php
2
3
/*
4
 * This file is part of the API Platform project.
5
 *
6
 * (c) Kévin Dunglas <[email protected]>
7
 *
8
 * For the full copyright and license information, please view the LICENSE
9
 * file that was distributed with this source code.
10
 */
11
12
declare(strict_types=1);
13
14
namespace ApiPlatform\Core\Bridge\Symfony\Routing;
15
16
use ApiPlatform\Core\Cache\CacheTrait;
17
use Psr\Cache\CacheItemPoolInterface;
18
19
/**
20
 * {@inheritdoc}
21
 *
22
 * @author Teoh Han Hui <[email protected]>
23
 */
24
final class CachedRouteNameResolver implements RouteNameResolverInterface
25
{
26
    use CacheTrait;
27
28
    const CACHE_KEY_PREFIX = 'route_name_';
29
30
    private $cacheItemPool;
31
    private $decorated;
32
33
    public function __construct(CacheItemPoolInterface $cacheItemPool, RouteNameResolverInterface $decorated)
34
    {
35
        $this->cacheItemPool = $cacheItemPool;
36
        $this->decorated = $decorated;
37
    }
38
39
    /**
40
     * {@inheritdoc}
41
     */
42
    public function getRouteName(string $resourceClass, $operationType /**, array $context = []**/): string
43
    {
44
        $context = \func_num_args() > 2 ? func_get_arg(2) : [];
45
        $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType, $context['subresource_resources'] ?? null]));
46
47
        return $this->getOrSave($cacheKey, $this->cacheItemPool, function () use ($resourceClass, $operationType, $context) {
48
            return $this->decorated->getRouteName($resourceClass, $operationType, $context);
0 ignored issues
show
Unused Code introduced by
The call to ApiPlatform\Core\Bridge\...terface::getRouteName() 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

48
            return $this->decorated->/** @scrutinizer ignore-call */ getRouteName($resourceClass, $operationType, $context);

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...
49
        });
50
    }
51
}
52