Bankiru\Api\Rpc\Cache\RouterCacheWarmer

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

50%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	warmUp()	0	6	2
A	isOptional()	0	4	1

<?php

namespace Bankiru\Api\Rpc\Cache;

use Bankiru\Api\Rpc\Routing\MethodMatcher;
use Bankiru\Api\Rpc\Routing\Router;
use Symfony\Component\HttpKernel\CacheWarmer\CacheWarmerInterface;
use Symfony\Component\HttpKernel\CacheWarmer\WarmableInterface;

final class RouterCacheWarmer implements CacheWarmerInterface
{
    const CACHE_DIR = 'rpc';

    /**
     * @var Router
     */
    private $router;

    /**
     * Constructor.
     *
     * @param MethodMatcher $router A Router instance
     */
    public function __construct(MethodMatcher $router)
    {
        $this->router = $router;

$router is of type object<Bankiru\Api\Rpc\Routing\MethodMatcher>, but the property $router was declared to be of type object<Bankiru\Api\Rpc\Routing\Router>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * Warms up the cache.
     *
     * @param string $cacheDir The cache directory
     */
    public function warmUp($cacheDir)
    {
        if ($this->router instanceof WarmableInterface) {
            $this->router->warmUp($cacheDir);
        }
    }

    /**
     * Checks whether this warmer is optional or not.
     *
     * @return bool always true
     */
    public function isOptional()
    {
        return true;
    }
}