[Django] #27380: Add the 'raw' argument with the 'm2m_changed' signal

21 views
Skip to first unread message

Django

unread,
Oct 24, 2016, 3:29:38 PM10/24/16
to django-...@googlegroups.com
#27380: Add the 'raw' argument with the 'm2m_changed' signal
----------------------------------------------+--------------------
Reporter: Élie Bouttier | Owner: nobody
Type: New feature | Status: new
Component: Database layer (models, ORM) | Version: 1.10
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
----------------------------------------------+--------------------
The 'raw' argument is sent with pre_save and post_save signals in order to
detect fixture loading and avoid un-needed operations in this case.
Unfortunatly, this argument is not sent with m2m_changed signal !
I find a work-around on stack-overflow for tests which consist in
disconnecting the m2m_signal in the setUpClass.
However, this is not applicable for loaddata management command.

Please send the 'raw' argument with the m2m_changed signal, this is
needed!

--
Ticket URL: <https://code.djangoproject.com/ticket/27380>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Oct 24, 2016, 3:41:58 PM10/24/16
to django-...@googlegroups.com
#27380: Add the 'raw' argument with the 'm2m_changed' signal
-------------------------------------+-------------------------------------

Reporter: Élie Bouttier | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: 1.10
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* needs_better_patch: => 0
* needs_tests: => 0
* needs_docs: => 0


Comment:

This was also proposed in #17583 which was marked as a duplicate of #8399
but I don't see an elegant solution in that ticket, so maybe it's time to
accept this.

--
Ticket URL: <https://code.djangoproject.com/ticket/27380#comment:1>

Django

unread,
Oct 26, 2016, 12:27:46 PM10/26/16
to django-...@googlegroups.com
#27380: Add the 'raw' argument with the 'm2m_changed' signal
-------------------------------------+-------------------------------------

Reporter: Élie Bouttier | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: 1.10
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/27380#comment:2>

Django

unread,
Aug 1, 2017, 8:04:49 AM8/1/17
to django-...@googlegroups.com
#27380: Add the 'raw' argument with the 'm2m_changed' signal
-------------------------------------+-------------------------------------

Reporter: Élie Bouttier | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: 1.10
(models, ORM) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by direx):

* cc: direx (added)


--
Ticket URL: <https://code.djangoproject.com/ticket/27380#comment:3>

Reply all
Reply to author
Forward
0 new messages