Fixed #14477 -- Corrects several instances of "add" and "remove" m2m_changed signal actions not having been updated to "pre_add", "post_add", etc. Thanks to slink and andrews for reporting several instances of it, and andrews for the draft patch.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@14304 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
Gabriel Hurley 2010-10-20 22:14:26 +00:00
parent d1a632f9f9
commit a28823e364
1 changed files with 6 additions and 5 deletions

View File

@ -233,10 +233,11 @@ Arguments sent with this signal:
from the relation. from the relation.
``pk_set`` ``pk_set``
With the ``"add"`` and ``"remove"`` action, this is a list of For the ``pre_add``, ``post_add``, ``pre_remove`` and ``post_remove``
primary key values that have been added to or removed from the relation. actions, this is a list of primary key values that have been added to
or removed from the relation.
For the ``"clear"`` action, this is ``None``. For the ``pre_clear`` and ``post_clear`` actions, this is ``None``.
``using`` ``using``
The database alias being used. The database alias being used.
@ -270,7 +271,7 @@ the arguments sent to a :data:`m2m_changed` handler would be:
``instance`` ``p`` (the ``Pizza`` instance being modified) ``instance`` ``p`` (the ``Pizza`` instance being modified)
``action`` ``"add"`` ``action`` ``"pre_add"`` (followed by a separate signal with ``"post_add"``)
``reverse`` ``False`` (``Pizza`` contains the :class:`ManyToManyField`, ``reverse`` ``False`` (``Pizza`` contains the :class:`ManyToManyField`,
so this call modifies the forward relation) so this call modifies the forward relation)
@ -298,7 +299,7 @@ the arguments sent to a :data:`m2m_changed` handler would be:
``instance`` ``t`` (the ``Topping`` instance being modified) ``instance`` ``t`` (the ``Topping`` instance being modified)
``action`` ``"remove"`` ``action`` ``"pre_remove"`` (followed by a separate signal with ``"post_remove"``)
``reverse`` ``True`` (``Pizza`` contains the :class:`ManyToManyField`, ``reverse`` ``True`` (``Pizza`` contains the :class:`ManyToManyField`,
so this call modifies the reverse relation) so this call modifies the reverse relation)