1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00

use grep instead of select + is_a?

This commit is contained in:
Aaron Patterson 2010-10-20 16:41:50 -07:00
parent 4be0fc124a
commit 828bb94d5a

View file

@ -36,7 +36,7 @@ module ActiveRecord
# Returns an array of AggregateReflection objects for all the aggregations in the class. # Returns an array of AggregateReflection objects for all the aggregations in the class.
def reflect_on_all_aggregations def reflect_on_all_aggregations
reflections.values.select { |reflection| reflection.is_a?(AggregateReflection) } reflections.values.grep(AggregateReflection)
end end
# Returns the AggregateReflection object for the named +aggregation+ (use the symbol). # Returns the AggregateReflection object for the named +aggregation+ (use the symbol).
@ -58,7 +58,7 @@ module ActiveRecord
# Account.reflect_on_all_associations(:has_many) # returns an array of all has_many associations # Account.reflect_on_all_associations(:has_many) # returns an array of all has_many associations
# #
def reflect_on_all_associations(macro = nil) def reflect_on_all_associations(macro = nil)
association_reflections = reflections.values.select { |reflection| reflection.is_a?(AssociationReflection) } association_reflections = reflections.values.grep(AssociationReflection)
macro ? association_reflections.select { |reflection| reflection.macro == macro } : association_reflections macro ? association_reflections.select { |reflection| reflection.macro == macro } : association_reflections
end end