Add a closing tag flag for XML output

[derily]

when JSONObject value is empty string,i need XML.toString output <name></name> not <name/>

[stleary]

@derily Can you provide a sample input and the output that you want to get?

[derily]

@derily Can you provide a sample input and the output that you want to get?

input(JSONObject):
{ "name":"derily", "job":"" }
output:
<name>derily</name> <job/>
expect(just not use self closing tag syntax):
<name>derily</name> <job></job>

[stleary]

@derily It might be possible to accommodate this with an XMLParserConfiguration option. If no one else picks this up, I will try to get something working this weekend.

[madhavgupta2011]

Hi, Can I take this up? I'm thinking of adding one more boolean parameter in toString() method of XML class and handle the string builder append as per the boolean parameter. Please let me know if this solution can be improved. I'm willing to take up on this issue.

[stleary]

@madhavgupta2011 Thanks, yes you can work on this ticket - FYI you don't need to ask first. Please check whether you can use XMLParserConfiguration to trigger the new behavior.

[madhavgupta2011]

@stleary I have opened a pull request with my changes and added two unit test cases as well.
Can you please review it?
Link to PR

I couldn't get my head around XMLParserConfiguration so I used the appropriate changes.

[stleary]

Fix in progress - #754.

[stleary]

Re-opening since the PR was not accepted

[stleary]

Please use XMLParserConfiguration for the implementation.

[stleary]

See comments in your repo.

[rudrajyotib]

@stleary - is this issue still open to be picked?

[rudrajyotib]

@stleary - if no one else is working on it, can you assign it to my name?

[stleary]

@rudrajyotib You can work on this issue. The new behavior should be opt-in, by adding a new property to XMLParserConfiguration.

[rudrajyotib]

@stleary sure, I'll start working on it and share the design.

[rudrajyotib]

@stleary @johnjaylward - I am creating a configuration "explicitClosureOfEmptyTag" in XMLParserConfiguration. By default this value will be false, and an explicit end tag will not be created for tag with empty values.
Do you see any issue with the name?

[stleary]

@rudrajyotib Please use closeEmptyTag or something similar with fewer chars.

[stleary]

Closing due to fixed in #820