SimpleDIC\Console\CacheDestroyCommand::execute()

Complexity

Conditions	8
Paths	10

Size

Total Lines	27
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

<?php
    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.</>');
    }