Permalink
Browse files

Allow extra arguments for Observers

  • Loading branch information...
1 parent ad2c5ea commit 24c068d67dab4559ab24f77482cd671563161ec2 @marcandre marcandre committed Apr 29, 2012
Showing with 26 additions and 11 deletions.
  1. +17 −9 activemodel/lib/active_model/observing.rb
  2. +9 −2 activemodel/test/cases/observing_test.rb
View
26 activemodel/lib/active_model/observing.rb
@@ -112,13 +112,21 @@ def inherited(subclass)
private
# Fires notifications to model's observers
#
- # def save
- # notify_observers(:before_save)
- # ...
- # notify_observers(:after_save)
- # end
- def notify_observers(method)
- self.class.notify_observers(method, self)
+ # def save
+ # notify_observers(:before_save)
+ # ...
+ # notify_observers(:after_save)
+ # end
+ #
+ # Custom notifications can be sent in a similar fashion:
+ #
+ # notify_observers(:custom_notification, :foo)
+ #
+ # This will call +custom_notification+, passing as arguments
+ # the current object and :foo.
+ #
+ def notify_observers(method, *extra_args)
+ self.class.notify_observers(method, self, *extra_args)
end
end
@@ -229,10 +237,10 @@ def observed_classes #:nodoc:
# Send observed_method(object) if the method exists and
# the observer is enabled for the given object's class.
- def update(observed_method, object, &block) #:nodoc:
+ def update(observed_method, object, *extra_args, &block) #:nodoc:
return unless respond_to?(observed_method)
return if disabled_for?(object)
- send(observed_method, object, &block)
+ send(observed_method, object, *extra_args, &block)
end
# Special method sent by the observed class when it is inherited.
View
11 activemodel/test/cases/observing_test.rb
@@ -14,8 +14,8 @@ class << self
attr_accessor :stub
- def on_spec(record)
- stub.event_with(record) if stub
+ def on_spec(record, *args)
+ stub.event_with(record, *args) if stub
end
def around_save(record)
@@ -133,6 +133,13 @@ def teardown
Foo.send(:notify_observers, :on_spec, foo)
end
+ test "passes extra arguments" do
+ foo = Foo.new
+ FooObserver.instance.stub = stub
+ FooObserver.instance.stub.expects(:event_with).with(foo, :bar)
+ Foo.send(:notify_observers, :on_spec, foo, :bar)
+ end
+
test "skips nonexistent observer event" do
foo = Foo.new
Foo.send(:notify_observers, :whatever, foo)

0 comments on commit 24c068d

Please sign in to comment.