aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2010-11-06 14:32:41 -0700
committerAaron Patterson <aaron.patterson@gmail.com>2010-11-06 14:32:41 -0700
commitc11102d4f80267e3774985329ea93e7368e8684f (patch)
tree0b1d0ab22bb3266502d8789a0c0efb62f70f237e /activerecord
parent39a80567918af298b39898d78062e7a8fb5da601 (diff)
downloadrails-c11102d4f80267e3774985329ea93e7368e8684f.tar.gz
rails-c11102d4f80267e3774985329ea93e7368e8684f.tar.bz2
rails-c11102d4f80267e3774985329ea93e7368e8684f.zip
make sure we keep parent records in order
Diffstat (limited to 'activerecord')
-rw-r--r--activerecord/lib/active_record/associations.rb12
1 files changed, 6 insertions, 6 deletions
diff --git a/activerecord/lib/active_record/associations.rb b/activerecord/lib/active_record/associations.rb
index 8d415cdfdf..aad0cbb895 100644
--- a/activerecord/lib/active_record/associations.rb
+++ b/activerecord/lib/active_record/associations.rb
@@ -1873,15 +1873,15 @@ module ActiveRecord
def instantiate(rows)
primary_key = join_base.aliased_primary_key
- base_records_hash = {}
+ parents = {}
- rows.each do |model|
+ records = rows.map { |model|
primary_id = model[primary_key]
- base_records_hash[primary_id] ||= join_base.instantiate(model)
- construct(base_records_hash[primary_id], @associations, join_associations.dup, model)
- end
+ parent = parents[primary_id] ||= join_base.instantiate(model)
+ construct(parent, @associations, join_associations.dup, model)
+ parent
+ }.uniq
- records = base_records_hash.values
remove_duplicate_results!(join_base.active_record, records, @associations)
records
end