From 2afd6c75f18aee67fab85efef2b970572d459db3 Mon Sep 17 00:00:00 2001
From: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri, 14 Jan 2011 16:46:02 -0800
Subject: move complex logic to it's own method

---
 .../associations/association_collection.rb         | 34 ++++++++++++----------
 1 file changed, 19 insertions(+), 15 deletions(-)

(limited to 'activerecord/lib/active_record/associations/association_collection.rb')

diff --git a/activerecord/lib/active_record/associations/association_collection.rb b/activerecord/lib/active_record/associations/association_collection.rb
index bf40446871..60a8d71d26 100644
--- a/activerecord/lib/active_record/associations/association_collection.rb
+++ b/activerecord/lib/active_record/associations/association_collection.rb
@@ -353,21 +353,7 @@ module ActiveRecord
             unless loaded?
               begin
                 if @target.is_a?(Array) && @target.any?
-                  @target = find_target.map do |f|
-                    i = @target.index(f)
-                    if i
-                      @target.delete_at(i).tap do |t|
-                        keys = ["id"] + t.changes.keys + (f.attribute_names - t.attribute_names)
-                        # FIXME: this call to attributes causes many NoMethodErrors
-                        attributes = f.attributes
-                        (attributes.keys - keys).each do |k|
-                          t.send("#{k}=", attributes[k])
-                        end
-                      end
-                    else
-                      f
-                    end
-                  end + @target
+                  @target = merge_target_lists(find_target, @target)
                 else
                   @target = find_target
                 end
@@ -450,6 +436,24 @@ module ActiveRecord
         end
 
       private
+        def merge_target_lists(loaded, existing)
+          loaded.map do |f|
+            i = existing.index(f)
+            if i
+              existing.delete_at(i).tap do |t|
+                keys = ["id"] + t.changes.keys + (f.attribute_names - t.attribute_names)
+                # FIXME: this call to attributes causes many NoMethodErrors
+                attributes = f.attributes
+                (attributes.keys - keys).each do |k|
+                  t.send("#{k}=", attributes[k])
+                end
+              end
+            else
+              f
+            end
+          end + existing
+        end
+
         # Do the relevant stuff to insert the given record into the association collection. The
         # force param specifies whether or not an exception should be raised on failure. The
         # validate param specifies whether validation should be performed (if force is false).
-- 
cgit v1.2.3