Allow Nokogiri XmlMini backend to process cdata elements

[#3219 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
rails · Sep 17, 2009 · 0d76264 · 0d76264
1 parent 23e72d4
commit 0d76264
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/activesupport/lib/active_support/xml_mini/nokogiri.rb b/activesupport/lib/active_support/xml_mini/nokogiri.rb
@@ -44,7 +44,7 @@ def to_hash(hash = {})
           walker = lambda { |memo, parent, child, callback|
             next if child.blank? && 'file' != parent['type']
 
-            if child.text?
+            if child.text? || child.cdata?
               (memo[CONTENT_ROOT] ||= '') << child.content
               next
             end

diff --git a/activesupport/test/xml_mini/nokogiri_engine_test.rb b/activesupport/test/xml_mini/nokogiri_engine_test.rb
@@ -159,6 +159,17 @@ def test_parse_from_io
     XmlMini.parse(io)
   end
 
+  def test_children_with_cdata
+    assert_equal_rexml(<<-eoxml)
+    <root>
+      <products>
+        hello <![CDATA[everyone]]>
+        morning
+      </products>
+    </root>
+    eoxml
+  end
+
   private
   def assert_equal_rexml(xml)
     hash = XmlMini.with_backend('REXML') { XmlMini.parse(xml) }