From 5c21f006424d96a7dc70885886969664aaa0b534 Mon Sep 17 00:00:00 2001
From: Blythe Dunham <blythedunham@gmail.com>
Date: Fri, 22 May 2009 10:34:31 -0700
Subject: [PATCH] Add :margin option to XmlSerializer

---
 .../active_record/serializers/xml_serializer.rb    |    3 ++-
 activerecord/test/cases/xml_serialization_test.rb  |    6 ++++++
 2 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/activerecord/lib/active_record/serializers/xml_serializer.rb b/activerecord/lib/active_record/serializers/xml_serializer.rb
index 4eaf953..d659e37 100644
--- a/activerecord/lib/active_record/serializers/xml_serializer.rb
+++ b/activerecord/lib/active_record/serializers/xml_serializer.rb
@@ -169,7 +169,8 @@ module ActiveRecord #:nodoc:
       @builder ||= begin
         require 'builder' unless defined? ::Builder
         options[:indent] ||= 2
-        builder = options[:builder] ||= ::Builder::XmlMarkup.new(:indent => options[:indent])
+        options[:margin] ||= 0
+        builder = options[:builder] ||= ::Builder::XmlMarkup.new(:indent => options[:indent], :margin => options[:margin])
 
         unless options[:skip_instruct]
           builder.instruct!
diff --git a/activerecord/test/cases/xml_serialization_test.rb b/activerecord/test/cases/xml_serialization_test.rb
index b499976..53d122b 100644
--- a/activerecord/test/cases/xml_serialization_test.rb
+++ b/activerecord/test/cases/xml_serialization_test.rb
@@ -38,6 +38,12 @@ class XmlSerializationTest < ActiveRecord::TestCase
     assert_match %r{<CreatedAt},    @xml
   end
 
+  def test_should_serialize_custom_root
+    @xml = Contact.new.to_xml :margin => 3
+    assert_match %r{^ {6}<contact>},  @xml
+    assert_match %r{ {8}<created-at}, @xml
+  end
+
   def test_should_allow_skipped_types
     @xml = Contact.new(:age => 25).to_xml :skip_types => true
     assert %r{<age>25</age>}.match(@xml)
-- 
1.6.1.3