produced xml with ' instead of " #11

Closed
rednaxus opened this Issue Dec 7, 2011 · 3 comments

Projects

None yet

2 participants

@rednaxus

Hi

I'm having trouble with some systems out there when the json2xml produces xml with ' instead of "

so i changed line 41 in my local json2xml code to

            xml += ' ' + key + '="' + obj[key] + '">';

to get it to work.

I'm not sure what the more accepted standard is, but it seems like some servers don't do it right, and the " seems to be more acceptable. Maybe you could have an option in the args

@c4milo

what kind of errors are you getting in your servers due to this? just curious.

@rednaxus

really meaningless ones, like cannot parse xml, or unknown name, i'm using travelport gds, which is barfing on '

@c4milo

I see, not big deal. I'm in the middle of some changes to address #3 and will include this change as well.

@c4milo c4milo closed this Sep 8, 2012
@c4milo c4milo reopened this Sep 8, 2012
@c4milo c4milo added a commit that closed this issue Sep 8, 2012
@c4milo c4milo Switches from single to double quotes in json2xml script due to crapp…
…y commercial software. Closes #11
4aec894
@c4milo c4milo closed this in 4aec894 Sep 8, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment