SimpleDIC\Console\CacheDestroyCommand

Complexity

Total Complexity

9

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
B	execute()	0	27	8

<?php

namespace SimpleDIC\Console;

use SimpleDIC\DIC;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Helper\Table;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;

class CacheDestroyCommand extends Command
{
    protected function configure()
    {
        $this
            ->setName('dic:cache-destroy')
            ->setDescription('Destroy the cache created by DIC.')
            ->setHelp('This command try to destroy cache files created by DIC and clear apcu cache.')
            ->addArgument('cache_dir', InputArgument::OPTIONAL)
        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $cacheDir = !empty($input->getArgument('cache_dir')) ? $input->getArgument('cache_dir') : __DIR__.'/../../_cache';

        if (false === is_dir($cacheDir)) {

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

            throw new \InvalidArgumentException($cacheDir . ' is not a valid dir');

Are you sure $cacheDir of type null|string|string[] can be used in concatenation?

        }

        foreach (scandir($cacheDir) as $file) {

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

            if (!in_array($file, ['.', '..'])) {

                // destroy apcu
                if (extension_loaded('apc') && ini_get('apc.enabled')) {
                    $filePath = $cacheDir . DIRECTORY_SEPARATOR . $file;
                    $array = include($filePath);

                    foreach ($array as $id => $entry) {
                        apcu_delete(md5(sha1_file($filePath). DIRECTORY_SEPARATOR .$id));
                    }
                }

                // delete file
                unlink($filePath);

The variable $filePath does not seem to be defined for all execution paths leading up to this point.

            }
        }

        $output->writeln('<fg=green>Cache was successfully cleared.</>');
    }
}