From 404b73bce1f9347ca05b1496db8fc64438d66bd2 Mon Sep 17 00:00:00 2001 From: kennyj Date: Sat, 25 Feb 2012 23:01:43 +0900 Subject: [PATCH] Remove NULLS FIRST/LAST. closes #5152 --- .../lib/active_record/connection_adapters/postgresql_adapter.rb | 2 +- .../test/cases/adapters/postgresql/postgresql_adapter_test.rb | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb index c675b64..d2126a3 100644 --- a/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/postgresql_adapter.rb @@ -1108,7 +1108,7 @@ def distinct(columns, orders) #:nodoc: # Construct a clean list of column names from the ORDER BY clause, removing # any ASC/DESC modifiers - order_columns = orders.collect { |s| s.gsub(/\s+(ASC|DESC)\s*/i, '') } + order_columns = orders.collect { |s| s.gsub(/\s+(ASC|DESC)\s*(NULLS\s+(FIRST|LAST)\s*)?/i, '') } order_columns.delete_if { |c| c.blank? } order_columns = order_columns.zip((0...order_columns.size).to_a).map { |s,i| "#{s} AS alias_#{i}" } -- 1.8.1.6