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

Complexity

Conditions	9
Paths	7

Size

Total Lines	59
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	29
CRAP Score	9.2578

Importance

Changes

0

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

        $em          = $this->getHelper('em')->getEntityManager();
        $entityClass = $input->getArgument('entity-class');
        $entityId    = $input->getArgument('entity-id');
        $cache       = $em->getCache();

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

        if ( ! $entityClass && ! $input->getOption('all')) {
            throw new \InvalidArgumentException('Invalid argument "--entity-class"');
        }

        if ($input->getOption('flush')) {
            $entityRegion  = $cache->getEntityCacheRegion($entityClass);

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

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

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

            $ui->comment(sprintf('Flushing cache provider configured for entity named <info>"%s"</info>', $entityClass));

It seems like $entityClass 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 entity regions');

            $cache->evictEntityRegions();

            return 0;
        }

        if ($entityId) {
            $ui->comment(
                sprintf(
                    'Clearing second-level cache entry for entity <info>"%s"</info> identified by <info>"%s"</info>',
                    $entityClass,
                    $entityId
                )
            );
            $cache->evictEntity($entityClass, $entityId);

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

            return 0;
        }

        $ui->comment(sprintf('Clearing second-level cache for entity <info>"%s"</info>', $entityClass));
        $cache->evictEntityRegion($entityClass);

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

        return 0;
    }