Ticket #8809: fix_ticket_8809_urlsresolvers_error_message.diff

django/core/urlresolvers.py

@@ -8 +8 @@
 """
 
 import re
+import os
 from threading import local
-
+from imp import find_module
 from django.http import Http404
 from django.conf import settings
 from django.core.exceptions import ImproperlyConfigured, ViewDoesNotExist
@@ -72 +73 @@
     # Don't make this raise an error when used in a template.
     silent_variable_failure = True
 
+def view_not_found_error(mod_name, view):
+    # If a view is not found, this function searches for the urls.py file and line from
+    # which the error originated.
+    
+    urls_file_path = find_module(mod_name)[1] + '/urls.py' # Path of the urls.py file where the error is
+    view_not_found = ''
+    if os.path.isfile(urls_file_path):
+        f = open(urls_file_path)
+        for n,line in enumerate(f):
+            if line.find(view) != -1:
+                view_not_found = "\nError occurred in %s at line %d:\n\"%s\"" % (urls_file_path,n,line)   
+    return view_not_found
+
+
 def get_callable(lookup_view, can_fail=False):
     """
     Convert a string version of a function name to the callable object.
@@ -164 +179 @@
             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)))
+            raise ViewDoesNotExist("Could not import %s. Error was: %s.\n%s" % (mod_name, str(e),view_not_found_error(mod_name, self._callback_str)))
         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)))
+            raise ViewDoesNotExist("Tried %s in module %s. Error was: %s.\n%s" % (func_name, mod_name, str(e),view_not_found_error(mod_name, self._callback_str)))
         return self._callback
     callback = property(_get_callback)