#18080: @transaction.commit_manually plus context processors doesn't work well
-------------------------------+--------------------------------------
Reporter: jstpierre@… | Owner: nobody
Type: Uncategorized | Status: closed
Component: Uncategorized | Version: 1.3
Severity: Normal | Resolution: invalid
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+--------------------------------------
Comment (by akaariai):
My take is that the shown usage of commit_manually is a bug in the user
code - it does not take into account that the render_to_response might do
queries. The render_to_response is clearly part of the function, and thus
the commit_manually applies to it, too.
Disabling transactions as a side-effect of rendering under commit_manually
decorator is a big no-no in my opinion.
--
Ticket URL: <https://code.djangoproject.com/ticket/18080#comment:3>
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 post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
