This project is archived and is in readonly mode.
Nokogiri XmlMini backend doesn't properly parse cdata elements
Reported by John Pignata | September 16th, 2009 @ 07:25 PM
In a production project I have I'm using Hash#from_xml to parse a bunch of XML from a foreign CMS. I found that changing the backend to Nokogiri gives me an over 100% speed boost (woo!), but the backend unfortunately doesn't process CDATA sections correctly.
This small patch allows the nokogiri backend to parse CDATA sections and adds a test. It should apply to both 2-3-stable and master.
There's also a pending patch to clean-up this nokogiri backend from Aaron Patterson that is pending -- https://rails.lighthouseapp.com/projects/8994/tickets/2243-nokogiri...
Comments and changes to this ticket
-
Repository September 17th, 2009 @ 07:02 PM
- State changed from “new” to “committed”
(from [179b4512d1e98c7657151b25f159262ea1860f52]) Allow Nokogiri XmlMini backend to process cdata elements
[#3219 state:committed]
Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/179b4512d1e98c7657151b25f15926... -
David Smalley October 22nd, 2009 @ 01:02 PM
This ticket is basically the same as the one for LibXML XMLmini backend - any chance of someone taking a look at that?
https://rails.lighthouseapp.com/projects/8994/tickets/2835-libxml-x...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Jeremy Kemper
John Pignata
Attachments
Referenced by
-
3219
Nokogiri XmlMini backend doesn't properly parse cdata elements
[#3219 state:committed]