ThinKit\Console\Commands\DeleteExpiredCommand::handle()

Complexity

Conditions	9
Paths	8

Size

Total Lines	36
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	19
CRAP Score	9

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle()
    {
        $modelName        = $this->argument('model');
        $timeoutInMinutes = (int)$this->option('timeout');
        $isForceDelete    = (bool)$this->option('force');

        if (!is_subclass_of($modelName, Model::class, true)) {
            $modelName = Relation::getMorphedModel($modelName) ?: $modelName;

It seems like $modelName can also be of type array; however, parameter $alias of Illuminate\Database\Eloquent\Relations\Relation::getMorphedModel() does only seem to accept string, maybe add an additional type check?

        }

        if (!is_subclass_of($modelName, Model::class, true)) {
            $this->error("Model [{$modelName}] name should extends model class");

            return static::FAILURE;
        }

        if (!in_array(HasExpiration::class, class_uses_recursive($modelName))) {
            $this->error("Model [{$modelName}] should use trait 'HasExpiration'");

            return static::FAILURE;
        }

        $query = $modelName::query()->willBeExpiredAt(Carbon::now()->addMinutes($timeoutInMinutes));

        if ($isForceDelete && in_array(SoftDeletes::class, class_uses_recursive($modelName))) {
            do {
                $deleted = $query->limit(100)->forceDelete();
            } while ($deleted > 0);
        } else {
            do {
                $deleted = $query->limit(100)->delete();
            } while ($deleted > 0);
        }


        return static::SUCCESS;
    }