Opened 12 days ago

Last modified 8 days ago

#35401 assigned Cleanup/optimization

Document the `conditional_page` decorator

Reported by: Keryn Knight Owned by: Lufafa Joshua
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The GZip Middleware documentation makes mention of the gzip_page decorator, but the Conditional GET middleware doesn't mention there's an equivalent conditional_page decorator, likely ecause there's no mention of that in the HTTP decorators or in the Conditional View Processing topics.

As far as I can tell, the named decorator has been around since 0.9x and it's probably just accidentally not described anywhere, when everything else in that module is.

See also: #6181 and this blog post describing the intent, from James Bennett back in 2007 (yay for cool URIs which haven't suffered link rot!)

Change History (2)

comment:1 by Sarah Boyce, 12 days ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization

Good find, thank you!
It might also make sense for the ​CsrfViewMiddleware to similarly mention csrf_protect.

comment:2 by Lufafa Joshua, 8 days ago

Owner: changed from nobody to Lufafa Joshua
Status: newassigned
Note: See TracTickets for help on using tickets.
Back to Top