fractalzombie / frzb-request-mapper

PropertyMapper/PropertyMapperLocator.php

<?php

declare(strict_types=1);

namespace FRZB\Component\RequestMapper\PropertyMapper;

use Fp\Collections\ArrayList;
use FRZB\Component\DependencyInjection\Attribute\AsService;
use FRZB\Component\RequestMapper\Exception\PropertyMapperLocatorException;
use FRZB\Component\RequestMapper\PropertyMapper\Mapper\PropertyMapperInterface as PropertyMapper;
use Symfony\Component\DependencyInjection\Attribute\TaggedIterator;

#[AsService]
class PropertyMapperLocator implements PropertyMapperLocatorInterface
{
    /** @var ArrayList<PropertyMapper> */
    private readonly ArrayList $mappers;

    public function __construct(
        #[TaggedIterator(PropertyMapper::class)] iterable $mappers,
    ) {
        $this->mappers = ArrayList::collect($mappers);

The property mappers is declared read-only in FRZB\Component\RequestMapper\PropertyMapper\PropertyMapperLocator.

    }

    public function get(\ReflectionProperty $property): PropertyMapper
    {
        return $this->mappers
            ->first(fn (PropertyMapper $pm) => $pm->canMap($property))
            ->getOrThrow(PropertyMapperLocatorException::notFound($property))

It seems like FRZB\Component\RequestMapper\Exception\PropertyMapperLocatorException::notFound($property) can also be of type FRZB\Component\RequestMapper\Exception\PropertyMapperLocatorException; however, parameter $fallback of Fp\Functional\Option\Option::getOrThrow() does only seem to accept callable, maybe add an additional type check?

        ;
    }
}