Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Serializer] Fix discriminator class mapping with allow_extra_attributes=false#61889

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@chris54721
Copy link
Contributor

QA
Branch?6.4
Bug fix?yes
New feature?no
Deprecations?no
IssuesFix#61887
LicenseMIT

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

@carsonbotcarsonbot added this to the6.4 milestoneSep 29, 2025
@chris54721chris54721force-pushed thefix/serializer-discriminator-extra-attribute branch from5f9b2b0 to04882d8CompareSeptember 29, 2025 15:42
@xabbuh
Copy link
Member

Don't we also have to patch theGetSetMethodNormalizer and thePropertyNormalizer?

@chris54721chris54721force-pushed thefix/serializer-discriminator-extra-attribute branch from04882d8 to6b56ec9CompareSeptember 30, 2025 15:55
@chris54721
Copy link
ContributorAuthor

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

…llow_extra_attributes=falseMake sure that the discriminator mapping type property is considered an allowed attributewhen the `AbstractNormalizer::ALLOW_EXTRA_ATTRIBUTES` context option is set to `false`.
@chris54721chris54721force-pushed thefix/serializer-discriminator-extra-attribute branch from6b56ec9 to4afa086CompareOctober 1, 2025 15:08
@nicolas-grekas
Copy link
Member

Thank you@chris54721.

@nicolas-grekasnicolas-grekas merged commitb8509c7 intosymfony:6.4Oct 2, 2025
10 of 11 checks passed
This was referencedOct 28, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nicolas-grekasnicolas-grekasnicolas-grekas left review comments

@xabbuhxabbuhxabbuh approved these changes

@dunglasdunglasAwaiting requested review from dunglasdunglas is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

6.4

Development

Successfully merging this pull request may close these issues.

4 participants

@chris54721@xabbuh@nicolas-grekas@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp