diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-19 08:28:57 -0700 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-19 08:28:57 -0700 |
commit | 1dbf1c80ac4693634f536b410e7d4910cc2e173f (patch) | |
tree | 687385bd46b8a6d0761ea35fc82e9e6417c5eccb /activerecord/lib | |
parent | f9fc26e8007d5e2936cf5374a8168b2c37273490 (diff) | |
parent | bf80522be4d7ac521fa3c30d889afa68450a0bc2 (diff) | |
download | rails-1dbf1c80ac4693634f536b410e7d4910cc2e173f.tar.gz rails-1dbf1c80ac4693634f536b410e7d4910cc2e173f.tar.bz2 rails-1dbf1c80ac4693634f536b410e7d4910cc2e173f.zip |
Merge pull request #7392 from ernie/real-fix-for-last-equality-wins-in-merge
Fix "last equality wins" logic in relation merge
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/relation/merger.rb | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/activerecord/lib/active_record/relation/merger.rb b/activerecord/lib/active_record/relation/merger.rb index 7531f22494..e5b50673da 100644 --- a/activerecord/lib/active_record/relation/merger.rb +++ b/activerecord/lib/active_record/relation/merger.rb @@ -97,18 +97,13 @@ module ActiveRecord merged_wheres = relation.where_values + values[:where] unless relation.where_values.empty? - # Remove duplicate ARel attributes. Last one wins. - seen = Hash.new { |h,table| h[table] = {} } + # Remove equalities with duplicated left-hand. Last one wins. + seen = {} merged_wheres = merged_wheres.reverse.reject { |w| nuke = false - # We might have non-attributes on the left side of equality nodes, - # so we need to make sure they quack like an attribute. - if w.respond_to?(:operator) && w.operator == :== && - w.left.respond_to?(:relation) - name = w.left.name - table = w.left.relation.name - nuke = seen[table][name] - seen[table][name] = true + if w.respond_to?(:operator) && w.operator == :== + nuke = seen[w.left] + seen[w.left] = true end nuke }.reverse |