Skip to content

Commit 89d14f3

Browse files
committed
Order should be unique to first in line, not the reverse, duh!
1 parent 67e28d4 commit 89d14f3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/arel/visitors/sqlserver.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def visit_Arel_Nodes_SelectStatementWithOutOffset(o, windowed=false)
129129
core = o.cores.first
130130
projections = core.projections
131131
groups = core.groups
132-
orders = o.orders.reverse.uniq.reverse
132+
orders = o.orders.uniq
133133
if windowed
134134
projections = function_select_statement?(o) ? projections : projections.map { |x| projection_without_expression(x) }
135135
elsif eager_limiting_select_statement?(o)
@@ -320,7 +320,7 @@ def rowtable_orders(o)
320320
[table_from_select_statement(o).primary_key.asc]
321321
else
322322
[table_from_select_statement(o).primary_key.asc]
323-
end.reverse.uniq.reverse
323+
end.uniq
324324
end
325325

326326
# TODO: We use this for grouping too, maybe make Grouping objects vs SqlLiteral.

0 commit comments

Comments
 (0)