search

GitbookIO / node-onix

Node.JS library to parse/generate ONIX XML files
Apache License 2.0
18 stars 13 forks source link

Naive xml parsing #2

Open reduxdj opened 7 years ago

reduxdj commented 7 years ago

Your naive xml parser is unfortunate, why not use something better. you didn't plan for CDATA

christianroy commented 7 years ago

@reduxdj if that may be of any help, I have made a fork of node-onix that depends on my fork for xml-schema, which itself depends on my CDATA-supporting fork of xml-parser. :-\ It's here: https://github.com/christianroy/node-onix (my initial intent was to add ONIX 3 support, which I did, for a limited subset of tags).

reduxdj commented 7 years ago

Cool, thanks i'll have a look.