From be68c0bf637d9b28780753575b889946a734ca0e Mon Sep 17 00:00:00 2001 From: John D'Ambrosio Date: Wed, 22 Mar 2017 12:06:01 -0400 Subject: [PATCH] Fixed #28071 -- Fixed {% extends %} origin history. --- AUTHORS | 1 + django/template/loader_tags.py | 2 +- tests/template_tests/test_extends.py | 21 +++++++++++++++++++++ 3 files changed, 23 insertions(+), 1 deletion(-) diff --git a/AUTHORS b/AUTHORS index fbf66bacbf..8a7e34d1e1 100644 --- a/AUTHORS +++ b/AUTHORS @@ -390,6 +390,7 @@ answer newbie questions, and generally made Django that much better: Johann Queuniet john@calixto.net John D'Agostino + John D'Ambrosio John Huddleston John Moses John Paulett diff --git a/django/template/loader_tags.py b/django/template/loader_tags.py index 69e7ce471d..6cc8dfc8e3 100644 --- a/django/template/loader_tags.py +++ b/django/template/loader_tags.py @@ -103,7 +103,7 @@ class ExtendsNode(Node): without extending the same template twice. """ history = context.render_context.setdefault( - self.context_key, [context.template.origin], + self.context_key, [self.origin], ) template, origin = context.template.engine.find_template( template_name, skip=history, diff --git a/tests/template_tests/test_extends.py b/tests/template_tests/test_extends.py index 270160a82a..0950340f52 100644 --- a/tests/template_tests/test_extends.py +++ b/tests/template_tests/test_extends.py @@ -117,3 +117,24 @@ class ExtendsBehaviorTests(SimpleTestCase): template = engine.get_template('base.html') output = template.render(Context({})) self.assertEqual(output.strip(), 'loader2 loader1') + + def test_block_override_in_extended_included_template(self): + """ + ExtendsNode.find_template() initializes history with self.origin + (#28071). + """ + engine = Engine( + loaders=[ + ['django.template.loaders.locmem.Loader', { + 'base.html': "{% extends 'base.html' %}{% block base %}{{ block.super }}2{% endblock %}", + 'included.html': + "{% extends 'included.html' %}{% block included %}{{ block.super }}B{% endblock %}", + }], + ['django.template.loaders.locmem.Loader', { + 'base.html': "{% block base %}1{% endblock %}{% include 'included.html' %}", + 'included.html': "{% block included %}A{% endblock %}", + }], + ], + ) + template = engine.get_template('base.html') + self.assertEqual(template.render(Context({})), '12AB')