0
@@ -1,3 +1,6 @@
0
+* Add some event callbacks, #aasm_event_fired(from, to), and #aasm_event_failed(event)
0
+  Based on transition logging suggestion [Artem Vasiliev] and timestamp column suggestion [Mike Ferrier]
0
+
0
 * Add #aasm_events_for_state and #aasm_events_for_current_state [Joao Paulo Lins]
0
 
0
 * Ensure that a state is written for a new record even if aasm_current_state or

0
@@ -9,5 +9,3 @@ Cool ideas from users:
0
 
0
 * Support multiple state machines on one object (Chris Nelson)
0
 * http://justbarebones.blogspot.com/2007/11/actsasstatemachine-enhancements.html (Chetan Patil)
0
-* transition logging (Artem Vasiliev)
0
-* timestamp column for state change on AR (Mike Ferrier)

0
@@ -41,9 +41,17 @@ module AASM
0
       define_method("#{name.to_s}!") do
0
         new_state = self.class.aasm_events[name].fire(self)
0
         unless new_state.nil?
0
+          if self.respond_to?(:aasm_event_fired)
0
+            self.aasm_event_fired(self.aasm_current_state, new_state)
0
+          end
0
+          
0
           self.aasm_current_state = new_state
0
           true
0
         else
0
+          if self.respond_to?(:aasm_event_failed)
0
+            self.aasm_event_failed(name)
0
+          end
0
+          
0
           false
0
         end
0
       end

0
@@ -9,6 +9,14 @@ class Foo
0
   aasm_event :close do
0
     transitions :to => :closed, :from => [:open]
0
   end
0
+
0
+  aasm_event :null do
0
+    transitions :to => :closed, :from => [:open], :guard => :always_false
0
+  end
0
+
0
+  def always_false
0
+    false
0
+  end
0
 end
0
 
0
 class Bar
0
@@ -122,3 +130,27 @@ describe AASM, '- getting events for a state' do
0
     foo.aasm_events_for_current_state
0
   end
0
 end
0
+
0
+describe AASM, '- event callbacks' do
0
+  it 'should call aasm_event_fired if defined and successful' do
0
+    foo = Foo.new
0
+    def foo.aasm_event_fired(from, to)
0
+    end
0
+
0
+    foo.should_receive(:aasm_event_fired)
0
+
0
+    foo.close!
0
+  end
0
+
0
+  it 'should call aasm_event_failed if defined and transition failed' do
0
+    foo = Foo.new
0
+    def foo.aasm_event_failed(event)
0
+    end
0
+
0
+    foo.should_receive(:aasm_event_failed)
0
+
0
+    foo.null!
0
+  end
0
+end
0
+
0
+