Opened 7 years ago
Last modified 7 years ago
#28825 closed Uncategorized
Templated email with {% extends parent_name %} doesn't re-resolve the base template — at Version 3
Reported by: | Andrew | Owned by: | nobody |
---|---|---|---|
Component: | Template system | Version: | 1.11 |
Severity: | Normal | Keywords: | |
Cc: | awbacker@… | Triage Stage: | Unreviewed |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In django templating you can use {% extends parent_name %}
to indicate that the template is derived from another. We use this feature along with django-templated-email to have different base email templates for white-label purposes.
The issue appears to be that {% extends parent_name %}
does not get re-evaluated when the variable changes, eg. from email/app1.email
to email/app2.email
.
- The Engine uses
django.template.loaders.cached.Loader
- The first time the template is loaded, the parent template is loaded based on
parent_name
is loaded - Subsequent emails load the template, but do not re-evaluate the
extends
If this is as-designed, how would one go about clearing out that cache? When setting up my test case I call the code below, but it doesn't appear to clear the cache, even though my print()
debugging shows its the same object (<django.template.loaders.cached.Loader object at 0x113e99828>
_
for template in engines.all(): for loader in template.engine.template_loaders: loader.reset() for x in loader.loaders: x.reset()
Change History (3)
comment:1 by , 7 years ago
Description: | modified (diff) |
---|
comment:2 by , 7 years ago
comment:3 by , 7 years ago
Description: | modified (diff) |
---|
It would be best to know, going in, if {extends} is actually *expected* to act like a meta-{%include%} or if the template engine actually combines the two.
I'll work on a better test case or sample project, but it'll probably be a few days. I've not had any cause to ever pull down the source before, and certainly don't know how write proper tests for it, i'm just investigating templates for the first time now.
- we use the
django-templated-email
package to wrap doing it, but it ends up in aDjangoTemplates
processor.
The Base Email (object_created.email
). The base template includes {%include%}
directives which seem to be re-evaluated each time, the opposite of extends. My working theory is that somehow the two just get combined on the *first run* and ever-after are the same template somehow (even though I can't find the cache, or figure out who's doing it)
{% extends base_template %}{% load i18n %} {% block subject_line %}{% blocktrans %}New Something Happened{% endblocktrans %}{% endblock %} {% block html_content %} <h2 style="font-size:15px;">Dear {{ some_person }},</h2>
In the two tests I set the base_template to something different, and also try to reset()
all the loaders including the children of the cached.Loader
, to no affect.
context_goog = { base_template="email/google_template.email", some_person="Wonko the Sane" } context_amzn = { base_template="email/amazon_template.email", some_person="Wonko the Sane" } send_templated_email("object_created.email", context_goog) ... django.template.loader.get_template('email/object_created.email') uses engine: <django.template.backends.django.DjangoTemplates object at 0x108573668> DjangoTemplates.get_template('email/object_created.email') <django.template.engine.Engine object at 0x108d7a2e8> .find_template('email/object_created.email') loader=<django.template.loaders.cached.Loader object at 0x10b957ef0> .find_template('email/google_template.email') loader=<django.template.loaders.cached.Loader object at 0x10b957ef0> .find_template('email/partials/contents-footer.html') loader=<django.template.loaders.cached.Loader object at 0x10b957ef0> send_templated_email("object_created.email", context_amzn) ... django.template.loader.get_template('email/object_created.email') uses engine: <django.template.backends.django.DjangoTemplates object at 0x108573668> DjangoTemplates.get_template('email/object_created.email') <django.template.engine.Engine object at 0x108d7a2e8> .find_template('email/object_created.email') loader=<django.template.loaders.cached.Loader object at 0x10b957ef0> .find_template('email/partials/contents-footer.html') loader=<django.template.loaders.cached.Loader object at 0x10b957ef0>
I hope that printf_trace stuff helps in evaluating what is going on. It might take me a few days to formulate a test case (if I can) or make a new project (if I can't) demonstrating what is going on.
Please include more explicit steps to reproduce (that don't rely on a third-party library, to rule out an issue there). Based on your report, I tried the following change in Django's test suite but don't see a failure.
tests/template_tests/syntax_tests/test_extends.py