Consolidation\SiteProcess\Remo\RemoCommands::run()

Complexity

Conditions	3
Paths	4

Size

Total Lines

22

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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());
    }
}