Ticket #3349: 3349_r11897.diff

django/conf/__init__.py

@@ -11 +11 @@
 import time     # Needed for Windows
 
 from django.conf import global_settings
+from django.core.exceptions import wrap_and_raise
 from django.utils.functional import LazyObject
 from django.utils import importlib
 
@@ -72 +73 @@
         try:
             mod = importlib.import_module(self.SETTINGS_MODULE)
         except ImportError, e:
-            raise ImportError, "Could not import settings '%s' (Is it on sys.path? Does it have syntax errors?): %s" % (self.SETTINGS_MODULE, e)
+            wrap_and_raise(ImportError("Could not import settings '%s' (Is it on sys.path? Does it have syntax errors?): %s" % (self.SETTINGS_MODULE, e)))
 
         # Settings that should be converted into tuples if they're mistakenly entered
         # as strings.

django/core/urlresolvers.py

@@ -11 +11 @@
 
 from django.http import Http404
 from django.conf import settings
-from django.core.exceptions import ImproperlyConfigured, ViewDoesNotExist
+from django.core.exceptions import ImproperlyConfigured, ViewDoesNotExist, wrap_and_raise
 from django.utils.datastructures import MultiValueDict
 from django.utils.encoding import iri_to_uri, force_unicode, smart_str
 from django.utils.functional import memoize
@@ -135 +135 @@
             self._callback = get_callable(self._callback_str)
         except ImportError, e:
             mod_name, _ = get_mod_func(self._callback_str)
-            raise ViewDoesNotExist, "Could not import %s. Error was: %s" % (mod_name, str(e))
+            wrap_and_raise(ViewDoesNotExist("Could not import %s. Error was: %s" % (mod_name, str(e))))
         except AttributeError, e:
             mod_name, func_name = get_mod_func(self._callback_str)
-            raise ViewDoesNotExist, "Tried %s in module %s. Error was: %s" % (func_name, mod_name, str(e))
+            wrap_and_raise(ViewDoesNotExist("Tried %s in module %s. Error was: %s" % (func_name, mod_name, str(e))))
         return self._callback
     callback = property(_get_callback)
 

django/core/handlers/base.py

@@ -2 +2 @@
 
 from django import http
 from django.core import signals
+from django.core.exceptions import wrap_and_raise
 from django.utils.encoding import force_unicode
 from django.utils.importlib import import_module
 
@@ -34 +35 @@
             try:
                 dot = middleware_path.rindex('.')
             except ValueError:
-                raise exceptions.ImproperlyConfigured, '%s isn\'t a middleware module' % middleware_path
+                wrap_and_raise(exceptions.ImproperlyConfigured('%s isn\'t a middleware module' % middleware_path))
             mw_module, mw_classname = middleware_path[:dot], middleware_path[dot+1:]
             try:
                 mod = import_module(mw_module)

django/core/exceptions.py

@@ -35 +35 @@
 class ValidationError(Exception):
     """An error while validating data."""
     pass
+
+def wrap_and_raise(new_exception):
+    """
+    Call from an except: block to raise a new exception, while keeping
+    the traceback info from the source exception.
+    
+    new_exception is be the Exception (or subclass) instance to raise.
+    """
+    import sys
+    exc_class, exc, tb = sys.exc_info()
+
+    if issubclass(exc_class, Exception):
+        raise new_exception.__class__, new_exception, tb
+    else:
+        raise new_exception
+

django/contrib/auth/__init__.py

@@ -1 +1 @@
 import datetime
 from warnings import warn
-from django.core.exceptions import ImproperlyConfigured
+from django.core.exceptions import ImproperlyConfigured, wrap_and_raise
 from django.utils.importlib import import_module
 
 SESSION_KEY = '_auth_user_id'
@@ -13 +13 @@
     try:
         mod = import_module(module)
     except ImportError, e:
-        raise ImproperlyConfigured, 'Error importing authentication backend %s: "%s"' % (module, e)
+        wrap_and_raise(ImproperlyConfigured('Error importing authentication backend %s: "%s"' % (module, e)))
     except ValueError, e:
         raise ImproperlyConfigured, 'Error importing authentication backends. Is AUTHENTICATION_BACKENDS a correctly defined list or tuple?'
     try:

