Spatie\DbSnapshots\Commands\Delete::handle()

Complexity

Conditions	4
Paths	3

Size

Total Lines	23
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function handle()
    {
        $snapShots = app(SnapshotRepository::class)->getAll();

The assignment to $snapShots is dead and can be removed.

        if (app(SnapshotRepository::class)->getAll()->isEmpty()) {
            $this->warn('No snapshots found. Run `snapshot:create` to create snapshots.');

            return;
        }

        $name = $this->argument('name') ?: $this->askForSnapshotName();

        $snapshot = app(SnapshotRepository::class)->findByName($name);

It seems like $name can also be of type string[]; however, parameter $name of Spatie\DbSnapshots\SnapshotRepository::findByName() does only seem to accept string, maybe add an additional type check?

        if (! $snapshot) {
            $this->warn("Snapshot `{$name}` does not exist!");

            return;
        }

        $snapshot->delete();

        $this->info("Snapshot `{$snapshot->name}` deleted!");
    }