undocumented context side-effect in {% ifchanged %}

[Antti Kaihola]

The {% ifchanged %} template tag sets the {{ ifchanged }} variable in its local context, in a way similar to how {% for %} sets the {{ forloop }} variable. This behavior is not documented. The value stored into the variable is a dictionary and doesn't seem to have any use elsewhere in Django. Maybe it's related to some removed functionality?

These quick Google code searches didn't turn up any uses of the context variable in indexed open source projects either:

• ​context['ifchanged']
• ​{{ ifchanged }}
• ​{{ifchanged}}

The following patch stops {% ifchanged %} from touching the context and adds a test for this.

[Antti Kaihola]

Patch: stop {% ifchanged %} from changing context, includes test

[Jacob]

(In [10438]) Fixed #10644: removed an undocumented, unused, and (as far as I can tell) unnecessary "feature" of the ifchanged tag. Thanks, akaihola.

[Jacob]

Milestone 1.1 deleted