Ticket #16842: django13-redirects.patch

django/views/generic/simple.py

@@ -48 +48 @@
     from the request is appended to the URL.
 
     """
-    args = request.META["QUERY_STRING"]
-    if args and query_string and url is not None:
-        url = "%s?%s" % (url, args)
+    if url is not None:
+        url = url % kwargs
+        args = request.META["QUERY_STRING"]
+        if args and query_string:
+            url = "%s?%s" % (url, args)
 
-    if url is not None:
         klass = permanent and HttpResponsePermanentRedirect or HttpResponseRedirect
-        return klass(url % kwargs)
+        return klass(url)
     else:
         logger.warning('Gone: %s' % request.path,
                     extra={

django/views/generic/base.py

@@ -138 +138 @@
         are provided as kwargs to this method.
         """
         if self.url:
+            url = self.url % kwargs
             args = self.request.META["QUERY_STRING"]
             if args and self.query_string:
-                url = "%s?%s" % (self.url, args)
-            else:
-                url = self.url
-            return url % kwargs
+                url = "%s?%s" % (url, args)
+            return url
         else:
             return None