X Tutup
Skip to content

fix(serializer): Use serializer when denormalizing relation inside Input#7830

Open
VincentLanglet wants to merge 2 commits intoapi-platform:4.2from
VincentLanglet:poc-denormalize-relations
Open

fix(serializer): Use serializer when denormalizing relation inside Input#7830
VincentLanglet wants to merge 2 commits intoapi-platform:4.2from
VincentLanglet:poc-denormalize-relations

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Mar 9, 2026

Q A
Branch? 4.2
Tickets Closes #7831
License MIT
Doc PR api-platform/docs#...

@VincentLanglet VincentLanglet force-pushed the poc-denormalize-relations branch 2 times, most recently from ee0dfc1 to 56a6135 Compare March 9, 2026 15:58
@VincentLanglet VincentLanglet changed the title Try using serializer when denormalizing relation fix(serializer): Use serializer when denormalizing relation inside Input Mar 9, 2026
@VincentLanglet VincentLanglet force-pushed the poc-denormalize-relations branch from 56a6135 to 66f5e6f Compare March 9, 2026 16:13
$propertyAccessorProphecy = $this->prophesize(PropertyAccessorInterface::class);

$serializerProphecy = $this->prophesize(SerializerInterface::class);
$serializerProphecy->willImplement(DenormalizerInterface::class);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the serializer is a denormalizer, it will call it instead of the iriConverter and the mock are not properly configured for this in this test.

@VincentLanglet VincentLanglet force-pushed the poc-denormalize-relations branch from 0e4b22c to 3b9728d Compare March 9, 2026 16:56
@VincentLanglet VincentLanglet marked this pull request as ready for review March 9, 2026 17:05
@VincentLanglet VincentLanglet requested a review from soyuka March 9, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup