Consolidation\SiteProcess\Remo\RemoCommands

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	run()	0	22	3

<?php

namespace Consolidation\SiteProcess\Remo;

use Consolidation\SiteProcess\SiteProcess;

use Consolidation\SiteAlias\SiteAliasManagerAwareInterface;
use Consolidation\SiteAlias\SiteAliasManagerAwareTrait;
use Consolidation\SiteAlias\SiteAliasManager;

class RemoCommands extends \Robo\Tasks
{
    use SiteAliasManagerAwareTrait;

    /**
     * Run a command identified by a site alias
     *
     * @command run
     */
    public function run($aliasName, array $args, $options = ['foo' => 'bar'])

The parameter $options is not used and could be removed.

    {
        // The site alias manager has not been added to the DI container yet.
        if (!$this->hasSiteAliasManager()) {
            // TODO: Provide some way to initialize the alias file loaders, so
            // that there is some way to specify where alias files may be
            // loaded from.
            $manager = new SiteAliasManager();
            // $manager->setRoot($root);
            $this->setSiteAliasManager($manager);
        }

        // In theory this might do something once we get an alias manager.
        $siteAlias = $this->siteAliasManager()->get($aliasName);
        if (!$siteAlias) {
            throw new \Exception("Alias name $aliasName not found.");
        }
        $process = new SiteProcess($siteAlias, $args);

The call to SiteProcess::__construct() misses a required argument $args.

$args is of type array, but the function expects a object<Consolidation\SiteProcess\Transport\TransportInterface>.

        $process->setRealtimeOutput($this->io());

The method Robo\Common\IO::io() has been deprecated with message: Use a style injector instead

        $process->setTty($this->input()->isInteractive());
        $process->mustRun($process->showRealtime());
    }
}