Ticket #20757: patch_commit_f8ce5686938b.patch

django/core/urlresolvers.py

@@ -8 +8 @@
 """
 from __future__ import unicode_literals
 
+from collections import namedtuple
 from importlib import import_module
 import re
 from threading import local
@@ -36 +37 @@
 # Overridden URLconfs for each thread are stored here.
 _urlconfs = local()
 
+URLEntry = namedtuple('URLEntry', ['urlbits', 'p_pattern', 'default_args'])
+URLBit = namedtuple('URLBit', ['format', 'argument_names'])
+
 
 class ResolverMatch(object):
     def __init__(self, func, args, kwargs, url_name=None, app_name=None, namespaces=None):
@@ -269 +273 @@
                     if pattern.app_name:
                         apps.setdefault(pattern.app_name, []).append(pattern.namespace)
                 else:
-                    parent = normalize(pattern.regex.pattern)
+                    parent = [URLBit(*p) for p in normalize(pattern.regex.pattern)]
                     for name in pattern.reverse_dict:
                         for matches, pat, defaults in pattern.reverse_dict.getlist(name):
                             new_matches = []
                             for piece, p_args in parent:
-                                new_matches.extend((piece + suffix, p_args + args) for (suffix, args) in matches)
-                            lookups.appendlist(name, (new_matches, p_pattern + pat, dict(defaults, **pattern.default_kwargs)))
+                                new_matches.extend([URLBit(piece + suffix, p_args + args) for (suffix, args) in matches])
+                            lookups.appendlist(name, URLEntry(new_matches, p_pattern + pat, dict(defaults, **pattern.default_kwargs)))
                     for namespace, (prefix, sub_pattern) in pattern.namespace_dict.items():
                         namespaces[namespace] = (p_pattern + prefix, sub_pattern)
                     for app_name, namespace_list in pattern.app_dict.items():
                         apps.setdefault(app_name, []).extend(namespace_list)
             else:
-                bits = normalize(p_pattern)
-                lookups.appendlist(pattern.callback, (bits, p_pattern, pattern.default_args))
+                bits = [URLBit(*p) for p in normalize(p_pattern)]
+                lookups.appendlist(pattern.callback, URLEntry(bits, p_pattern, pattern.default_args))
                 if pattern.name is not None:
-                    lookups.appendlist(pattern.name, (bits, p_pattern, pattern.default_args))
+                    lookups.appendlist(pattern.name, URLEntry(bits, p_pattern, pattern.default_args))
         self._reverse_dict[language_code] = lookups
         self._namespace_dict[language_code] = namespaces
         self._app_dict[language_code] = apps
@@ -575 +579 @@
         return True
     except Resolver404:
         return False
+
+def get_urlformat(urlname):
+    """
+    Given a URL name, returns the URL as a string suitable for string.format.
+
+    Example::
+
+    urlpatterns = patterns('',
+        url(r'^extra/(?P<extra>\w+)/$', empty_view, name="named-url2"),
+    )
+
+    >>> get_urlformat('named-url2')
+    '/extra/%(extra)s/'
+    """
+    urlconf = get_urlconf()
+    resolver = get_resolver(urlconf)
+    urlbit = resolver.reverse_dict[urlname].urlbits[0]
+    return "%s%s" % (get_script_prefix(), urlbit.format)

tests/urlpatterns_reverse/tests.py

@@ -9 +9 @@
 from django.contrib.auth.models import User
 from django.core.exceptions import ImproperlyConfigured, ViewDoesNotExist
 from django.core.urlresolvers import (reverse, reverse_lazy, resolve, get_callable,
-    get_resolver, NoReverseMatch, Resolver404, ResolverMatch, RegexURLResolver,
-    RegexURLPattern)
+    get_resolver, get_urlformat, NoReverseMatch, Resolver404, ResolverMatch,
+    RegexURLResolver, RegexURLPattern)
 from django.http import HttpRequest, HttpResponseRedirect, HttpResponsePermanentRedirect
 from django.shortcuts import redirect
 from django.test import TestCase
@@ -657 +657 @@
         self.assertRaises(AttributeError, get_callable,
             'urlpatterns_reverse.views_broken.i_am_broken')
 
+
+class URLObjects(TestCase):
+    urls = 'urlpatterns_reverse.named_urls'
+
+    def test_URL_objects(self):
+        format = get_urlformat('named-url2')
+        self.assertEqual(format, '/extra/%(extra)s/')