Sebastienheyd\BoilerplatePackager\Commands\Crud\Routes

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	22	2

<?php

namespace Sebastienheyd\BoilerplatePackager\Commands\Crud;

class Routes extends Command
{
    protected $signature = 'boilerplate:packager:crud:routes {package} {tables}';
    protected $description = '';

    public function handle()
    {
        $package = $this->argument('package');
        [$vendor, $packageName] = explode('/', $package);

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

        $relations = [];
        foreach ($this->argument('tables') as $table) {
            $relations[$table] = $this->getTableRelations($table);
        }

//        dd($relations);

        $routes = (string) view('packager::routes', [
            'models' => $this->argument('tables'),
            'namespace' => $this->getNamespace($this->argument('package')),
            'vendor' => $vendor,
            'package' => $packageName,
            'relations' => $relations,
        ]);

        $this->info('Writing routes');
        $this->storage->put($package.'/src/routes/'.$packageName.'.php', $routes);

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

    }
}