Doctrine\ORM\Tools\Console\Command\ClearCache\QueryRegionCommand::execute()

Complexity

Conditions	7
Paths	10

Size

Total Lines	51
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	25
CRAP Score	7.1284

Importance

Changes

0

<?php
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $ui = new SymfonyStyle($input, $output);

        $em    = $this->getHelper('em')->getEntityManager();
        $name  = $input->getArgument('region-name');
        $cache = $em->getCache();

        if ($name === null) {
            $name = Cache::DEFAULT_QUERY_REGION_NAME;
        }

        if ( ! $cache instanceof Cache) {
            throw new \InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
        }

        if ($input->getOption('flush')) {
            $queryCache  = $cache->getQueryCache($name);

It seems like $name can also be of type string[]; however, parameter $regionName of Doctrine\ORM\Cache::getQueryCache() does only seem to accept null|string, maybe add an additional type check?

            $queryRegion = $queryCache->getRegion();

            if ( ! $queryRegion instanceof DefaultRegion) {
                throw new \InvalidArgumentException(sprintf(
                    'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
                    is_object($queryRegion) ? get_class($queryRegion) : gettype($queryRegion)
                ));
            }

            $queryRegion->getCache()->flushAll();

            $ui->comment(
                sprintf(
                    'Flushing cache provider configured for second-level cache query region named <info>"%s"</info>',
                    $name

It seems like $name can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

                )
            );

            return 0;
        }

        if ($input->getOption('all')) {
            $ui->comment('Clearing <info>all</info> second-level cache query regions');

            $cache->evictQueryRegions();

            return 0;
        }

        $ui->comment(sprintf('Clearing second-level cache query region named <info>"%s"</info>', $name));
        $cache->evictQueryRegion($name);

It seems like $name can also be of type string[]; however, parameter $regionName of Doctrine\ORM\Cache::evictQueryRegion() does only seem to accept null|string, maybe add an additional type check?

        return 0;
    }