Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\DynamicDatabaseConfigMigrateCommand

Complexity

Total Complexity

9

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?php

namespace Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands;

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

class DynamicDatabaseConfigMigrateCommand extends Command
{
    use DatabaseConfig;

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

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'dynamic:migrate
                            {ref : The ref for the database configuration}
                            {--P|--path= : The path where the database migration files are stored}
                            {--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 dynamically';

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

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

        [$database, $configuration, $name] = $this->getDynamicDatabaseConfiguration($ref);

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

        if (!$database) {
            $this->components->error("No database configuration was found");

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

            return;
        }

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

        $newConfig = $this->setNewDynamicConfig($database, $configuration);

$configuration of type null|string is incompatible with the type array expected by parameter $configuration of Ikechukwukalu\Dynamicdatabaseconfig\Console\Commands\DynamicDatabaseConfigMigrateCommand::setNewDynamicConfig().

        $this->addNewConfig($database, $name, $newConfig);
        $this->createDatabase($database, $configuration['database']);
        $this->execMigrationCommands($name, $path);
    }

    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]);
        }
    }
}