slince / upload

src/Constraint/MimeTypeConstraint.php

<?php

namespace Slince\Upload\Constraint;

use Symfony\Component\HttpFoundation\File\UploadedFile;

class MimeTypeConstraint implements ConstraintInterface
{
    /**
     * @var string
     */
    protected $errorMessageTemplate = 'File type {type} is invalid';

    /**
     * @var array
     */
    protected $allowedMimeTypes;

    public function __construct(array $allowedMimeTypes)
    {
        $this->allowedMimeTypes = $allowedMimeTypes;
    }

    /**
     * {@inheritdoc]
     */
    public function validate(UploadedFile $file): bool
    {
        foreach ($this->allowedMimeTypes as $mimeType) {
            if ($mimeType === $file->getClientMimeType()
                || (strpos($mimeType, '/*') !== false
                    && explode('/', $mimeType)[0] === explode('/', $file->getMimeType())[0])

It seems like $file->getMimeType() can also be of type null; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

            ) {
                return true;
            }
        }
        return false;
    }

    /**
     * {@inheritdoc}
     */
    public function getErrorMessage(UploadedFile $file): string
    {
        return str_replace('{type}', $file->getClientMimeType(), $this->errorMessageTemplate);
    }

    /**
     * {@inheritdoc}
     */
    public function setErrorMessage(string $messageTemplate): void
    {
        $this->errorMessageTemplate = $messageTemplate;
    }
}