Thinktomorrow\Chief\App\Console\Seed

Complexity

Total Complexity

6

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

1 Method

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

<?php

namespace Thinktomorrow\Chief\App\Console;

use Thinktomorrow\Chief\Pages\Page;
use Thinktomorrow\Chief\Authorization\AuthorizationDefaults;
use Illuminate\Database\Eloquent\Factory as ModelFactory;
use Illuminate\Support\Facades\Artisan;

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.';

    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?

    }
}