Spatie\MediaLibrary\Commands\ClearCommand

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handle()	0	10	2
A	getMediaItems()	0	22	5

<?php

namespace Spatie\MediaLibrary\Commands;

use Illuminate\Console\Command;
use Illuminate\Console\ConfirmableTrait;
use Illuminate\Database\Eloquent\Collection;
use Spatie\MediaLibrary\MediaRepository;

class ClearCommand extends Command
{
    use ConfirmableTrait;

    protected $signature = 'medialibrary:clear {modelType?} {collectionName?}
    {-- force : Force the operation to run when in production}';

    protected $description = 'Delete all items in a media collection.';

    /** @var \Spatie\MediaLibrary\MediaRepository */
    protected $mediaRepository;

    public function __construct(MediaRepository $mediaRepository)
    {
        parent::__construct();
        $this->mediaRepository = $mediaRepository;
    }

    public function handle()
    {
        if (! $this->confirmToProceed()) {
            return;
        }

        $this->getMediaItems()->each->delete();

        $this->info('All done!');
    }

    public function getMediaItems(): Collection
    {
        $modelType = $this->argument('modelType');
        $collectionName = $this->argument('collectionName');

        if (! is_null($modelType) && ! is_null($collectionName)) {
            return $this->mediaRepository->getByModelTypeAndCollectionName(
                $modelType,

It seems like $modelType defined by $this->argument('modelType') on line 41 can also be of type array; however, Spatie\MediaLibrary\MediaRepository::getByModelTypeAndCollectionName() does only seem to accept string, maybe add an additional type check?

                $collectionName

It seems like $collectionName defined by $this->argument('collectionName') on line 42 can also be of type array; however, Spatie\MediaLibrary\MediaRepository::getByModelTypeAndCollectionName() does only seem to accept string, maybe add an additional type check?

            );
        }

        if (! is_null($modelType)) {
            return $this->mediaRepository->getByModelType($modelType);

It seems like $modelType defined by $this->argument('modelType') on line 41 can also be of type array; however, Spatie\MediaLibrary\MediaRepository::getByModelType() does only seem to accept string, maybe add an additional type check?

        }

        if (! is_null($collectionName)) {
            return $this->mediaRepository->getByCollectionName($collectionName);

It seems like $collectionName defined by $this->argument('collectionName') on line 42 can also be of type array; however, Spatie\MediaLibrary\MediaRepository::getByCollectionName() does only seem to accept string, maybe add an additional type check?

        }

        return $this->mediaRepository->all();
    }
}