Support to add the casting node to the AST at build time

Ref rails@93d7213
truffaut · May 5, 2014 · 801a151 · 801a151
1 parent e746106
commit 801a151
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/arel/visitors/oracle.rb b/lib/arel/visitors/oracle.rb
@@ -17,7 +17,7 @@ def visit_Arel_Nodes_SelectStatement o, collector
 
         if o.limit && o.offset
           o        = o.dup
-          limit    = o.limit.expr.to_i
+          limit    = o.limit.expr.expr
           offset   = o.offset
           o.offset = nil
           collector << "

diff --git a/test/visitors/test_oracle.rb b/test/visitors/test_oracle.rb
@@ -100,7 +100,7 @@ def compile node
 
           it 'creates a different subquery when there is an offset' do
             stmt = Nodes::SelectStatement.new
-            stmt.limit = Nodes::Limit.new(10)
+            stmt.limit = Nodes::Limit.new(Nodes.build_quoted(10))
             stmt.offset = Nodes::Offset.new(10)
             sql = compile stmt
             sql.must_be_like %{
@@ -115,7 +115,7 @@ def compile node
 
           it 'is idempotent with different subquery' do
             stmt = Nodes::SelectStatement.new
-            stmt.limit = Nodes::Limit.new(10)
+            stmt.limit = Nodes::Limit.new(Nodes.build_quoted(10))
             stmt.offset = Nodes::Offset.new(10)
             sql = compile stmt
             sql2 = compile stmt