Opened 7 years ago

Closed 7 years ago

#15756 closed Bug (invalid)

Template regression with property lookups

Reported by: Keryn Knight <keryn@…> Owned by: nobody
Component: Template system Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


Whilst upgrading from 1.2.5 to 1.3, a colleague discovered a somewhat weird and problematic variable lookup issue in the template system. I've attempted to compress it down into a replicable test. I've tested this on Windows and Mac, both running Python 2.6 and Django 1.2.5 & 1.3

from django.template import Context, Template

class A(object):
    name = 'A'
    def __init__(self, anything):
        ''' this is the crux of the problem? '''

class B(A):
    name = u'B'

class C(B):
    name = u'C'

out = Template('{% for c in class_list %}{{ }}{% endfor %}')

# this works under 1.2.5, but not 1.3
fail_list = (A, B, C)
# this works under both.
pass_list = (A(1), B(2), C(3))

# using classes; expected historic output u'ABC', in 1.3 returns u''
out.render(Context({'class_list': fail_list}))
# using instantiated objects, always returns u'ABC'
out.render(Context({'class_list': pass_list}))

Interestingly, if the init method of the class doesn't take additional parameters (ie: remove the anything kwarg), it seems to be fine. I have absolutely no clue about the inner workings of the template compiler, but I'm presuming it does some sort of clever variable resolution that's changed in this edge case.

Symptomatic of my ignorance of the templating voodoo is that I don't really know how to summarise the problem, so the title may be better expressed, and I may have missed pre-existing tickets for the same thing.

Change History (1)

comment:1 Changed 7 years ago by Łukasz Rekucki

Resolution: invalid
Status: newclosed

This is actually a fix of previous inconsistent behaviour. See Django 1.3 release notes:

PS. A similar "problem" involving classes was reported in #15660.

Note: See TracTickets for help on using tickets.
Back to Top