I don't understand the

xml.split('&').join('&')

- first of all why don't you just put the &amp into your template, and secondly would replace() be much more performant?