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

Complexity

Conditions	6
Paths	4

Size

Total Lines	18
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 array; however, parameter $abstract of app() does only seem to accept null|string, maybe add an additional type check?

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