diff options
author | Sven Fuchs <svenfuchs@artweb-design.de> | 2010-12-07 10:05:42 +0100 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2010-12-07 10:48:24 -0800 |
commit | 16ee8f66bcc527406e44c28a441e8f25481c5a0d (patch) | |
tree | 9ab0a6549a8c4c8af7bf911fa1ec326e6e8c95cb /lib/arel/visitors | |
parent | 1c0d8c980c3b528ce2aeb0a528590822a31b5ef2 (diff) | |
download | rails-16ee8f66bcc527406e44c28a441e8f25481c5a0d.tar.gz rails-16ee8f66bcc527406e44c28a441e8f25481c5a0d.tar.bz2 rails-16ee8f66bcc527406e44c28a441e8f25481c5a0d.zip |
implementation for passing a subquery to #in and #not_in
Diffstat (limited to 'lib/arel/visitors')
-rw-r--r-- | lib/arel/visitors/to_sql.rb | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/lib/arel/visitors/to_sql.rb b/lib/arel/visitors/to_sql.rb index 2fb464b265..6fc206c12c 100644 --- a/lib/arel/visitors/to_sql.rb +++ b/lib/arel/visitors/to_sql.rb @@ -72,6 +72,10 @@ module Arel }.join ', '})" end + def visit_Arel_SelectManager o + o.to_sql + end + def visit_Arel_Nodes_SelectStatement o [ o.cores.map { |x| visit_Arel_Nodes_SelectCore x }.join, @@ -204,17 +208,11 @@ module Arel end def visit_Arel_Nodes_In o - right = o.right - "#{visit o.left} IN (#{ - right.empty? ? 'NULL' : right.map { |x| visit x }.join(', ') - })" + "#{visit o.left} IN (#{visit o.right})" end def visit_Arel_Nodes_NotIn o - right = o.right - "#{visit o.left} NOT IN (#{ - right.empty? ? 'NULL' : right.map { |x| visit x }.join(', ') - })" + "#{visit o.left} NOT IN (#{visit o.right})" end def visit_Arel_Nodes_And o @@ -291,6 +289,10 @@ module Arel alias :visit_ActiveSupport_StringInquirer :visit_String alias :visit_Class :visit_String + def visit_Array o + o.empty? ? 'NULL' : o.map { |x| visit x }.join(', ') + end + def quote value, column = nil @connection.quote value, column end |