#9154: Huge improvements in templates rendering
--------------------------------------+-------------------------------------
Reporter: msaelices | Owner: msaelices
Status: new | Milestone:
Component: Template system | Version: SVN
Resolution: | Keywords:
Stage: Unreviewed | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
--------------------------------------+-------------------------------------
Comment (by mtredinnick):
It feels like this sort of thing is better done as a template loader that
accepts any other template loader and caches the result (basically a
wrapper around other loaders). That saves having to make the same kind of
internal modifications in every case. I'd prefer that type of approach,
since it's really just caching an compiled template object, however that
happened to have been loaded.
In addition to the things Chris mentioned, if you think there's some
inheritance problem that is unrelated to this caching feature addition,
please make that a separate ticket. If it's only relevant with this
enhancement, then one patch is appropriate, rather than multiple patches.
But we need to keep the bug fixes and the feature additions separate.
One question I always have with this sort of thing is what the behavioural
changes are going to be (since they need to be documented). If a template
contains a tag that returns a random number each time the template is
rendered, how will that be changed with this sort of thing? Will the tag
be re-evaluated each time? That's going to be necessary to mention in the
documentation changes that haven't been written yet.
Somebody (you and SmileyChris, basically) have to decide which ticket out
of this one and #6262 to close as a dupe (it should probably be this one).
Having two tickets addressing the same thing doesn't make things any
easier for us.
--
Ticket URL: <http://code.djangoproject.com/ticket/9154#comment:11>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
