Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand

Complexity

Total Complexity

8

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	0

2 Methods

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

<?php

namespace Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands;

use Illuminate\Console\Command;
use Ikechukwukalu\Dynamicdatabaseconfig\Trait\DatabaseConfig;

class EnvDatabaseConfigMigrateCommand extends Command
{
    use DatabaseConfig;

The trait Ikechukwukalu\Dynamicdatabaseconfig\Trait\DatabaseConfig requires some properties which are not provided by Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand: $database, $configuration

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'env:migrate
                            {database : The type of database}
                            {name : The name of the new database connection}
                            {postfix : The postfix for the database configuration}
                            {--P|--path= : The path where the database migration files are kept}
                            {--seed : Running seeders}
                            {--seeder= : Running a single seeder class}
                            {--refresh : Refreshing all migration}
                            {--rollback : Reverting migrations}
                            {--fresh : Re-run all migrations afresh}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Migrate databases from env';

    /**
     * Execute the console command.
     */
    public function handle(): void
    {
        $path = config('dynamicdatabaseconfig.default_path');
        $database = $this->argument('database');
        $name = $this->argument('name');
        $postFix = $this->argument('postfix');

        if ($this->option('path')) {
            $path = $this->option('path');
        }

        $this->components->info("Running migrations for {$name} and path={$path}");

The property components does not exist on Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand. Did you maybe forget to declare it?

        $newConfig = $this->setNewEnvConfig($database, $postFix);

It seems like $postFix can also be of type array and null; however, parameter $postFix of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::setNewEnvConfig() does only seem to accept string, maybe add an additional type check?

It seems like $database can also be of type array and null; however, parameter $database of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::setNewEnvConfig() does only seem to accept string, maybe add an additional type check?

        $this->addNewConfig($database, $name, $newConfig);

It seems like $name can also be of type array and null; however, parameter $name of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::addNewConfig() does only seem to accept string, maybe add an additional type check?

It seems like $database can also be of type array and null; however, parameter $database of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::addNewConfig() does only seem to accept string, maybe add an additional type check?

        $this->createDatabase($database, $newConfig['database']);

It seems like $database can also be of type array and null; however, parameter $database of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::createDatabase() does only seem to accept string, maybe add an additional type check?

        $this->execMigrationCommands($name, $path);

It seems like $name can also be of type array and null; however, parameter $name of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\EnvDatabaseConfigMigrateCommand::execMigrationCommands() does only seem to accept string, maybe add an additional type check?

    }

    private function execMigrationCommands(string $name, null|string $path = '')
    {
        if ($this->option('rollback')) {
            $this->call('migrate:rollback', ['--database' => $name, '--path' => $path]);
            return;
        }

        if ($this->option('refresh')) {
            $this->call('migrate:refresh', ['--database' => $name, '--path' => $path]);
        } elseif ($this->option('fresh')) {
            $this->call('migrate:fresh', ['--database' => $name, '--path' => $path]);
        } else {
            $this->call('migrate', ['--database' => $name, '--path' => $path]);
        }

        if ($this->option('seed')) {
            $this->call('db:seed');
            return;
        }

        if ($seeder = $this->option('seeder')) {
            $this->call('db:seed', ['--class' => $seeder]);
        }
    }
}