Fixed acts_as_list to trigger remove_from_list on destroy after the fact, not before, so a unique position can be maintained #871 [Alisdair McDiarmid]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@923 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

activerecord/CHANGELOG

@@ -1,5 +1,7 @@
 *SVN*
 
+* Fixed acts_as_list to trigger remove_from_list on destroy after the fact, not before, so a unique position can be maintained #871 [Alisdair McDiarmid]
+
 * Added the possibility of specifying fixtures in multiple calls #816 [kim@tinker.com]
 
 * Added Base.exists?(id) that'll return true if an object of the class with the given id exists #854 [stian@grytoyr.net]

activerecord/lib/active_record/acts/list.rb

@@ -60,7 +60,7 @@ module ActiveRecord
             
             #{scope_condition_method}
             
-            before_destroy :remove_from_list
+            after_destroy  :remove_from_list
             before_create  :add_to_list_bottom
           EOV
         end