#29557: add on_commit decorator
-------------------------------------+-------------------------------------
Reporter: obayemi | Owner: nobody
Type: New | Status: new
feature |
Component: | Version: master
Uncategorized | Keywords: transaction
Severity: Normal | on_commit decorator
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
I recently had to add on_commit hooks with lambdas to some functions
(mainly email notifications in post_save signals handler)
That was a lot of boilerplate so I wrote a simple decorator for making a
function call only trigger on transaction commit
{{{
def call_on_commit(f):
"""
only call the decorated function at transaction commit
warning the return value will be ignored
"""
def handle(*args, **kwargs):
transaction.on_commit(lambda: f(*args, **kwargs))
return handle
}}}
leading to
{{{
@call_on_commit
def send_message(user, context):
# send message
}}}
instead of
{{{
def send_message(user, context):
def _send_message():
# send message
on_commit(do_stuff)
}}}
I made a PR on github to implement this feature
https://github.com/django/django/pull/10167
--
Ticket URL: <https://code.djangoproject.com/ticket/29557>
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/050.2e5916aa653c4f85994d445150726066%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
