Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Serializer] Fix denormalizing XML array with empty body (5.x)#43204
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
carsonbot commentedSep 28, 2021
Hey! I think@mkrauser has recently worked with this code. Maybe they can help review this? Cheers! Carsonbot |
dunglas left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM
src/Symfony/Component/Serializer/Normalizer/AbstractObjectNormalizer.php OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
6470a7a to44c7114Comparesrc/Symfony/Component/Serializer/Normalizer/AbstractObjectNormalizer.php OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
This happens for example with XML empty tags denormalizing to an object array attribute.
44c7114 tod94b7c1Compare…4.4) (alexandre-daubois)This PR was merged into the 4.4 branch.Discussion----------[Serializer] Fix denormalizing XML array with empty body (4.4)This happens for example with XML empty tags denormalizing to an object array attribute.| Q | A| ------------- | ---| Branch? | 4.4| Bug fix? | yes| New feature? | no| Deprecations? | no| Tickets |Fix#43193 on Symfony 4.4| License | MIT| Doc PR | _NA_Not sure how to deal with versions differences as they differ just a little bit. Anyway, here's the fix for 4.4 and you can find the fix for 5.x on#43204 😄Commits-------d6b7e2d [Serializer] Fix denormalizing of array with empty body
fabpot commentedSep 29, 2021
Thank you@alexandre-daubois. |
Uh oh!
There was an error while loading.Please reload this page.
This happens for example with XML empty tags denormalizing to an object array attribute.
Fix for 4.4 can be found in#43205