Ticket #9005: 9005-with-tests.diff

django/core/management/__init__.py

@@ -323 +323 @@
     else:
         os.environ['DJANGO_SETTINGS_MODULE'] = '%s.%s' % (project_name, settings_name)
 
-    # Import the project module. We add the parent directory to PYTHONPATH to 
+    # Import the project module. We add the parent directory to PYTHONPATH to
     # avoid some of the path errors new users can have.
     sys.path.append(os.path.join(project_directory, os.pardir))
     project_module = import_module(project_name)

django/template/defaulttags.py

@@ -371 +371 @@
         try:
             url = reverse(self.view_name, args=args, kwargs=kwargs)
         except NoReverseMatch:
-            project_name = settings.SETTINGS_MODULE.split('.')[0]
-            try:
-                url = reverse(project_name + '.' + self.view_name,
-                              args=args, kwargs=kwargs)
-            except NoReverseMatch:
+            if settings.SETTINGS_MODULE is None:
+                # SETTINGS_MODULE is not set, so we can't guess the "main" app.
                 if self.asvar is None:
                     raise
+            else:
+                project_name = settings.SETTINGS_MODULE.split('.')[0]
+                try:
+                    url = reverse(project_name + '.' + self.view_name,
+                              args=args, kwargs=kwargs)
+                except NoReverseMatch:
+                    if self.asvar is None:
+                        raise
 
         if self.asvar:
             context[self.asvar] = url

tests/regressiontests/templates/tests.py

@@ -152 +152 @@
         split = token.split_contents()
         self.assertEqual(split, ["sometag", '_("Page not found")', 'value|yesno:_("yes,no")'])
 
+    def test_url_reverse_no_settings_module(self):
+        #Regression test for #9005
+        from django.template import Template, Context, TemplateSyntaxError
+        old_settings_module = settings.SETTINGS_MODULE
+        settings.SETTINGS_MODULE = None
+        t = Template('{% url will_not_match %}')
+        c = Context()
+        try:
+            rendered = t.render(c)
+        except TemplateSyntaxError, e:
+            #Assert that we are getting the template syntax error and not the
+            #string encoding error.
+            self.assertEquals(e.message, "Caught an exception while rendering: Reverse for 'will_not_match' with arguments '()' and keyword arguments '{}' not found.")
+        settings.SETTINGS_MODULE = old_settings_module
+
     def test_templates(self):
         template_tests = self.get_template_tests()
         filter_tests = filters.get_filter_tests()