Log message for revision 68584:
- Collector #1944: HTTPRequest.resolve_url has error in raising errors
Changed:
U Zope/branches/2.9/doc/CHANGES.txt
U Zope/branches/2.9/lib/python/ZPublisher/HTTPRequest.py
U Zope/branches/2.9/lib/python/ZPublisher/tests/testHTTPRequest.py
-=-
Modified: Zope/branches/2.9/doc/CHANGES.txt
===================================================================
--- Zope/branches/2.9/doc/CHANGES.txt 2006-06-12 05:19:47 UTC (rev 68583)
+++ Zope/branches/2.9/doc/CHANGES.txt 2006-06-12 05:20:44 UTC (rev 68584)
@@ -27,6 +27,8 @@
- Collector #2073: fixed misbehaviour of OFS.Owned.changeOwnership
+ - Collector #1944: HTTPRequest.resolve_url has error in raising errors
+
Zope 2.9.3 (2006/05/13)
Bugs fixed
Modified: Zope/branches/2.9/lib/python/ZPublisher/HTTPRequest.py
===================================================================
--- Zope/branches/2.9/lib/python/ZPublisher/HTTPRequest.py 2006-06-12
05:19:47 UTC (rev 68583)
+++ Zope/branches/2.9/lib/python/ZPublisher/HTTPRequest.py 2006-06-12
05:20:44 UTC (rev 68584)
@@ -1039,7 +1039,7 @@
except: rsp.exception()
if object is None:
req.close()
- raise rsp.errmsg, sys.exc_info()[1]
+ raise sys.exc_info()[0], rsp.errmsg
# The traversal machinery may return a "default object"
# like an index_html document. This is not appropriate
Modified: Zope/branches/2.9/lib/python/ZPublisher/tests/testHTTPRequest.py
===================================================================
--- Zope/branches/2.9/lib/python/ZPublisher/tests/testHTTPRequest.py
2006-06-12 05:19:47 UTC (rev 68583)
+++ Zope/branches/2.9/lib/python/ZPublisher/tests/testHTTPRequest.py
2006-06-12 05:20:44 UTC (rev 68584)
@@ -700,7 +700,20 @@
self.assertEqual(f.next(),'test\n')
f.seek(0)
self.assertEqual(f.xreadlines(),f)
+
+ def testResolveUrl(self):
+ # Check that ResolveUrl really raises the same error
+ # it received from ZPublisher.BaseRequest.traverse
+ # collector entry 1944
+ from ZPublisher.HTTPRequest import HTTPRequest
+ from zExceptions import NotFound
+ env = TEST_ENVIRON.copy()
+ req = HTTPRequest(None, env, None)
+ req['PARENTS'] = ['Nobody', 'cares', 'here']
+ testmethod = req.resolve_url
+ self.assertRaises(NotFound, testmethod,
'http://localhost/does_not_exist')
+
def test_suite():
suite = unittest.TestSuite()
suite.addTest(unittest.makeSuite(AuthCredentialsTestsa, 'test'))
_______________________________________________
Zope-Checkins maillist - Zope-Checkins@zope.org
http://mail.zope.org/mailman/listinfo/zope-checkins