django/template/__init__.py

@@ -52 +52 @@
 from inspect import getargspec
 
 from django.conf import settings
+from django.core.exceptions import wrap_and_raise
 from django.template.context import Context, RequestContext, ContextPopException
 from django.utils.importlib import import_module
 from django.utils.itercompat import is_iterable
@@ -970 +971 @@
         try:
             mod = import_module(module_name)
         except ImportError, e:
-            raise InvalidTemplateLibrary("Could not load template library from %s, %s" % (module_name, e))
+            wrap_and_raise(InvalidTemplateLibrary("Could not load template library from %s, %s" % (module_name, e)))
         try:
             lib = mod.register
             libraries[module_name] = lib

django/template/loaders/app_directories.py

@@ -7 +7 @@
 import sys
 
 from django.conf import settings
-from django.core.exceptions import ImproperlyConfigured
+from django.core.exceptions import ImproperlyConfigured, wrap_and_raise
 from django.template import TemplateDoesNotExist
 from django.template.loader import BaseLoader
 from django.utils._os import safe_join
@@ -20 +20 @@
     try:
         mod = import_module(app)
     except ImportError, e:
-        raise ImproperlyConfigured, 'ImportError %s: %s' % (app, e.args[0])
+        wrap_and_raise(ImproperlyConfigured('ImportError %s: %s' % (app, e.args[0])))
     template_dir = os.path.join(os.path.dirname(mod.__file__), 'templates')
     if os.path.isdir(template_dir):
         app_template_dirs.append(template_dir.decode(fs_encoding))

django/template/defaulttags.py

@@ -13 +13 @@
 from django.template import get_library, Library, InvalidTemplateLibrary
 from django.template.smartif import IfParser, Literal
 from django.conf import settings
+from django.core.exceptions import wrap_and_raise
 from django.utils.encoding import smart_str, smart_unicode
 from django.utils.itercompat import groupby
 from django.utils.safestring import mark_safe
@@ -924 +925 @@
             lib = get_library("django.templatetags.%s" % taglib)
             parser.add_library(lib)
         except InvalidTemplateLibrary, e:
-            raise TemplateSyntaxError("'%s' is not a valid tag library: %s" %
-                                      (taglib, e))
+            wrap_and_raise(TemplateSyntaxError("'%s' is not a valid tag library: %s" %
+                                      (taglib, e)))
     return LoadNode()
 load = register.tag(load)
 

django/template/context.py

@@ -1 @@
-from django.core.exceptions import ImproperlyConfigured
+from django.core.exceptions import ImproperlyConfigured, wrap_and_raise
 from django.utils.importlib import import_module
 
 # Cache of actual callables.
@@ -123 +123 @@
             try:
                 mod = import_module(module)
             except ImportError, e:
-                raise ImproperlyConfigured('Error importing request processor module %s: "%s"' % (module, e))
+                wrap_and_raise(ImproperlyConfigured('Error importing request processor module %s: "%s"' % (module, e)))
             try:
                 func = getattr(mod, attr)
             except AttributeError:

django/template/loader.py

@@ -20 +20 @@
 # Python eggs) sets is_usable to False if the "pkg_resources" module isn't
 # installed, because pkg_resources is necessary to read eggs.
 
-from django.core.exceptions import ImproperlyConfigured
+from django.core.exceptions import ImproperlyConfigured, wrap_and_raise
 from django.template import Origin, Template, Context, TemplateDoesNotExist, add_to_builtins
 from django.utils.importlib import import_module
 from django.conf import settings
@@ -81 +81 @@
         try:
             mod = import_module(module)
         except ImportError:
-            raise ImproperlyConfigured('Error importing template source loader %s: "%s"' % (loader, e))
+            wrap_and_raise(ImproperlyConfigured('Error importing template source loader %s: "%s"' % (loader, e)))
         try:
             TemplateLoader = getattr(mod, attr)
         except AttributeError, e: