Spatie\MediaLibrary\Commands\RegenerateCommand

Complexity

Total Complexity

7

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	9	1
A	__construct()	0	7	1
B	getMediaToBeRegenerated()	0	20	5

<?php

namespace Spatie\MediaLibrary\Commands;

use Illuminate\Console\Command;
use Spatie\{
    MediaLibrary\FileManipulator, 
    MediaLibrary\Media, 
    MediaLibrary\MediaRepository
};

class RegenerateCommand extends Command
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $signature = 'medialibrary:regenerate {modelType?} {--ids=*}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Regenerate the derived images of media';

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

    /**
     * @var \Spatie\MediaLibrary\FileManipulator
     */
    protected $fileManipulator;

    /**
     * RegenerateCommand constructor.
     *
     * @param MediaRepository $mediaRepository
     * @param FileManipulator $fileManipulator
     */
    public function __construct(MediaRepository $mediaRepository, FileManipulator $fileManipulator)
    {
        parent::__construct();

        $this->mediaRepository = $mediaRepository;
        $this->fileManipulator = $fileManipulator;
    }

    /**
     * Handle regeneration.
     */
    public function handle()
    {
        $this->getMediaToBeRegenerated()->map(function (Media $media) {
            $this->fileManipulator->createDerivedFiles($media);
            $this->info(sprintf('Media %s regenerated', $media->id));
        });

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

    /**
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function getMediaToBeRegenerated()
    {
        $modelType = $this->argument('modelType') ?? '';
        $mediaIds = $this->option('ids');

        if ($modelType === '' && !$mediaIds) {
            return $this->mediaRepository->all();
        }

        if ($mediaIds) {

            if (! is_array($mediaIds)) {
                $mediaIds = explode(',', $mediaIds);
            }

            return $this->mediaRepository->getByIds($mediaIds);
        }

        return $this->mediaRepository->getByModelType($modelType);

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

    }
}