Follow up #370 for Oracle, to fix undefined method `expr' for 10:Fixn…

…um bug.
rails · Jul 13, 2015 · 74563c6 · 74563c6
1 parent 14df08d
commit 74563c6
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.expr
+          limit    = o.limit.expr
           offset   = o.offset
           o.offset = nil
           collector << "

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