Use a lazy match in add_order_by_for_association_limiting! to allow sub selects to be used. Resolves ticket #11

Author: metaskills

CHANGELOG

@@ -1,6 +1,9 @@
 
 MASTER
 
+* Use a lazy match in add_order_by_for_association_limiting! to allow sub selects to be used. Resolves 
+  ticket #11.
+
 * Add default rake task back for testing. Runs the namespaced sqlserver:test_sqlserver_odbc. 
   Resolves ticket #10 [Ken Collins]
 

lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -698,7 +698,7 @@ def add_order_by_for_association_limiting!(sql, options)
         # Disertation http://gist.github.com/24073
         # Information http://weblogs.sqlteam.com/jeffs/archive/2007/12/13/select-distinct-order-by-error.aspx
         return sql if options[:order].blank?
-        columns = sql.match(/SELECT\s+DISTINCT(.*)FROM/)[1].strip
+        columns = sql.match(/SELECT\s+DISTINCT(.*?)FROM/)[1].strip
         sql.sub!(/SELECT\s+DISTINCT/,'SELECT')
         sql << "GROUP BY #{columns} ORDER BY #{order_to_min_set(options[:order])}"
       end