FondBot\Toolbelt\InstallDriver

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	handle()	0	32	4

<?php

declare(strict_types=1);

namespace FondBot\Toolbelt;

use FondBot\Foundation\API;
use Illuminate\Console\Command;
use FondBot\Foundation\Composer;

class InstallDriver extends Command
{
    protected $signature = 'fondbot:driver-install 
                            {name : Driver name to be installed}';

    protected $description = 'Install driver';

    public function handle(API $api, Composer $composer): void
    {
        // Check if package is listed in store
        $name = $this->argument('name');
        $driver = $api->findDriver($this->argument('name'));

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

        if ($driver === null) {
            $this->error('"'.$name.'" is not found in the available drivers list.');

            exit(0);
        }

        if ($composer->installed($driver['package'])) {
            $this->error('Driver is already installed.');

            return;
        }

        // Install driver
        $this->info('Installing driver...');

        $result = $composer->install($driver['package'], function ($_, $line) use (&$output) {
            $output .= $line;
        });

        if ($result !== 0) {
            $this->error($output);
            exit($result);
        }

        $this->info('Driver installed.');
    }
}