Improve template cache tag documentation
|Reported by:||julianb||Owned by:||nobody|
|Cc:||florian+django@…||Triage Stage:||Ready for checkin|
|Has patch:||no||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
For me it was not clear that the fragment name cannot be a variable. I just found out by wondering about errors and having a quick look into Django's code. It should be made more clear that the second argument will not be resolved even though all the others will be (even the cache time gets resolved).
"It takes at least two arguments: the cache timeout, in seconds, and the name to give the cache fragment. For example:"
should at least be something like
"It takes at least two arguments: the cache timeout, in seconds, and the name to give the cache fragment. The name will be taken as is, do not use a variable. For example:"
Change History (6)
comment:1 Changed 4 years ago by jezdez
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
- Triage Stage changed from Unreviewed to Accepted
comment:5 Changed 4 years ago by Julian Bez <web-spy@…>
- Resolution set to fixed
- Status changed from new to closed