forked from orbit-oss/flask
addressed a few review concerns
This commit is contained in:
parent
8c054f04b8
commit
b31252db55
1 changed files with 6 additions and 6 deletions
12
flask/app.py
12
flask/app.py
|
|
@ -1080,7 +1080,7 @@ class Flask(_PackageBoundObject):
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _get_exc_class_and_code(exc_class_or_code):
|
def _get_exc_class_and_code(exc_class_or_code):
|
||||||
"""ensure that we register only exceptions as handler keys"""
|
"""Ensure that we register only exceptions as handler keys"""
|
||||||
if isinstance(exc_class_or_code, integer_types):
|
if isinstance(exc_class_or_code, integer_types):
|
||||||
exc_class = default_exceptions[exc_class_or_code]
|
exc_class = default_exceptions[exc_class_or_code]
|
||||||
else:
|
else:
|
||||||
|
|
@ -1412,15 +1412,15 @@ class Flask(_PackageBoundObject):
|
||||||
"""
|
"""
|
||||||
exc_class, code = self._get_exc_class_and_code(type(e))
|
exc_class, code = self._get_exc_class_and_code(type(e))
|
||||||
|
|
||||||
def find_superclass(d):
|
def find_superclass(handler_map):
|
||||||
if not d:
|
if not handler_map:
|
||||||
return None
|
return None
|
||||||
for superclass in exc_class.mro():
|
for superclass in exc_class.__mro__:
|
||||||
if superclass is BaseException:
|
if superclass is BaseException:
|
||||||
return None
|
return None
|
||||||
handler = d.get(superclass)
|
handler = handler_map.get(superclass)
|
||||||
if handler is not None:
|
if handler is not None:
|
||||||
d[exc_class] = handler # cache for next time exc_class is raised
|
handler_map[exc_class] = handler # cache for next time exc_class is raised
|
||||||
return handler
|
return handler
|
||||||
return None
|
return None
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue