Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#4649 closed (invalid)

"per-view cache" and decorator syntax.

Reported by: anonymous Owned by: Jacob
Component: Documentation Version: master
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


The docu about the "per-view cache" syntax seems to be wrong:

Malcolm Tredinnick writes about the "cache_page" here:

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

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

Change History (5)

comment:1 Changed 12 years ago by Gary Wilson <gary.wilson@…>

Resolution: invalid
Status: newclosed

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

comment:2 in reply to:  1 Changed 12 years ago by anonymous

Resolution: invalid
Status: closedreopened

Replying to Gary Wilson <>:

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 Changed 12 years ago by Malcolm Tredinnick

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 Changed 12 years ago by anonymous

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 Changed 12 years ago by Chris Beaven

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