#to_sym is unnecessary

rails · May 4, 2012 · 780e893 · 780e893
1 parent 4bd9482
commit 780e893
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/activerecord/lib/active_record/dynamic_matchers.rb b/activerecord/lib/active_record/dynamic_matchers.rb
@@ -63,15 +63,15 @@ def initialize(model, name)
       def expand_attribute_names_for_aggregates
         attribute_names.map do |attribute_name|
           if aggregation = model.reflect_on_aggregation(attribute_name.to_sym)
-            aggregation.mapping.map { |m| m.first.to_sym }
+            aggregation.mapping.map(&:first)
           else
-            attribute_name.to_sym
+            attribute_name
           end
         end.flatten
       end
 
       def valid?
-        (expand_attribute_names_for_aggregates - model.column_methods_hash.keys).empty?
+        (expand_attribute_names_for_aggregates - model.column_names).empty?
       end
 
       def define