Sausin\DBSetAutoIncrement\Commands\SetAutoIncrementCommand

Complexity

Total Complexity

8

Size/Duplication

Total Lines	135
Duplicated Lines	14.07 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	handle()	0	35	5
A	updateMysqlTables()	8	8	1
A	updateSqliteTables()	11	11	1

<?php

namespace Sausin\DBSetAutoIncrement\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Config;
use Sausin\DBSetAutoIncrement\DatabaseInfo;
use Sausin\DBSetAutoIncrement\GetAttribute;
use Sausin\DBSetAutoIncrement\UpdateAttribute;

class SetAutoIncrementCommand extends Command
{
    use DatabaseInfo;
    use GetAttribute;
    use UpdateAttribute;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'db:set-auto-increment
                            {--tables=* : The table(s) for which auto increment should be set}
                            {--value= : The auto increment value to be set}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Set auto increment for database table(s)';

    /** @var array */
    protected $skipTables;

    /** @var array */
    protected $onlyTables;

    /** @var string */
    protected $mode;

    /** @var string */
    protected $driver;

    /** @var int */
    protected $autoIncrement;

    /** @var array */
    protected $supportedDrivers = ['mysql', 'sqlite'];

    /**
     * The name of the queue the job should be sent to.
     *
     * @var string|null
     */
    public $queue = 'monitoring';

    /**
     * Create the event listener.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()
    {
        parent::__construct();

        $this->mode = Config::get('auto-increment.mode', 'skip');
        $this->skipTables = Config::get('auto-increment.skipTables', ['migrations']);

It seems like \Illuminate\Support\Facades\Config::get('auto-increment.skipTables', array('migrations')) of type * is incompatible with the declared type array of property $skipTables.

        $this->onlyTables = Config::get('auto-increment.onlyTables', []);

It seems like \Illuminate\Support\Facades\Config::get('auto-increment.onlyTables', array()) of type * is incompatible with the declared type array of property $onlyTables.

    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $this->autoIncrement = $this->option('value') ?? Config::get('auto-increment.autoIncrement', 100001);

        if (! $this->isDatabaseCompatible()) {
            $this->info("Database {$this->driver} not supported");

            return;
        }

        $driver = ucfirst($this->driver);

        if ($this->option('tables')) {
            $this->{"update{$driver}Tables"}(collect($this->option('tables')));

            $this->info('Specified tables have been updated');

            return;
        }

        $tables = collect([]);

        if ($this->mode === 'only') {
            $tables = collect($this->onlyTables);
        }

        if ($this->mode === 'skip') {
            $tables = collect($this->getTableList())->reject(function ($value) {
                return in_array($value, $this->skipTables, true);
            });
        }

        $this->{"update{$driver}Tables"}($tables);
        $this->info('Tables have been updated as per config');
    }

    /**
     * Update AUTO INCREMENT value in mysql tables.
     *
     * @param  Collection $tables
     * @return void
     */
    protected function updateMysqlTables(Collection $tables): void

This method seems to be duplicated in your project.

    {
        $tables->filter(function ($table) {
            return $this->getAutoIncrement('Mysql', $table) < $this->autoIncrement;
        })->map(function ($table) {
            $this->updateAutoIncrement('Mysql', $table);
        });
    }

    /**
     * Update AUTO INCREMENT value in sqlite tables.
     *
     * @param  Collection $tables
     * @return void
     */
    protected function updateSqliteTables(Collection $tables): void

This method seems to be duplicated in your project.

    {
        // the auto increment value is reduced by 1 as SQLITE uses it in this way
        $this->autoIncrement--;

        $tables->filter(function ($table) {
            return $this->getAutoIncrement('Sqlite', $table) < $this->autoIncrement;
        })->map(function ($table) {
            $this->updateAutoIncrement('Sqlite', $table);
        });
    }
}