amtrack / force-dev-tool

[DEPRECATED] Command line tool supporting the Force.com development lifecycle
MIT License
108 stars 37 forks source link

fix: character '&' in xml documents #242

Closed froucher closed 4 years ago

froucher commented 4 years ago

Hello,

It's about the conversion from & to &, related to https://github.com/isaacs/sax-js/issues/167 issue.

The fix with text = text.replace(new RegExp('&',"gm"), '&'); doesn't not resolve any issue, even creates new bugs, if the metadata contains an unescaped & character then means that metadata is not the properly written.

Added some tests with & that ensure it is working right.

Thanks

froucher commented 4 years ago

Issue #243

amtrack commented 4 years ago

:tada: This PR is included in version 2.7.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: