Consigliere\Components\Commands\InstallCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	138
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	fire()	0	15	2
A	installFromFile()	0	22	3
B	install()	0	29	6
A	getArguments()	0	7	1
A	getOptions()	0	10	1

<?php

namespace Consigliere\Components\Commands;

use Illuminate\Console\Command as ComponentCommand;
use Consigliere\Components\Json;
use Consigliere\Components\Process\Installer;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputOption;

class InstallCommand extends ComponentCommand
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'component:install';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Install the specified component by given package name (vendor/name).';

    /**
     * Create a new command instance.
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function fire()
    {
        if (is_null($this->argument('name'))) {
            $this->installFromFile();

            return;
        }

        $this->install(
            $this->argument('name'),

It seems like $this->argument('name') targeting Illuminate\Console\Command::argument() can also be of type array; however, Consigliere\Components\Commands\InstallCommand::install() does only seem to accept string, maybe add an additional type check?

            $this->argument('version'),

It seems like $this->argument('version') targeting Illuminate\Console\Command::argument() can also be of type array; however, Consigliere\Components\Commands\InstallCommand::install() does only seem to accept string, maybe add an additional type check?

            $this->option('type'),

It seems like $this->option('type') targeting Illuminate\Console\Command::option() can also be of type array; however, Consigliere\Components\Commands\InstallCommand::install() does only seem to accept string, maybe add an additional type check?

            $this->option('tree')

$this->option('tree') is of type string|array, but the function expects a boolean.

        );
    }

    /**
     * Install components from components.json file.
     */
    protected function installFromFile()
    {
        if (!file_exists($path = base_path('components.json'))) {
            $this->error("File 'components.json' does not exist in your project root.");

            return;
        }

        $components = Json::make($path);

        $dependencies = $components->get('require', []);

        foreach ($dependencies as $component) {
            $component = collect($component);

            $this->install(
                $component->get('name'),
                $component->get('version'),
                $component->get('type')
            );
        }
    }

    /**
     * Install the specified component.
     *
     * @param string $name
     * @param string $version
     * @param string $type
     * @param bool $tree
     */
    protected function install($name, $version = 'dev-master', $type = 'composer', $tree = false)
    {
        $installer = new Installer(
            $name,
            $version,
            $type ?: $this->option('type'),

It seems like $type ?: $this->option('type') can also be of type array; however, Consigliere\Components\Process\Installer::__construct() does only seem to accept string|null, maybe add an additional type check?

            $tree ?: $this->option('tree')
        );

        $installer->setRepository($this->laravel['components']);

        $installer->setConsole($this);

$this is of type this<Consigliere\Components\Commands\InstallCommand>, but the function expects a object<Consigliere\Components\Process\Command>.

        if ($timeout = $this->option('timeout')) {
            $installer->setTimeout($timeout);

$timeout is of type string|array, but the function expects a integer.

        }

        if ($path = $this->option('path')) {
            $installer->setPath($path);

It seems like $path defined by $this->option('path') on line 107 can also be of type array; however, Consigliere\Components\Process\Installer::setPath() does only seem to accept string, maybe add an additional type check?

        }

        $installer->run();

        if (!$this->option('no-update')) {
            $this->call('component:update', [
                'component' => $installer->getComponentName(),
            ]);
        }
    }

    /**
     * Get the console command arguments.
     *
     * @return array
     */
    protected function getArguments()
    {
        return [
            ['name', InputArgument::OPTIONAL, 'The name of component will be installed.'],
            ['version', InputArgument::OPTIONAL, 'The version of component will be installed.'],
        ];
    }

    /**
     * Get the console command options.
     *
     * @return array
     */
    protected function getOptions()
    {
        return [
            ['timeout', null, InputOption::VALUE_OPTIONAL, 'The process timeout.', null],
            ['path', null, InputOption::VALUE_OPTIONAL, 'The installation path.', null],
            ['type', null, InputOption::VALUE_OPTIONAL, 'The type of installation.', null],
            ['tree', null, InputOption::VALUE_NONE, 'Install the component as a git subtree', null],
            ['no-update', null, InputOption::VALUE_NONE, 'Disables the automatic update of the dependencies.', null],
        ];
    }
}