mostafaznv / larupload

src/Actions/Cover/SetCoverAction.php

<?php

namespace Mostafaznv\Larupload\Actions\Cover;

use Illuminate\Http\UploadedFile;
use Illuminate\Support\Facades\Storage;
use Mostafaznv\Larupload\Actions\GuessLaruploadFileTypeAction;
use Mostafaznv\Larupload\DTOs\CoverActionData;

class SetCoverAction
{
    public function __construct(
        private readonly ?UploadedFile   $file,
        private readonly mixed           $cover,
        private readonly CoverActionData $data
    ) {}

    public static function make(?UploadedFile $file, mixed $cover, CoverActionData $data): static
    {
        return new static($file, $cover, $data);
    }


    public function run(string $path): array
    {
        Storage::disk($this->data->disk)->deleteDirectory($path);

        $output = $this->data->output;

        if ($this->shouldDeleteCover()) {
            $output = DeleteCoverAction::make($this->data->type, $output)->run();
        }
        else if ($this->shouldUploadCover()) {
            $output = UploadCoverAction::make($this->cover, $this->data)->run($path);
        }
        else if ($this->data->generateCover) {
            $output = GenerateCoverFromFileAction::make($this->file, $this->data)->run($path);

It seems like $this->file can also be of type null; however, parameter $file of Mostafaznv\Larupload\Actions\Cover\GenerateCoverFromFileAction::make() does only seem to accept Illuminate\Http\UploadedFile, maybe add an additional type check?

        }

        return $output;
    }

    private function shouldDeleteCover(): bool
    {
        return isset($this->cover) and $this->cover == LARUPLOAD_NULL;
    }

    private function shouldUploadCover(): bool
    {
        return file_has_value($this->cover) and GuessLaruploadFileTypeAction::make($this->cover)->isImage();
    }
}