Author: jacob
Date: 2009-05-13 21:35:16 -0500 (Wed, 13 May 2009)
New Revision: 10780

Modified:
   django/trunk/docs/ref/contrib/admin/actions.txt
Log:
Fixed #11022: documented that the admin bulk delete action calls 
`QuerySet.delete()`, not `Model.delete()`. Thanks, Idan Gazit.

Modified: django/trunk/docs/ref/contrib/admin/actions.txt
===================================================================
--- django/trunk/docs/ref/contrib/admin/actions.txt     2009-05-14 02:30:25 UTC 
(rev 10779)
+++ django/trunk/docs/ref/contrib/admin/actions.txt     2009-05-14 02:35:16 UTC 
(rev 10780)
@@ -23,7 +23,20 @@
 :mod:`django.contrib.auth` app:
 
 .. image:: _images/user_actions.png
+
+.. warning::
+
+    The "delete selected objects" action uses :meth:`QuerySet.delete()
+    <django.db.models.QuerySet.delete>` for efficiency reasons, which has an
+    important caveat: your model's ``delete()`` method will not be called.
     
+    If you wish to override this behavior, simply write a custom action which
+    accomplishes deletion in your preferred manner -- for example, by calling
+    ``Model.delete()`` for each of the selected items.
+    
+    For more background on bulk deletion, see the documentation on :ref:`object
+    deletion <topics-db-queries-delete>`.
+
 Read on to find out how to add your own actions to this list.
 
 Writing actions


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to