[LiveComponent] Fix collections hydration with serializer

[squrious]

Q	A
Bug fix?	yes
New feature?	no
Issues	N/A
License	MIT

Hello!

I encountered issues when trying to hydrate a LiveProp as a collection of objects, using the Symfony Serializer.

Given phpdocumentor/reflection-dockblock is installed in the project, and I have the following component:

#[AsLiveComponent]
class MyComponent
{
    /**
     * @var MyDto[] 
     */
    #[LiveProp(useSerializerForHydration: true)]
    public array $prop = [];
}

Then I should be able to hydrate $prop as a collection of MyDto.

However, the LiveComponentHydrator doesn't consider the collection type when useSerializerForHydration is enabled, and tries to deserialize to array type, throwing the following exception:

Symfony\Component\Serializer\Exception\NotNormalizableValueException : Could not denormalize object of type "array", no supporting normalizer found.

So here is a proposal to fix the issue.

Cheers!

[weaverryan]

Do we need to also check for Type::BUILTIN_TYPE_OBJECT === $propMetadata->collectionValueType()->getBuiltinType() like we do on line 267? Basically, what if this is typed as string[]?

[squrious]

Yes absolutely, I missed those cases. Next commit fixes that, and also ensures the built-in collection type is scalar (if used). Otherwise the "not guessable type" exception is thrown

[weaverryan]

Thanks Nicolas!

[smnandre]

Oh i missed this one ! So cool 👏