port the bugfix for accepting and passing on the status and redirect …

…arguments to the redirect method

CHANGES.txt

@@ -1,11 +1,11 @@
 Changes
 =======
 
-1.14.0 (unreleased)
+1.13.4 (unreleased)
 -------------------
 
-- None yet.
-
+- Fix the url() method to accept the "status" and "trusted" arguments, passed
+  on to the redirect method on the response object.
 
 1.13.3 (2010-05-27)
 -------------------

setup.py

@@ -36,7 +36,7 @@ def read(*rnames):
 
 setup(
     name='grokcore.view',
-    version = '1.14.0dev',
+    version = '1.13.4dev',
     author='Grok Team',
     author_email='grok-dev@zope.org',
     url='http://grok.zope.org',

src/grokcore/view/components.py

@@ -45,8 +45,9 @@ def response(self):
     def body(self):
         return self.request.bodyStream.getCacheStream().read()
 
-    def redirect(self, url):
-        return self.request.response.redirect(url)
+    def redirect(self, url, status=None, trusted=False):
+        return self.request.response.redirect(
+            url, status=status, trusted=trusted)
 
     def url(self, obj=None, name=None, data=None):
         """Return string for the URL based on the obj and name.

src/grokcore/view/ftests/url/redirect.py

@@ -12,7 +12,18 @@
   >>> browser.open('http://localhost/manfred')
   >>> browser.url
   'http://localhost/manfred/another'
-  
+
+  >>> browser.open('http://localhost/manfred/trustedredirect')
+  >>> browser.url
+  'http://www.google.com/'
+
+  >>> browser.open('http://localhost/manfred/redirectwithstatus')
+  Traceback (most recent call last):
+  ...
+  HTTPError: HTTP Error 418: Unknown
+  >>> browser.url
+  'http://localhost/manfred/redirectwithstatus'
+
 """
 import grokcore.view as grok
 
@@ -23,7 +34,14 @@ class Index(grok.View):
     def render(self):
         self.redirect(self.url('another'))
 
+class TrustedRedirect(grok.View):
+    def render(self):
+        self.redirect('http://www.google.com/ncr', trusted=True)
+
+class RedirectWithStatus(grok.View):
+    def render(self):
+        self.redirect(self.url(), status=418)
+
 class Another(grok.View):
     def render(self):
         return "Another view"
-