Spatie\MediaLibraryPro\Rules\FileExtensionRule

Complexity

Total Complexity

3

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	passes()	0	6	1
A	__construct()	0	5	1
A	message()	0	4	1

<?php

namespace Spatie\MediaLibraryPro\Rules;

use Illuminate\Contracts\Validation\Rule;

class FileExtensionRule implements Rule
{
    protected array $validExtensions = [];

    public function __construct(array $validExtensions = [])
    {
        $this->validExtensions = array_map(
            fn (string $extension) => strtolower($extension),
            $validExtensions,
        );
    }

    /**
     * @param string $attribute
     * @param \Illuminate\Http\UploadedFile $value
     *
     * @return bool
     */
    public function passes($attribute, $value): bool
    {
        return in_array(
            strtolower($value->getClientOriginalExtension()),
            $this->validExtensions,
            strict: true,
        );
    }

    public function message(): string
    {
        return trans('media-library::validation.mime', [

The expression return trans('media-library::validation.mime', array('mimes' => implode(', ', $this->validExtensions))) could return the type array which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

            'mimes' => implode(', ', $this->validExtensions),
        ]);
    }
}