Skip to content

Commit

Permalink
Fixed #11022: documented that the admin bulk delete action calls `Que…
Browse files Browse the repository at this point in the history
…rySet.delete()`, not `Model.delete()`. Thanks, Idan Gazit.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@10780 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information
jacob committed May 14, 2009
1 parent e53c9e7 commit 7b36d2a
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions docs/ref/contrib/admin/actions.txt
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,20 @@ models. For example, here's the user module from Django's built-in
: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
Expand Down

0 comments on commit 7b36d2a

Please sign in to comment.