Changeset 8367

Timestamp:

12/10/07 09:12:18 (9 months ago)

Author:

bitsweat

Message:

Update destroy_all and delete_all documentation to better describe their tradeoffs. Closes #10447.

Files:

• trunk/activerecord/CHANGELOG (modified) (1 diff)
• trunk/activerecord/lib/active_record/base.rb (modified) (1 diff)

trunk/activerecord/CHANGELOG

@@ -1 +1 @@
-*SVN*
-
- Active Record exceptions.  #10444
+ation: Active Record exceptions, destroy_all and delete_all.  #10444, #10447
-
-

trunk/activerecord/lib/active_record/base.rb

@@ -679 +679 @@
-      end
-
-
-
+
+
+
+
+
+
+
+
+
+
+
-      #   Person.destroy_all "last_login < '2004-04-04'"
+      #
+      # This loads and destroys each person one by one, including its dependent associations and before_ and
+      # after_destroy callbacks.
-      def destroy_all(conditions = nil)
-        find(:all, :conditions => conditions).each { |object| object.destroy }
-      end
-
-
-
+
+
+
+
+
+
+
+
+
+
-      #   Post.delete_all "person_id = 5 AND (category = 'Something' OR category = 'Else')"
+      #
+      # This deletes the affected posts all at once with a single DELETE query. If you need to destroy dependent
+      # associations or call your before_ or after_destroy callbacks, use the +destroy_all+ method instead.
-      def delete_all(conditions = nil)
-        sql = "DELETE FROM #{quoted_table_name} "