#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)
follow-up: 2 comment:1 by , 17 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
comment:2 by , 17 years ago
Resolution: | invalid |
---|---|
Status: | closed → reopened |
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 , 17 years ago
Resolution: | → invalid |
---|---|
Status: | reopened → closed |
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 , 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 , 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.
The documentation is correct, it's the code that's wrong. See #1015.