Commit 254445ba authored by 's avatar

simplified code: The 'can_raise' check was broken and useless.

- *Broken* because re-raising doesn't work with more than one required argument. "Needs to be able to take more than two arguments" was a wrong assumption.
- *Useless* because the code below never tries to re-raise rendered errors. That's the reason why the wrong check never caused trouble.
parent 8e7f7d56
...@@ -20,7 +20,6 @@ item types. ...@@ -20,7 +20,6 @@ item types.
$Id$ $Id$
""" """
import inspect
import marshal import marshal
import re import re
import sys import sys
...@@ -238,35 +237,10 @@ class Item(Base, ...@@ -238,35 +237,10 @@ class Item(Base,
handle_errors = getattr(getattr(REQUEST, 'RESPONSE', None), handle_errors = getattr(getattr(REQUEST, 'RESPONSE', None),
'handle_errors', False) 'handle_errors', False)
# Can we re-raise the exception with a rendered-to-HTML if not handle_errors:
# exception value? To be able to do so, the exception # If we have been asked not to handle errors don't even bother
# constructor needs to be able to take more than two # with transforming the exception into HTML. Just re-raise the
# arguments (some Zope exceptions can't). # original exception right away.
can_raise = False
ctor = getattr(error_type, '__init__', None)
if inspect.ismethoddescriptor(ctor):
# If it's a method descriptor, it means we've got a
# base ``__init__`` method that was not overriden,
# likely from the base ``Exception`` class.
can_raise = True
else:
if inspect.ismethod(ctor):
ctor = getattr(ctor, 'im_func', None)
if inspect.isbuiltin(ctor):
# In Python 2.4, the ``__init__`` method of the
# base ``Exception`` class is a ``builtin
# method``.
can_raise = True
elif ctor is not None and inspect.isfunction(ctor):
can_raise = (
len(inspect.getargspec(error_type.__init__)[0]) > 2)
if not (can_raise and handle_errors):
# If we have been asked not to handle errors and we
# can't re-raise a transformed exception don't even
# bother with transforming the exception into
# HTML. Just re-raise the original exception right
# away.
raise error_type, error_value, tb raise error_type, error_value, tb
try: try:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment