Enjoys\Forms\Rule\UploadCheck\ExtensionsCheck

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	check()	0	17	4
A	__construct()	0	10	1

<?php

declare(strict_types=1);

namespace Enjoys\Forms\Rule\UploadCheck;

use Enjoys\Forms\Interfaces\Ruleable;
use Psr\Http\Message\UploadedFileInterface;

final class ExtensionsCheck implements UploadCheckInterface
{
    private UploadedFileInterface|false $value;
    private Ruleable $element;
    private array $expectedExtensions;
    private ?string $message;

    public function __construct(
        false|UploadedFileInterface $value,
        Ruleable $element,
        string $expectedExtensions,
        ?string $message = null
    ) {
        $this->value = $value;
        $this->element = $element;
        $this->expectedExtensions = \array_map('trim', \explode(",", \strtolower($expectedExtensions)));
        $this->message = $message;
    }

    public function check(): bool
    {
        if ($this->value === false) {
            return true;
        }
        $extension = pathinfo($this->value->getClientFilename() ?? '', PATHINFO_EXTENSION);

        if (is_null($this->message)) {
            $this->message = 'Загрузка файлов с расширением .' . $extension . ' запрещена';

Are you sure $extension of type array|string can be used in concatenation?

        }


        if (!in_array(\strtolower($extension), $this->expectedExtensions, true)) {

It seems like $extension can also be of type array; however, parameter $string of strtolower() does only seem to accept string, maybe add an additional type check?

            $this->element->setRuleError($this->message);
            return false;
        }
        return true;
    }
}