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.

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

Implement writeRawValue in ToXmlGenerator #232

Merged
merged 2 commits into from
May 11, 2017

Conversation

Falland
Copy link

@Falland Falland commented May 9, 2017

Currently only writeRaw methods are supported.
Default implementation for writeRawValue from GeneratorBase redirects call to writeRaw.
This lead to element name to be ignored which in order leads to invalid XML.

This change will allow to write 'writeRawValue' code in the same fashion as Json one and generating valid XML

This will allow to write 'writteRawValue' code in the same fashion as Json one.
@cowtowncoder
Copy link
Member

Sounds good. Would it be possible to add unit tests to show that this works?
I'd be happy to merge this.

One other thing: before merge, we need Contributor Licenses Agreement (CLA):

https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

(unless we have asked and gotten one before -- only ever need to be done once)

Usual way is to print, fill & sign, scan, email to info at fasterxml dot com. Once this is done contribution may be merged.

Looking forward to merging this patch, thank you for submitting it!

@Falland
Copy link
Author

Falland commented May 10, 2017

Hi,

Thanks for quick reply.
I've added another commit with the tests.
Ad I've also filled and sent the form.

Looking forward to see this change merged in repo.

Kind regards,

Yury

@cowtowncoder
Copy link
Member

Excellent, thank you! Will merge.

@cowtowncoder cowtowncoder merged commit 7df1d64 into FasterXML:master May 11, 2017
@cowtowncoder cowtowncoder changed the title Adding writeRawValue method support to ToXmlGenerator. Implement writeRawValue in ToXmlGenerator May 11, 2017
@cowtowncoder cowtowncoder added this to the 2.9.0.pr4 milestone May 11, 2017
cowtowncoder added a commit that referenced this pull request May 11, 2017
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.

2 participants