Changeset 6540

Timestamp:

04/19/07 22:32:39 (3 years ago)

Author:

david

Message:

Added yielding of builder in Hash#to_xml [DHH]

Files:

• trunk/activesupport/CHANGELOG (modified) (1 diff)
• trunk/activesupport/lib/active_support/core_ext/hash/conversions.rb (modified) (1 diff)
• trunk/activesupport/test/core_ext/hash_ext_test.rb (modified) (1 diff)

trunk/activesupport/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Added yielding of builder in Hash#to_xml [DHH]
-
-* Hash#with_indifferent_access now also converts hashes kept in arrays to indifferent access (makes it easier to treat HTML and XML parameters the same) [DHH]

trunk/activesupport/lib/active_support/core_ext/hash/conversions.rb

@@ -100 +100 @@
-              end
-            end
+            
+            yield options[:builder] if block_given?
-          end
-

trunk/activesupport/test/core_ext/hash_ext_test.rb

@@ -341 +341 @@
-    assert xml.include?(%(<name>David</name>))
-    assert xml.include?(%(<age nil="true"></age>))
+  end
+
+  def test_one_level_with_yielding
+    xml = { :name => "David", :street => "Paulina" }.to_xml(@xml_options) do |xml|
+      xml.creator("Rails")
+    end
+
+    assert_equal "<person>", xml.first(8)
+    assert xml.include?(%(<street>Paulina</street>))
+    assert xml.include?(%(<name>David</name>))
+    assert xml.include?(%(<creator>Rails</creator>))
-  end
-