Permalink
Browse files

Rename Observing#count_observers to Observing#observers_count

  • Loading branch information...
1 parent 60ac540 commit 27d6ccd8c86cffaa5f74ebfb6861ece429172854 @oscardelben oscardelben committed Apr 14, 2012
Showing with 9 additions and 2 deletions.
  1. +8 −1 activemodel/lib/active_model/observing.rb
  2. +1 −1 activemodel/test/cases/observing_test.rb
@@ -4,6 +4,7 @@
require 'active_support/core_ext/module/remove_method'
require 'active_support/core_ext/string/inflections'
require 'active_support/core_ext/enumerable'
+require 'active_support/deprecation'
require 'active_support/descendants_tracker'
module ActiveModel
@@ -74,10 +75,16 @@ def notify_observers(*arg)
end
# Total number of observers.
- def count_observers
+ def observers_count
observer_instances.size
end
+ def count_observers
+ msg = "count_observers is deprecated in favor of observers_count"
+ ActiveSupport::Deprecation.warn(msg)
+ observers_count
+ end
+
protected
def instantiate_observer(observer) #:nodoc:
# string/symbol
@@ -73,7 +73,7 @@ def setup
test "passes observers to subclasses" do
FooObserver.instance
bar = Class.new(Foo)
- assert_equal Foo.count_observers, bar.count_observers
+ assert_equal Foo.observers_count, bar.observers_count
end
end

0 comments on commit 27d6ccd

Please sign in to comment.