search

IATI / D-Portal

http://d-portal.org/
Other
31 stars 23 forks source link

CDATA (again) #582

Closed rolfkleef closed 4 years ago

rolfkleef commented 4 years ago

It seems that texts encoded in CDATA in IATI files don't get included in d-portal (for example in this Hivos project)

I can find an earlier reference to cdata in #124, and when looking at the code in jml.js and the sax-js parser I would expect a parser.oncdata event handler in the list (I'm not sure how sax-js does this, maybe the ontext event is also fired for cdata?).

xriss commented 4 years ago

Should be fixed now. so will pick up data over night.

I had to change the sax parser as the old one was causing build problems and didn't notice that the new one dealt with CDATA slightly differently to the old one.

rolfkleef commented 4 years ago

thanks @xriss - looks alright now!