Skip to content

Conversation

@chris54721
Copy link
Contributor

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #61887
License MIT

Make sure that the discriminator mapping type property is considered an allowed attribute when the AbstractNormalizer::ALLOW_EXTRA_ATTRIBUTES context option is set to false.

@chris54721 chris54721 requested a review from dunglas as a code owner September 29, 2025 15:40
@carsonbot carsonbot added this to the 6.4 milestone Sep 29, 2025
@chris54721 chris54721 force-pushed the fix/serializer-discriminator-extra-attribute branch from 5f9b2b0 to 04882d8 Compare September 29, 2025 15:42
@xabbuh
Copy link
Member

xabbuh commented Sep 30, 2025

Don't we also have to patch the GetSetMethodNormalizer and the PropertyNormalizer?

@chris54721 chris54721 force-pushed the fix/serializer-discriminator-extra-attribute branch from 04882d8 to 6b56ec9 Compare September 30, 2025 15:55
@chris54721
Copy link
Contributor Author

@xabbuh you're right, I have patched those as well, thanks.

…llow_extra_attributes=false

Make sure that the discriminator mapping type property is considered an allowed attribute
when the `AbstractNormalizer::ALLOW_EXTRA_ATTRIBUTES` context option is set to `false`.
@chris54721 chris54721 force-pushed the fix/serializer-discriminator-extra-attribute branch from 6b56ec9 to 4afa086 Compare October 1, 2025 15:08
@nicolas-grekas
Copy link
Member

Thank you @chris54721.

@nicolas-grekas nicolas-grekas merged commit b8509c7 into symfony:6.4 Oct 2, 2025
10 of 11 checks passed
This was referenced Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants