Opened 17 years ago
Closed 17 years ago
#6201 closed (fixed)
Improved cache tag to allow timeout to be a template variable
| Reported by: | Owned by: | ||
|---|---|---|---|
| Component: | Template system | Version: | dev |
| Severity: | Keywords: | template caching | |
| Cc: | stefan@… | Triage Stage: | Ready for checkin |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
from ticket:1065
Somehow the idea of having cache time spread all over the place seems a bit cluttering especially if you use the tag more frequently throughout templates. Having the timeout in a single variable is helpful.
The patch allows the timeout to be a template variable and also adds some extra tests.
Attachments (2)
Change History (7)
by , 17 years ago
| Attachment: | cache_tag.diff added |
|---|
comment:1 by , 17 years ago
| Triage Stage: | Unreviewed → Design decision needed |
|---|
comment:2 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
| Triage Stage: | Design decision needed → Ready for checkin |
I don't think a design decision is necessary. This is useful functionality and it's backwards compatible.
comment:3 by , 17 years ago
| Owner: | removed |
|---|---|
| Status: | assigned → new |
comment:4 by , 17 years ago
| Cc: | added |
|---|
I really think this is necessary, I was thinking of doing this if I hadn't found this ticket.
Please check it in ;)
comment:5 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Improved tag to accept time as variable