Replacement of non strings

[kartharer]

Thanks for this helpful code!

But in line 17 I had to say 'return String(txt).replace …' because of an runtime error with numbers.

And different from your samples I had to use {filter:…} always for the &.

[davidcalhoun]

(sorry for the delay - I've been on a sort of sabbatical)

Weird, I don't see this on my end, but it technically should be failing with numbers. I added your suggestion to coerse it into a String every time.

Looking into the other issue now.

Thanks for the feedback!

[davidcalhoun]

Not sure I see what you mean with the &. What environment are you running on?

I see this on my end:

jstoxml.toXML({
  foo: '&'
}
// Output: <foo>&</foo>

jstoxml.toXML({
  foo: '&'
},
{
  filter: {
    '&': '&'
  }
}
// Output: <foo>&amp;</foo>

I added a few more tests based on your issue. Thanks!