Permalink
Browse files

Fix a problem where nil was appearing in the list

  • Loading branch information...
1 parent 7d7f9cc commit 2f98032fc92ce16125f8628b4d3c283f10494f4d Yehuda Katz committed Feb 19, 2010
Showing with 6 additions and 4 deletions.
  1. +6 −4 activesupport/lib/active_support/dependencies.rb
@@ -76,17 +76,19 @@ def get(key)
locked :concat, :each, :delete_if, :<<
def new_constants_for(frames)
- frames.map do |mod_name, prior_constants|
+ constants = []
+ frames.each do |mod_name, prior_constants|
mod = Inflector.constantize(mod_name) if Dependencies.qualified_const_defined?(mod_name)
next unless mod.is_a?(Module)
new_constants = mod.local_constant_names - prior_constants
get(mod_name).concat(new_constants)
- new_constants.map do |suffix|
- ([mod_name, suffix] - ["Object"]).join("::")
+ new_constants.each do |suffix|
+ constants << ([mod_name, suffix] - ["Object"]).join("::")
end
- end.flatten
+ end
+ constants
end
# Add a set of modules to the watch stack, remembering the initial constants

0 comments on commit 2f98032

Please sign in to comment.