diff options
| author | Tim Heap <tim@timheap.me> | 2015-03-31 15:37:17 +1100 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2015-04-01 12:26:57 -0400 |
| commit | 4ea1909d3c420ba1fbdbf7221cad518d43aef885 (patch) | |
| tree | 0415d0bb11470c5fe4681129662caf484362a1c8 | |
| parent | 48ddc66219deaac2aa5fa3d2cb0f420404d6b8ec (diff) | |
Fixed #24538 -- Allowed self in Jinja context
Rendering a Jinja template with self in the context threw an error.
While self is a reserved variable in Jinja, including self in the
context is not an error, so Django should respect that.
| -rw-r--r-- | django/template/backends/jinja2.py | 2 | ||||
| -rw-r--r-- | tests/template_backends/test_jinja2.py | 18 |
2 files changed, 19 insertions, 1 deletions
diff --git a/django/template/backends/jinja2.py b/django/template/backends/jinja2.py index b26cb391f2..0863773ddf 100644 --- a/django/template/backends/jinja2.py +++ b/django/template/backends/jinja2.py @@ -60,4 +60,4 @@ class Template(object): context['request'] = request context['csrf_input'] = csrf_input_lazy(request) context['csrf_token'] = csrf_token_lazy(request) - return self.template.render(**context) + return self.template.render(context) diff --git a/tests/template_backends/test_jinja2.py b/tests/template_backends/test_jinja2.py index 5f69669446..4c22c0958b 100644 --- a/tests/template_backends/test_jinja2.py +++ b/tests/template_backends/test_jinja2.py @@ -21,3 +21,21 @@ class Jinja2Tests(TemplateStringsTests): engine_class = Jinja2 backend_name = 'jinja2' options = {'keep_trailing_newline': True} + + def test_self_context(self): + """ + #24538 -- Using 'self' in the context should not throw errors + """ + engine = Jinja2({ + 'DIRS': [], + 'APP_DIRS': False, + 'NAME': 'django', + 'OPTIONS': {}, + }) + + # self will be overridden to be a TemplateReference, so the self + # variable will not come through. Attempting to use one though should + # not throw an error. + template = engine.from_string('hello {{ foo }}!') + content = template.render(context={'self': 'self', 'foo': 'world'}) + self.assertEqual(content, 'hello world!') |
