Passed
Push — master ( 8bd912...d93388 )
by Alan
06:58 queued 02:20
created

Bridge/Symfony/Routing/CachedRouteNameResolver.php (1 issue)

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\CachedTrait;
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 CachedTrait;
27
28
    public const CACHE_KEY_PREFIX = 'route_name_';
29
30
    private $decorated;
31
32
    public function __construct(CacheItemPoolInterface $cacheItemPool, RouteNameResolverInterface $decorated)
33
    {
34
        $this->cacheItemPool = $cacheItemPool;
35
        $this->decorated = $decorated;
36
    }
37
38
    /**
39
     * {@inheritdoc}
40
     */
41
    public function getRouteName(string $resourceClass, $operationType /*, array $context = []*/): string
42
    {
43
        $context = \func_num_args() > 2 ? func_get_arg(2) : [];
44
        $cacheKey = self::CACHE_KEY_PREFIX.md5(serialize([$resourceClass, $operationType, $context['subresource_resources'] ?? null]));
45
46
        return $this->getCached($cacheKey, function () use ($resourceClass, $operationType, $context) {
47
            return $this->decorated->getRouteName($resourceClass, $operationType, $context);
0 ignored issues
show
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

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