[1.7.x] Fixed #22486 -- Restored the ability to reverse views created using functools.partial.
Regression in8b93b31487
. Thanks rcoup for the report. Backport of3c06b2f2a3
from master
This commit is contained in:
parent
c3228ef3e2
commit
e192f13103
|
@ -8,6 +8,7 @@ a string) and returns a tuple in this format:
|
|||
"""
|
||||
from __future__ import unicode_literals
|
||||
|
||||
import functools
|
||||
from importlib import import_module
|
||||
import re
|
||||
from threading import local
|
||||
|
@ -270,6 +271,9 @@ class RegexURLResolver(LocaleRegexProvider):
|
|||
self._callback_strs.add(pattern._callback_str)
|
||||
elif hasattr(pattern, '_callback'):
|
||||
callback = pattern._callback
|
||||
if isinstance(callback, functools.partial):
|
||||
callback = callback.func
|
||||
|
||||
if not hasattr(callback, '__name__'):
|
||||
lookup_str = callback.__module__ + "." + callback.__class__.__name__
|
||||
else:
|
||||
|
|
|
@ -0,0 +1,14 @@
|
|||
===========================
|
||||
Django 1.4.12 release notes
|
||||
===========================
|
||||
|
||||
*Under development*
|
||||
|
||||
Django 1.4.12 fixes a regression in the 1.4.11 security release.
|
||||
|
||||
Bugfixes
|
||||
========
|
||||
|
||||
* Restored the ability to :meth:`~django.core.urlresolvers.reverse` views
|
||||
created using :func:`functools.partial()`
|
||||
(`#22486 <http://code.djangoproject.com/ticket/22486>`_)
|
|
@ -0,0 +1,14 @@
|
|||
==========================
|
||||
Django 1.5.7 release notes
|
||||
==========================
|
||||
|
||||
*Under development*
|
||||
|
||||
Django 1.5.7 fixes a regression in the 1.5.6 security release.
|
||||
|
||||
Bugfixes
|
||||
========
|
||||
|
||||
* Restored the ability to :meth:`~django.core.urlresolvers.reverse` views
|
||||
created using :func:`functools.partial()`
|
||||
(`#22486 <http://code.djangoproject.com/ticket/22486>`_)
|
|
@ -12,3 +12,7 @@ Bugfixes
|
|||
* Added backwards compatibility support for the :mod:`django.contrib.messages`
|
||||
cookie format of Django 1.4 and earlier to facilitate upgrading to 1.6 from
|
||||
1.4 (`#22426 <http://code.djangoproject.com/ticket/22426>`_).
|
||||
|
||||
* Restored the ability to :meth:`~django.core.urlresolvers.reverse` views
|
||||
created using :func:`functools.partial()`
|
||||
(`#22486 <http://code.djangoproject.com/ticket/22486>`_)
|
||||
|
|
|
@ -40,6 +40,7 @@ Final releases
|
|||
.. toctree::
|
||||
:maxdepth: 1
|
||||
|
||||
1.5.7
|
||||
1.5.6
|
||||
1.5.5
|
||||
1.5.4
|
||||
|
@ -53,6 +54,7 @@ Final releases
|
|||
.. toctree::
|
||||
:maxdepth: 1
|
||||
|
||||
1.4.12
|
||||
1.4.11
|
||||
1.4.10
|
||||
1.4.9
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
from django.conf.urls import patterns, url, include
|
||||
|
||||
from .views import empty_view, absolute_kwargs_view
|
||||
from .views import empty_view, empty_view_partial, empty_view_wrapped, absolute_kwargs_view
|
||||
|
||||
|
||||
other_patterns = patterns('',
|
||||
|
@ -51,6 +51,10 @@ urlpatterns = patterns('',
|
|||
# This is non-reversible, but we shouldn't blow up when parsing it.
|
||||
url(r'^(?:foo|bar)(\w+)/$', empty_view, name="disjunction"),
|
||||
|
||||
# Partials should be fine.
|
||||
url(r'^partial/', empty_view_partial, name="partial"),
|
||||
url(r'^partial_wrapped/', empty_view_wrapped, name="partial_wrapped"),
|
||||
|
||||
# Regression views for #9038. See tests for more details
|
||||
url(r'arg_view/$', 'kwargs_view'),
|
||||
url(r'arg_view/(?P<arg1>\d+)/$', 'kwargs_view'),
|
||||
|
|
|
@ -1,3 +1,5 @@
|
|||
from functools import partial, update_wrapper
|
||||
|
||||
from django.http import HttpResponse
|
||||
from django.views.generic import RedirectView
|
||||
from django.core.urlresolvers import reverse_lazy
|
||||
|
@ -55,3 +57,11 @@ def login_required_view(request):
|
|||
|
||||
def bad_view(request, *args, **kwargs):
|
||||
raise ValueError("I don't think I'm getting good value for this view")
|
||||
|
||||
|
||||
empty_view_partial = partial(empty_view, template_name="template.html")
|
||||
|
||||
|
||||
empty_view_wrapped = update_wrapper(
|
||||
partial(empty_view, template_name="template.html"), empty_view,
|
||||
)
|
||||
|
|
Loading…
Reference in New Issue