Thinktomorrow\Chief\App\Console\Seed::handle()

Complexity

Conditions	6
Paths	4

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle()
    {
        if (app()->environment() != 'local' && !$this->option('force')) {

The method environment() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

            throw new \Exception('You can only run the seeder in the local environment since this will inject a ton of default data');
        }

        if (app()->environment() != 'local' && $this->option('force')) {
            if (!$this->confirm('You are about to inject default seeding data in the '.app()->environment().' database! Are you sure?')) {
                $this->info('You are welcome. I have just saved your job.');
                return;
            }
        }

        $seederClass = $this->argument('seeder');
        app($seederClass)->run();

It seems like $seederClass can also be of type string[]; however, parameter $abstract of app() does only seem to accept null|string, maybe add an additional type check?

The method run() does not exist on Illuminate\Contracts\Foundation\Application.

        $this->info($seederClass.' has run successfully.');

Are you sure $seederClass of type null|string|string[] can be used in concatenation?

    }