#21461: Add pre_update and post_update signals
-------------------------------------+-------------------------------------
     Reporter:  loic84               |                    Owner:  loic84
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |               Resolution:
     Severity:  Normal               |             Triage Stage:  Accepted
     Keywords:                       |      Needs documentation:  0
    Has patch:  1                    |  Patch needs improvement:  1
  Needs tests:  0                    |                    UI/UX:  0
Easy pickings:  0                    |
-------------------------------------+-------------------------------------
Changes (by bendavis78):

 * cc: bendavis78 (added)


Comment:

 Yeah, you're right, using the dict is a messy way to solve the problem. I
 guess idea was to put the onus of "type" handling on the pre_save &
 post_save. Ideally the update() method would be agnostic regarding the
 payloads being passed between the signals. I think you've got the right
 idea with using `sender`, though I don't think the API should be
 prescribing those types (ie pks vs querysets).

 Not sure what the right answer is at this point -- maybe someone will come
 up with a more elegant approach.

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

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.e0e8eb37442bd03aad6ecba4f71d367b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to