From 24a9050b6e34c859909f4e16ee68b8b34407a771 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Mon, 2 May 2005 16:46:30 +0000
Subject: Changed the callbacks such that observers are notified before the
 in-object callbacks are triggered. Without this change, it wasn't possible to
 act on the whole object in something like a before_destroy observer without
 having the objects own callbacks (like deleting associations) called first.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1273 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 activerecord/lib/active_record/base.rb      |  9 +++++++++
 activerecord/lib/active_record/callbacks.rb | 11 +++++++----
 2 files changed, 16 insertions(+), 4 deletions(-)

(limited to 'activerecord/lib/active_record')

diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb
index a74f8ce471..4705d9359c 100755
--- a/activerecord/lib/active_record/base.rb
+++ b/activerecord/lib/active_record/base.rb
@@ -1128,6 +1128,15 @@ module ActiveRecord #:nodoc:
         self.class.column_methods_hash[method.to_sym] || respond_to_without_attributes?(method, include_priv)
       end
       
+      # Just freeze the attributes hash, such that associations are still accessible even on destroyed records.
+      def freeze
+        @attributes.freeze
+      end
+      
+      def frozen?
+        @attributes.frozen?
+      end
+      
     private
       def create_or_update
         if new_record? then create else update end
diff --git a/activerecord/lib/active_record/callbacks.rb b/activerecord/lib/active_record/callbacks.rb
index 23675c3bd4..57745b344e 100755
--- a/activerecord/lib/active_record/callbacks.rb
+++ b/activerecord/lib/active_record/callbacks.rb
@@ -328,6 +328,8 @@ module ActiveRecord
 
     private
       def callback(method)
+        notify(method)
+
         callbacks_for(method).each do |callback|
           result = case callback
             when Symbol
@@ -345,9 +347,10 @@ module ActiveRecord
           end
           return false if result == false
         end
-
-        invoke_and_notify(method)
-        true
+        
+        send(method) if respond_to_without_attributes?(method)
+        
+        return true
       end
 
       def callbacks_for(method)
@@ -355,8 +358,8 @@ module ActiveRecord
       end
 
       def invoke_and_notify(method)
-        send(method) if respond_to_without_attributes?(method)
         notify(method)
+        send(method) if respond_to_without_attributes?(method)
       end
 
       def notify(method) #:nodoc:
-- 
cgit v1.2.3