xmldom Encode ">" to &gt; during serialization

Symbol ">" is not properly encoded during serialization.

Test code:

var xmldom = require('xmldom');
var domparser = new xmldom.DOMParser;
var xmlserializer = new xmldom.XMLSerializer;

var dom = domparser.parseFromString('<node attr="&lt;&gt;">&lt;&gt;</node>');
var s = xmlserializer.serializeToString(dom);
console.log(s);

// outputs '<node attr="&lt;>">&lt;></node>'

Dec 31 '12 10:12 private-face

thanks it is not required

is well-formed xml

Dec 31 '12 10:12 jindw

Created foo.xml using this source.

<node attr="1 &lt; 2 && 2 > 1 ">
&lt;> 
</node>

Ran it through xmllint.

$ xmllint foo.xml 
foo.xml:1: parser error : xmlParseEntityRef: no name
<node attr="1 &lt; 2 && 2 > 1 ">
                      ^
foo.xml:1: parser error : xmlParseEntityRef: no name
<node attr="1 &lt; 2 && 2 > 1 ">

Dec 31 '12 10:12 bigeasy

to: bigeasy

hi, you are so fast always . ^_^

it's my fault, && is valid but > is well-formed

i think we need a config about whether the '>' is encoded; In order to meet everyone's taste:)

Dec 31 '12 10:12 jindw

I don't have an opinion. I might have had an opinion at the time of #25, but not today. With XML, valid is valid.

Is your primary objection that you don't want to rewrite your tests to expect an encoded >, or is it that you prefer to leave them unencoded as a matter of style?

Dec 31 '12 10:12 bigeasy

In the beginning, because of my lazy, as your said, keep it as a style is also good.

But I worry about this style may give the user some trouble.

For example: can't pass the tests.

Dec 31 '12 10:12 jindw

Yes, I was wrong, well-formed XML doesn't require escaping ">" inside character data.

i think we need a config about whether the '>' is encoded; In order to meet everyone's taste:)

However, at least Gecko and Webkit implementations of XMLSerializer do escape ">", so adding such option would be great not only for meeting everyone's taste but for better compatibility with browser javascript :)

Dec 31 '12 10:12 private-face

Note that escaping of ">" is required in attribute values but not in character data.