Fix bug where calculations with long alias names return null. [Rick]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4269 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

1 files changed, 4 insertions, 4 deletions

diff --git a/activerecord/lib/active_record/calculations.rb b/activerecord/lib/active_record/calculations.rb
index 9a747f661b..83bf279785 100644
--- a/activerecord/lib/active_record/calculations.rb
+++ b/activerecord/lib/active_record/calculations.rb
@@ -172,10 +172,10 @@ module ActiveRecord
           end
           add_joins!(sql, options, scope)
           add_conditions!(sql, options[:conditions], scope)
-          sql << " GROUP BY #{options[:group_field]}" if options[:group]
-          sql << " HAVING #{options[:having]}"        if options[:group] && options[:having]
-          sql << " ORDER BY #{options[:order]}"       if options[:order]
           add_limited_ids_condition!(sql, options, join_dependency) if join_dependency && !using_limitable_reflections?(join_dependency.reflections) && ((scope && scope[:limit]) || options[:limit])
+          sql << " GROUP BY #{options[:group_field]} " if options[:group]
+          sql << " HAVING #{options[:having]} "        if options[:group] && options[:having]
+          sql << " ORDER BY #{options[:order]} "       if options[:order]
           add_limit!(sql, options, scope)
           sql << ')' if use_workaround
           sql
@@ -222,7 +222,7 @@ module ActiveRecord
         #   count(distinct users.id) #=> count_distinct_users_id
         #   count(*) #=> count_all
         def column_alias_for(*keys)
-          keys.join(' ').downcase.gsub(/\*/, 'all').gsub(/\W+/, ' ').strip.gsub(/ +/, '_')
+          connection.table_alias_for(keys.join(' ').downcase.gsub(/\*/, 'all').gsub(/\W+/, ' ').strip.gsub(/ +/, '_'))
         end
 
         def column_for(field)