Ticket #19890: 19890-fix-double-ifchanged-rendering.patch

django/template/defaulttags.py

@@ -216 +216 @@
         if self not in state_frame:
             state_frame[self] = None
 
+        nodelist_true_output = None
         try:
             if self._varlist:
                 # Consider multiple parameters.  This automatically behaves
@@ -223 +224 @@
                 compare_to = [var.resolve(context, True) for var in self._varlist]
             else:
                 # The "{% ifchanged %}" syntax (without any variables) compares the rendered output.
-                compare_to = self.nodelist_true.render(context)
+                compare_to = nodelist_true_output = self.nodelist_true.render(context)
         except VariableDoesNotExist:
             compare_to = None
 
         if compare_to != state_frame[self]:
             state_frame[self] = compare_to
-            return self.nodelist_true.render(context)
+            return nodelist_true_output or self.nodelist_true.render(context)  # render true block if not already rendered
         elif self.nodelist_false:
             return self.nodelist_false.render(context)
         return ''

tests/regressiontests/templates/tests.py

@@ -426 +426 @@
         output1 = template.render(Context({'foo': range(3), 'gen': gen()}))
         self.assertEqual(output1, '[0,1,2,3]', 'Expected [0,1,2,3] in first template, got {0}'.format(output1))
 
+    def test_ifchanged_render_once(self):
+        template = Template('[0{% for x in foo %},{% ifchanged %}{{ gen.next }}{% endifchanged %}{% endfor %}]')
+        def gen():
+            for i in xrange(1,10):
+                yield i
+
+        output = template.render(Context({'foo': range(3), 'gen': gen()}))
+        self.assertEqual(output, '[0,1,2,3]')
+
     def test_templates(self):
         template_tests = self.get_template_tests()
         filter_tests = filters.get_filter_tests()