Add redirect status codes (303, 307) to the set which prevent form re…

…ndering.

Fixes LP #1017884.

Index: CHANGES.txt
===================================================================
--- CHANGES.txt	(revision 126914)
+++ CHANGES.txt	(working copy)
@@ -5,6 +5,9 @@
 4.2.0 (unreleased)
 ==================
 
+- LP #1017884:  Add redirect status codes (303, 307) to the set which prevent
+  form rendering.
+
 - Replaced deprecated ``zope.component.adapts`` usage with equivalent
   ``zope.component.adapter`` decorator.

CHANGES.txt

@@ -5,6 +5,9 @@ Changes
 4.2.0 (unreleased)
 ==================
 
+- LP #1017884:  Add redirect status codes (303, 307) to the set which prevent
+  form rendering.
+
 - Replaced deprecated ``zope.component.adapts`` usage with equivalent
   ``zope.component.adapter`` decorator.
 

src/zope/formlib/form.py

@@ -810,7 +810,7 @@ def render(self):
 
     def __call__(self):
         self.update()
-        if self.request.response.getStatus() in [301, 302]:
+        if self.request.response.getStatus() in [301, 302, 303, 307]:
             # Avoid rendering if the action caused a redirect.
             result = self.form_result or ''
         else:

src/zope/formlib/tests/test_form.py

@@ -42,10 +42,10 @@ def _makeOne(self, context=None, request=None):
         return self._getTargetClass()(context, request)
 
     def test___call___does_not_render_on_redirects(self):
-        for status in (301, 302):
+        for status in (301, 302, 303, 307):
             request = self._makeRequest()
             request.response.setStatus(status)
-            def _raise(self, *args, **kw):
+            def _raise(*args, **kw):
                 self.fail("DON'T GO HERE")
             form = self._makeOne(request=request)
             form.form_fields = form.actions = ()