App\Commands\Begin::handle()

Complexity

Conditions	5
Paths	3

Size

Total Lines	36
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

0

<?php
    public function handle(): void
    {
        $force = $this->option('force');
        $home = realpath($this->argument('home') ?: $this->cli->currentWorkingDirectory());

It seems like $this->argument('home') ?: $this->cli->currentWorkingDirectory() can also be of type array; however, parameter $path of realpath() does only seem to accept string, maybe add an additional type check?

        if (! $force && Database::exists()) {
            $this->error("Already began, so we've stopped to avoid wiping your settings.");
            $this->error("If you definitely want to continue, you can force with the --force flag.");
            return;
        }

        $this->line("================");
        $this->line("PREPARING PORTER");
        $this->line("================");
        $this->line("");

        if (! file_exists(config('porter.library_path'))) {
            mkdir(config('porter.library_path'));
        }

        $this->callSilent('vendor:publish', ['--provider' => AppServiceProvider::class]);

        Database::ensureExists($force);

$force of type string is incompatible with the type boolean expected by parameter $forceRefresh of App\Support\Database\Database::ensureExists().

        $this->info("Your Porter settings are stored in ".config('porter.library_path'));
        $this->info("");

        $this->callSilent('home', ['path' => $home]);

        $this->info("Setting home to {$home}.");
        $this->comment("This is the used as the root directory for your sites.");
        $this->comment("If this is incorrect, you can change it using the 'porter home' command.");
        $this->comment("");

        $this->info("Retrieving docker images");
        $this->porter->pullImages();
    }