New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JS & C# conversion rewrite #660

Merged

rappen merged 3 commits into rappen:master from MarkMpn:js-convert

Feb 18, 2022

Collaborator

MarkMpn commented Feb 18, 2022

Replaced regular expressions with XmlDocument to parse the FetchXML
Use JSON.NET to ensure all values are correctly encoded for JavaScript

Comments and text nodes will be ignored during the JavaScript conversion, not sure if that's an issue?

Fixes #653

MarkMpn added 3 commits

February 17, 2022 22:10


          Use standard XML parser & JSON converter for Javascript conversion

e522101


          Updated C# code generator to use standard XML parser

27f6f23


          Escape values safely for XML

0d67122

rappen merged commit 29e03d9 into rappen:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet