Better implementation

lib/mongoid/orderable/installer.rb

@@ -48,7 +48,7 @@ def include_mixins
       klass.send :include, Mixins::Callbacks
       klass.send :include, Mixins::Movable
       klass.send :include, Mixins::Listable
-      klass.send :prepend, Mixins::Changeable
+      klass.send :prepend, Mixins::Cascadeable if klass.embedded?
     end
 
     def generate_all_helpers

lib/mongoid/orderable/mixins/changeable.rb → lib/mongoid/orderable/mixins/cascadeable.rb

@@ -3,10 +3,10 @@
 module Mongoid
 module Orderable
 module Mixins
-  # This is required for dirty tracking on embedded objects.
+  # This is required to trigger callbacks on embedded objects.
   # Otherwise, the #move_to parameter won't work when saving the parent.
-  module Changeable
-    def changed?
+  module Cascadeable
+    def in_callback_state?(kind)
       super || move_all.present?
     end
   end

lib/mongoid_orderable.rb

@@ -16,9 +16,9 @@
 require 'mongoid/orderable/errors/transaction_failed'
 require 'mongoid/orderable/mixins/helpers'
 require 'mongoid/orderable/mixins/callbacks'
-require 'mongoid/orderable/mixins/changeable'
 require 'mongoid/orderable/mixins/listable'
 require 'mongoid/orderable/mixins/movable'
+require 'mongoid/orderable/mixins/cascadeable'
 require 'mongoid/orderable/generators/base'
 require 'mongoid/orderable/generators/listable'
 require 'mongoid/orderable/generators/lock_collection'

spec/integration/cascadeable_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe Mongoid::Orderable::Mixins::Cascadeable do
+
+  it do
+    eo = EmbedsOrderable.create!
+    orderable = eo.embedded_orderables.create!
+    expect(orderable.in_callback_state?(:update)).to eq false
+    orderable.move_to = 2
+    expect(orderable.in_callback_state?(:update)).to eq true
+  end
+end