Skip to content
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.

Sign up for GitHub

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

Jump to bottom

disabling cdata by default #1065

Closed
lsmith77 opened this issue Mar 21, 2019 · 3 comments
Closed

disabling cdata by default #1065

lsmith77 opened this issue Mar 21, 2019 · 3 comments
Labels
Feature Request

Comments

@lsmith77
Copy link
Contributor

Q A
Bug report? no
Feature request? yes
BC Break report? no
RFC? yes

From my understanding there is no global out of the box way to disable the use of cdata. So one has to explicitly specify this on a per descriminator/xmlelement basis. Would it make sense to add some global configuration options for cdata (and maybe other xml settings)?
@goetas
Copy link
Collaborator

goetas commented Mar 21, 2019

related to #752

@goetas
Copy link
Collaborator

goetas commented Mar 21, 2019

It is not the first time I see this request.

CDATA semantically is the same as not having it (except that does not require you to escape manually the xml)

To not forget that createElement($name, $value) is not safe https://3v4l.org/jrDlu

@lsmith77
Copy link
Contributor Author

ok closing in favor of #752 .. will look into a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lsmith77 @goetas