before_destroy does not trigger any save callbacks.

lib/acts_as_list.rb

@@ -61,7 +61,7 @@ def position_column
 
         #{scope_condition_method}
 
-        before_destroy :remove_from_list
+        before_destroy :eliminate_current_position
         before_create  :add_to_list_bottom
       EOV
     end
@@ -248,6 +248,10 @@ def insert_at_position(position)
         increment_positions_on_lower_items(position)
         self.update_attribute(position_column, position)
       end
+
+      def eliminate_current_position          
+        decrement_positions_on_lower_items if in_list?
+      end
   end
 end
 

test/list_test.rb

@@ -32,9 +32,17 @@ class ListMixin < Mixin
   acts_as_list :column => "pos", :scope => :parent
 
   def self.table_name() "mixins" end
+
+  attr_accessor :before_save_triggered
+  before_save :log_before_save
+
+  def log_before_save
+    self.before_save_triggered = true
+  end
 end
 
 class ListMixinSub1 < ListMixin
+
 end
 
 class ListMixinSub2 < ListMixin
@@ -219,7 +227,14 @@ def test_remove_before_destroy_does_not_shift_lower_items_twice
     assert_equal 1, ListMixin.find(1).pos
     assert_equal 2, ListMixin.find(3).pos
     assert_equal 3, ListMixin.find(4).pos
-  end 
+  end
+
+  def test_should_not_trigger_unexpected_callbacks_on_destroy
+    element = ListMixin.find(2)
+    assert !element.before_save_triggered
+    element.destroy
+    assert !element.before_save_triggered
+  end
 
   # special thanks to openhood on github
   def test_delete_middle_with_holes