LaraGeoData\Console\Commands\MakeMigrationCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getSuffix()	0	8	2
A	handle()	0	7	1

<?php

namespace LaraGeoData\Console\Commands;

use Illuminate\Console\Command;
use Illuminate\Filesystem\Filesystem;
use Illuminate\Support\Str;

class MakeMigrationCommand extends Command
{
    use HasTablesClassesMap;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'geonames:make:migration
        {type : Migration type. }
        {--suffix= : Suffix used for specify country if need. }
    ';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Publish specific migrations.';

    /**
     * The filesystem instance.
     *
     * @var Filesystem
     */
    protected Filesystem $files;

    /**
     * Create a new command instance.
     *
     * @param Filesystem $files
     *
     * @return void
     */
    public function __construct(Filesystem $files)
    {
        parent::__construct();

        $this->files = $files;
    }

    public function handle()
    {
        $tableClassName = $this->getTableClassNameByType($this->argument('type'));

It seems like $this->argument('type') can also be of type array and null; however, parameter $type of LaraGeoData\Console\Commands\MakeMigrationCommand::getTableClassNameByType() does only seem to accept string, maybe add an additional type check?

        (new $tableClassName($this->files))->makeMigration($this->getSuffix());

        return 0;
    }

    /**
     * Get formatted suffix.
     *
     * @return string|null
     */
    protected function getSuffix(): ?string
    {
        $suffix = $this->option('suffix');
        if ($suffix) {
            $suffix = Str::snake($suffix);
        }

        return $suffix;
    }
}