zerobearing2 · September 5, 2010 19:01 · Sep 5, 2010 · Sep 5, 2010
diff --git a/embedded_callbacks.rb b/embedded_callbacks.rb
@@ -5,8 +5,6 @@ module EmbeddedCallbacks
 
       # bubble callbacks to embedded assocaitions
       def run_callbacks(*args)
-        parent_callback_result = super(args) # defer to parent
-
         # now bubble callbacks down
         self.associations.each_pair do |name, meta|
           if meta.association == Mongoid::Associations::EmbedsMany
@@ -15,8 +13,7 @@ def run_callbacks(*args)
             self.send(name).send(:run_callbacks, args)
           end
         end
-
-        parent_callback_result
+        super(args) # defer to parent
       end
 
     end

diff --git a/embedded_callbacks.rb b/embedded_callbacks.rb
@@ -0,0 +1,24 @@
+# encoding: utf-8
+module Mongoid #:nodoc:
+  module Associations #:nodoc:
+    module EmbeddedCallbacks
+
+      # bubble callbacks to embedded assocaitions
+      def run_callbacks(*args)
+        parent_callback_result = super(args) # defer to parent
+
+        # now bubble callbacks down
+        self.associations.each_pair do |name, meta|
+          if meta.association == Mongoid::Associations::EmbedsMany
+            self.send(name).each{|doc| doc.send(:run_callbacks, args)}
+          elsif meta.association == Mongoid::Associations::EmbedsOne
+            self.send(name).send(:run_callbacks, args)
+          end
+        end
+
+        parent_callback_result
+      end
+
+    end
+  end
+end
diff --git a/photo.rb b/photo.rb
@@ -0,0 +1,8 @@
+class Photo
+  include Mongoid::Document
+  include Mongoid::Timestamps
+
+  mount_uploader :image, ImageFileUploader
+  embedded_in :some_model, :inverse_of => :photos
+  validates_presence_of :image
+end
diff --git a/some_model.rb b/some_model.rb
@@ -0,0 +1,6 @@
+class SomeModel
+  include Mongoid::Document
+  include Mongoid::Associations::EmbeddedCallbacks
+  include Mongoid::Timestamps
+  embeds_many :photos
+end