Ticket #14027: django-reverse-none.diff

django/core/urlresolvers.py

@@ -183 +183 @@
             else:
                 bits = normalize(p_pattern)
                 lookups.appendlist(pattern.callback, (bits, p_pattern))
-                lookups.appendlist(pattern.name, (bits, p_pattern))
+                if pattern.name is not None:
+                    lookups.appendlist(pattern.name, (bits, p_pattern))
         self._reverse_dict = lookups
         self._namespace_dict = namespaces
         self._app_dict = apps

tests/regressiontests/urlpatterns_reverse/tests.py

@@ -118 +118 @@
                 self.assertEqual(expected, NoReverseMatch)
             else:
                 self.assertEquals(got, expected)
+    
+    def test_reverse_none(self):
+        # Reversing None should raise an error, not return the last un-named
+        # view.
+        self.assertRaises(NoReverseMatch, reverse, None)
 
 class ResolverTests(unittest.TestCase):
     def test_non_regex(self):