Skip to content
Browse files

Fixed acts_as_list to trigger remove_from_list on destroy after the f…

…act, 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
  • Loading branch information...
1 parent 9c60522 commit 6717ed61c11e62c11eb7d5c42638962abfc6dd8b @dhh dhh committed
Showing with 3 additions and 1 deletion.
  1. +2 −0 activerecord/CHANGELOG
  2. +1 −1 activerecord/lib/active_record/acts/list.rb
View
2 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]
View
2 activerecord/lib/active_record/acts/list.rb
@@ -60,7 +60,7 @@ def position_column
#{scope_condition_method}
- before_destroy :remove_from_list
+ after_destroy :remove_from_list
before_create :add_to_list_bottom
EOV
end

0 comments on commit 6717ed6

Please sign in to comment.
Something went wrong with that request. Please try again.