Thinktomorrow\Chief\App\Console\Seed

Complexity

Total Complexity

6

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	18	6

<?php

namespace Thinktomorrow\Chief\App\Console;

class Seed extends BaseCommand
{
    protected $signature = 'chief:seed
                            {seeder=DatabaseSeeder : the classname of the seeder. }
                            {--force}';
    protected $description = 'This will run the seeders to inject new data into your project database.';

    /**
     * @return void
     */
    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.');
    }
}