Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#4649 closed (invalid)

"per-view cache" and decorator syntax.

Reported by: anonymous Owned by: Jacob
Component: Documentation Version: dev
Severity: Keywords: cache_page, decorator
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The docu about the "per-view cache" syntax seems to be wrong: http://www.djangoproject.com/documentation/cache/#the-per-view-cache

Malcolm Tredinnick writes about the "cache_page" here: http://groups.google.com/group/django-users/msg/581a32c198e6ae07

So, "@cache_page" can't get parameters.

The alternative non-decorator syntax seems wrong, too. Look at 4421

Change History (5)

comment:1 by Gary Wilson <gary.wilson@…>, 17 years ago

Resolution: invalid
Status: newclosed

The documentation is correct, it's the code that's wrong. See #1015.

in reply to:  1 comment:2 by anonymous, 17 years ago

Resolution: invalid
Status: closedreopened

Replying to Gary Wilson <gary.wilson@gmail.com>:

The documentation is correct, it's the code that's wrong. See #1015.

IMHO until #1015 is fixed/checked in, the docu should display the way it works *now*. Many users run directly into the "Bug". That's not the best way...

comment:3 by Malcolm Tredinnick, 17 years ago

Resolution: invalid
Status: reopenedclosed

If we're going to fix something, fixing the real problem is the right thing to do. Gary is right; the real bug here is #1015. Please don't reopen this.

comment:4 by anonymous, 17 years ago

Oh. Nice philosophy. :( We have a "like it should run" Documentation, without a note that this syntax dosn't work until a bug is fixed.

comment:5 by Chris Beaven, 17 years ago

I kinda agree with anonymous. That ticket has a patch two months old, and since it's not that critical it could sit there for much longer. Well maybe not in this case, now it has got some attention.

Documentation should be kept up to date even if there is a bug - it's easy to update it again when the main ticket fixed.

Note: See TracTickets for help on using tickets.
Back to Top