Nubs\Which\Application\WhichCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	6	1
A	execute()	0	18	3

<?php
namespace Nubs\Which\Application;

use Symfony\Component\Console\Command\Command as SymfonyCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class WhichCommand extends SymfonyCommand
{
    /**
     * Configures the command's options.
     *
     * @return void
     */
    protected function configure()
    {
        $this->setName('which')
            ->setDescription('Locate a command in the PATH environment')
            ->addArgument('commands', InputArgument::REQUIRED | InputArgument::IS_ARRAY, 'The commands to locate');
    }

    /**
     * Locates the commands given as arguments.
     *
     * @param \Symfony\Component\Console\Input\InputInterface $input The command
     *     input.
     * @param \Symfony\Component\Console\Output\OutputInterface $output The
     *     command output.
     * @return int The exit status of the command.
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $locator = $this->getApplication()->getLocator();

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Console\Application as the method getLocator() does only exist in the following sub-classes of Symfony\Component\Console\Application: Nubs\Which\Application\WhichApplication. Maybe you want to instanceof check for one of these explicitly?

        $exitStatus = 0;

        foreach ($input->getArgument('commands') as $command) {
            $location = $locator->locate($command);

            if ($location === null) {
                $output->writeln("<error>{$command} not found</error>");
                $exitStatus = 1;
            } else {
                $output->writeln($location);
            }
        }

        return $exitStatus;
    }
}