Skip to content

Commit

Permalink
Removed ActiveSupport#load_all!
Browse files Browse the repository at this point in the history
This is no longer used and actually raises an error when trying to load
`ActiveSupport::Dependencies`. I removed the related code and added the
`Dependencies` module to the autoload list.
  • Loading branch information
panthomakos committed Mar 18, 2012
1 parent d3bbb23 commit 0f8f99b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 18 deletions.
15 changes: 1 addition & 14 deletions activesupport/lib/active_support.rb
Original file line number Diff line number Diff line change
Expand Up @@ -22,20 +22,6 @@
#++

require 'securerandom'

module ActiveSupport
class << self
attr_accessor :load_all_hooks
def on_load_all(&hook) load_all_hooks << hook end
def load_all!; load_all_hooks.each { |hook| hook.call } end
end
self.load_all_hooks = []

on_load_all do
[Dependencies, Deprecation, Gzip, MessageVerifier, Multibyte]
end
end

require "active_support/dependencies/autoload"
require "active_support/version"
require "active_support/logger"
Expand All @@ -44,6 +30,7 @@ module ActiveSupport
extend ActiveSupport::Autoload

autoload :Concern
autoload :Dependencies
autoload :DescendantsTracker
autoload :FileUpdateChecker
autoload :LogSubscriber
Expand Down
4 changes: 0 additions & 4 deletions activesupport/lib/active_support/time.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ module ActiveSupport
autoload :Duration, 'active_support/duration'
autoload :TimeWithZone, 'active_support/time_with_zone'
autoload :TimeZone, 'active_support/values/time_zone'

on_load_all do
[Duration, TimeWithZone, TimeZone]
end
end

require 'date'
Expand Down

0 comments on commit 0f8f99b

Please sign in to comment.