doctrine / DoctrineModule

src/DoctrineModule/Validator/Service/ObjectExistsFactory.php

<?php

declare(strict_types=1);

namespace DoctrineModule\Validator\Service;

use DoctrineModule\Validator\ObjectExists;
use Interop\Container\ContainerInterface;

/**
 * Factory for creating ObjectExists instances
 *
 * @link    http://www.doctrine-project.org/
 */
class ObjectExistsFactory extends AbstractValidatorFactory
{
    /** @var string */
    protected $validatorClass = ObjectExists::class;

    /**
     * {@inheritDoc}
     */
    public function __invoke(ContainerInterface $container, $requestedName, ?array $options = null)
    {
        $container = $this->container($container);

        $repository = $this->getRepository($container, $options);

It seems like $options defined by parameter $options on line 23 can also be of type array; however, DoctrineModule\Validator\Service\AbstractValidatorFactory::getRepository() does only seem to accept array<integer,*>|null, maybe add an additional type check?

        return new ObjectExists($this->merge($options, [
            'object_repository' => $repository,
            'fields'            => $this->getFields($options),

$options is of type null|array, but the function expects a array<integer,*>.

        ]));
    }
}