aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Heinemeier Hansson <david@loudthinking.com>2005-03-20 13:39:15 +0000
committerDavid Heinemeier Hansson <david@loudthinking.com>2005-03-20 13:39:15 +0000
commit6717ed61c11e62c11eb7d5c42638962abfc6dd8b (patch)
tree47f035378203e19b9452206dacc427cadb0e56b0
parent9c605227ec14ba4217d4a9396e9986af73fa522e (diff)
downloadrails-6717ed61c11e62c11eb7d5c42638962abfc6dd8b.tar.gz
rails-6717ed61c11e62c11eb7d5c42638962abfc6dd8b.tar.bz2
rails-6717ed61c11e62c11eb7d5c42638962abfc6dd8b.zip
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
-rw-r--r--activerecord/CHANGELOG2
-rw-r--r--activerecord/lib/active_record/acts/list.rb2
2 files changed, 3 insertions, 1 deletions
diff --git a/activerecord/CHANGELOG b/activerecord/CHANGELOG
index 4bc7135995..9645f4fca3 100644
--- a/activerecord/CHANGELOG
+++ b/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]
diff --git a/activerecord/lib/active_record/acts/list.rb b/activerecord/lib/active_record/acts/list.rb
index ad61f9f47d..abd295d4e9 100644
--- a/activerecord/lib/active_record/acts/list.rb
+++ b/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