Skip to content

feat: add libxml options support to xml() and save()#68

Open
guillaumedelre wants to merge 1 commit into
downforcetech:mainfrom
guillaumedelre:feature/issue-15
Open

feat: add libxml options support to xml() and save()#68
guillaumedelre wants to merge 1 commit into
downforcetech:mainfrom
guillaumedelre:feature/issue-15

Conversation

@guillaumedelre
Copy link
Copy Markdown

xml() and save() now accept an $options integer bitmask forwarded to DOMDocument::saveXML(), enabling LIBXML_NOEMPTYTAG and other libxml flags. The same parameter propagates through FluidContext::xml(), FluidSaveTrait, and the FluidHelper serialization helpers.

Closes #15

xml() and save() now accept an $options integer bitmask forwarded to
DOMDocument::saveXML(), enabling LIBXML_NOEMPTYTAG and other libxml
flags. The same parameter propagates through FluidContext::xml(),
FluidSaveTrait, and the FluidHelper serialization helpers.

Closes downforcetech#15

Signed-off-by: Guillaume Delré <delre.guillaume@gmail.com>
@guillaumedelre
Copy link
Copy Markdown
Author

Hey @daniele-orlando, this PR fixes #15 — happy to adjust if anything looks off.

@guillaumedelre
Copy link
Copy Markdown
Author

Hey @daniele-orlando, gentle ping here. Let me know if you'd like any changes before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for LIBXML_NOEMPTYTAG for save()

1 participant