innov
/
django1
1
0
Fork 0
Code Issues Pull Requests 1 Projects Releases Wiki Activity

Fixed #24221 - Used precompiled regexp for percent-placeholder matching.

This commit is contained in:
Tomáš Ehrlich 2015-01-26 11:23:48 +01:00 committed by Tim Graham
parent ee23e03637
commit ea0ea7859a
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
django/views/generic/edit.py
View File

@ -12,6 +12,8 @@ from django.views.generic.base import TemplateResponseMixin, ContextMixin, View
from django.views.generic.detail import (SingleObjectMixin, from django.views.generic.detail import (SingleObjectMixin,
SingleObjectTemplateResponseMixin, BaseDetailView) SingleObjectTemplateResponseMixin, BaseDetailView)
PERCENT_PLACEHOLDER_REGEX = re.compile(r'%\([^\)]+\)') # RemovedInDjango20Warning
class FormMixinBase(type): class FormMixinBase(type):
def __new__(cls, name, bases, attrs): def __new__(cls, name, bases, attrs):
@ -163,7 +165,7 @@ class ModelFormMixin(FormMixin, SingleObjectMixin):
Returns the supplied URL. Returns the supplied URL.
""" """
if self.success_url: if self.success_url:
if re.search(r'%\([^\)]+\)', self.success_url): if PERCENT_PLACEHOLDER_REGEX.search(self.success_url):
warnings.warn( warnings.warn(
"%()s placeholder style in success_url is deprecated. " "%()s placeholder style in success_url is deprecated. "
"Please replace them by the {} Python format syntax.", "Please replace them by the {} Python format syntax.",
@ -297,7 +299,7 @@ class DeletionMixin(object):
def get_success_url(self): def get_success_url(self):
if self.success_url: if self.success_url:
if re.search(r'%\([^\)]+\)', self.success_url): if PERCENT_PLACEHOLDER_REGEX.search(self.success_url):
warnings.warn( warnings.warn(
"%()s placeholder style in success_url is deprecated. " "%()s placeholder style in success_url is deprecated. "
"Please replace them by the {} Python format syntax.", "Please replace them by the {} Python format syntax.",