Task #12207 (new)
Opened 10 years ago
Last modified 10 years ago
render_response and render_response_admin fail if given an HttpResponse — at Initial Version
Reported by: | drussell-x | Owned by: | web-team@… |
---|---|---|---|
Priority: | minor | Milestone: | Unscheduled |
Component: | Web | Version: | 5.0.1 |
Keywords: | n.a. | Cc: | |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | n.a. |
Sprint: | n.a. |
Description
Returning an HttpResponse? from the decorated function causes the decorator to bail out as it has already got a response.
This produces undesired behaviour in that (for example) none of the additional admin context is present when the template is rendered, causing it to render incorrectly.
I think that really the adding of context should be separated from the render_response decorator or I think that possibly a better way to add this context is with a template tag rather than a decorator as this can then be used only where required and doesn't have to be added to every admin function for maximum DRYness.