diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2013-07-23 05:29:17 -0700 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2013-07-23 05:29:17 -0700 |
| commit | c4b93f57e1c29d6e1726d23d6ac2fdb75af9bdab (patch) | |
| tree | 7fdbeebaac918d4cef96f2b294697e2c2cf850db /activerecord/lib | |
| parent | e0db277be0a57386d299fc0a231b585a612eb9ba (diff) | |
| parent | b289519f4ff7829271674d66d3196e1ee23056b6 (diff) | |
| download | rails-c4b93f57e1c29d6e1726d23d6ac2fdb75af9bdab.tar.gz rails-c4b93f57e1c29d6e1726d23d6ac2fdb75af9bdab.tar.bz2 rails-c4b93f57e1c29d6e1726d23d6ac2fdb75af9bdab.zip | |
Merge pull request #11563 from 90yukke/fix-nomethoderror-on-non-attribute-equalities
Fix merge error when Equality LHS is non-attribute.
Diffstat (limited to 'activerecord/lib')
| -rw-r--r-- | activerecord/lib/active_record/relation/spawn_methods.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/relation/spawn_methods.rb b/activerecord/lib/active_record/relation/spawn_methods.rb index 90f2ac3cde..b734fc5c51 100644 --- a/activerecord/lib/active_record/relation/spawn_methods.rb +++ b/activerecord/lib/active_record/relation/spawn_methods.rb @@ -32,11 +32,12 @@ module ActiveRecord merged_wheres = @where_values + r.where_values unless @where_values.empty? - # Remove duplicates, last one wins. + # Remove duplicate ARel attributes. Last one wins. seen = Hash.new { |h,table| h[table] = {} } merged_wheres = merged_wheres.reverse.reject { |w| nuke = false - if w.respond_to?(:operator) && w.operator == :== + 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] |