AppBundle\ShowUnusedPublicAssets\Command

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	execute()	0	9	1
A	configure()	0	9	1

<?php

namespace AppBundle\ShowUnusedPublicAssets;

use Symfony\Component\Console\Command\Command as BaseCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;

/**
 * Symfony-Console-Command wrapper for the ShowUnusedPublicAssets task.
 */
final class Command extends BaseCommand
{
    const ARGUMENT_PATH_TO_PUBLIC = 'pathToPublic';
    const ARGUMENT_PATH_TO_LOG_FILE = 'pathToLogFile';
    const OPTION_REG_EXP_TO_FIND_FILE = 'regExpToFindFile';
    const OPTION_PATH_TO_OUTPUT = 'pathToOutput';
    const OPTION_PATH_TO_BLACKLIST = 'pathToBlacklist';

    /**
     * @var Task
     */
    private $task;

    /**
     * @param Task $task
     */
    public function __construct(Task $task)
    {
        parent::__construct();
        $this->task = $task;
    }

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->setName('show-unused-public-assets')
             ->setDescription('Show a list of potentially unused public assets.')
             ->addArgument(self::ARGUMENT_PATH_TO_PUBLIC, InputArgument::REQUIRED, 'Path to the public web root of your project.')
             ->addArgument(self::ARGUMENT_PATH_TO_LOG_FILE, InputArgument::REQUIRED, 'Path to the web server\'s access log file.')
             ->addOption(self::OPTION_REG_EXP_TO_FIND_FILE, 'r', InputOption::VALUE_REQUIRED, 'Regular expression for the log file capturing the path of the accessed file as it\'s first capture group.', '#"(?:get|post) ([a-z0-9\_\-\.\/]*)#i')
             ->addOption(self::OPTION_PATH_TO_OUTPUT, 'o', InputOption::VALUE_REQUIRED, 'Path to the output file. If not set, it will be "potentially-unused-public-assets.txt" in the folder above the public web root.')
             ->addOption(self::OPTION_PATH_TO_BLACKLIST, 'b', InputOption::VALUE_REQUIRED, 'Path to a file containing a blacklist of regular expressions to exclude from the output. One regular expression per line, don\'t forget the delimiters. E.g.: ' . PHP_EOL . '#^/project/keepme.php#' . PHP_EOL . '#^/project/tmp/#' . PHP_EOL . '#.*Test.php#');
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->task->getUnusedPublicAssets(
            $input->getArgument(self::ARGUMENT_PATH_TO_PUBLIC),

It seems like $input->getArgument(self::ARGUMENT_PATH_TO_PUBLIC) can also be of type string[]; however, parameter $pathToPublic of AppBundle\ShowUnusedPublicAssets\Task::getUnusedPublicAssets() does only seem to accept string, maybe add an additional type check?

            $input->getArgument(self::ARGUMENT_PATH_TO_LOG_FILE),

It seems like $input->getArgument(self::ARGUMENT_PATH_TO_LOG_FILE) can also be of type string[]; however, parameter $pathToLogfile of AppBundle\ShowUnusedPublicAssets\Task::getUnusedPublicAssets() does only seem to accept string, maybe add an additional type check?

            $input->getOption(self::OPTION_REG_EXP_TO_FIND_FILE),

It seems like $input->getOption(self::OPTION_REG_EXP_TO_FIND_FILE) can also be of type string[]; however, parameter $regExpToFindFile of AppBundle\ShowUnusedPublicAssets\Task::getUnusedPublicAssets() does only seem to accept string, maybe add an additional type check?

            $input->getOption(self::OPTION_PATH_TO_OUTPUT),

It seems like $input->getOption(self::OPTION_PATH_TO_OUTPUT) can also be of type string[]; however, parameter $pathToOutput of AppBundle\ShowUnusedPublicAssets\Task::getUnusedPublicAssets() does only seem to accept null|string, maybe add an additional type check?

            $input->getOption(self::OPTION_PATH_TO_BLACKLIST),

It seems like $input->getOption(self::OPTION_PATH_TO_BLACKLIST) can also be of type string[]; however, parameter $pathToBlacklist of AppBundle\ShowUnusedPublicAssets\Task::getUnusedPublicAssets() does only seem to accept null|string, maybe add an additional type check?

            new SymfonyStyle($input, $output)
        );
    }
}