Skip to content

[Serializer] Update serializer.rst #16845

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

issamkhadiri1989
Copy link
Contributor

Hello

The $encoderIgnoredNodeTypes does not exist in the XmlEncoder's constructor and the $defaultContext argument should be used instead, So To escape the comments while encoding, we can either pass the \XML_COMMENT_NODE to the XmlEncoder::ENCODER_IGNORED_NODE_TYPES context option in the constructor or use it when calling the encode() method.

Hello 

The `$encoderIgnoredNodeTypes` does not exist in the XmlEncoder's constructor and the $defaultContext argument should be used instead, So To escape the comments while encoding, we can either pass the `\XML_COMMENT_NODE` to the  `XmlEncoder::ENCODER_IGNORED_NODE_TYPES` context option in the constructor  or use it when calling the encode() method.
@carsonbot carsonbot added this to the 6.0 milestone Jun 4, 2022
Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

I left some minor comments about code style, LGTM. Thanks!

@carsonbot carsonbot changed the title Update serializer.rst [Serializer] Update serializer.rst Oct 21, 2022
@javiereguiluz javiereguiluz merged commit 84fd8c6 into symfony:6.0 Oct 21, 2022
@javiereguiluz
Copy link
Member

Thanks @issamkhadiri1989 for this contribution ... and thanks Jules for the review. I made all the requested changes while merging.

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