Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#17492 closed Bug (fixed)

url backreferencing can not be reversed.

Reported by: Nate Bragg Owned by: Nate Bragg
Component: Core (Other) Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

If a url pattern uses backreferencing (i.e., "(?P=previously_named_group)"), the reversal fails, throwing up "Non-reversible reg-exp portion" ValueError. The problem occurs in django.utils.regex_helper.normalize which gives up on any character following a "(?P" other than "<".

Attachments (2)

17492.diff (5.9 KB ) - added by Chris Beaven 12 years ago.
regex_backreference.diff (6.7 KB ) - added by Nate Bragg 12 years ago.
A patch to regex_helper.py, along with unit tests for normalize.

Download all attachments as: .zip

Change History (9)

comment:1 by Nate Bragg, 12 years ago

Owner: changed from nobody to Nate Bragg
Status: newassigned

comment:2 by Nate Bragg, 12 years ago

Has patch: set
Version 0, edited 12 years ago by Nate Bragg (next)

comment:3 by Nate Bragg, 12 years ago

After being encouraged to on IRC - here's my rationale:

I had something like this in my urlpatterns:

url(r'^static_text/(?P<group_one>[a-z][a-z_]*)/(?P<group_two>\d+)/(?P=group_one)_(?P=group_two)_(?P<group_three>\w+)/$', views.a_view)

Because "(?P=" wasn't being parsed, it was choking on reversing... a DIFFERENT pattern. So, I figured the right thing to do would be to recognize this pattern too, since backreferences are so simple (i.e., they have to only have an identifier name between '=' and ')', and that identifier name had to previously have occurred).

Now, in my patch, I don't check that the backreference actually occurred! BUT - that's okay, because if your url pattern has that problem, it didn't compile anyhow, and you never got to 'reverse' anyhow (that, and I reasoned that this behavior is in line with other omissions that 'normalize' allows).

by Chris Beaven, 12 years ago

Attachment: 17492.diff added

comment:4 by Chris Beaven, 12 years ago

Patch needs improvement: set

I've added in a urlpatterns_reverse test, and it identifies a problem. See the commented test line in there:

Currently, reversing a named pattern still works with a list of arguments as well as a dictionary. Using a named backreference doesn't work with a list of arguments in the same way (you shouldn't have to provide the name twice).

Unless this can be resolved, I'm not quite sure that this can be added.

by Nate Bragg, 12 years ago

Attachment: regex_backreference.diff added

A patch to regex_helper.py, along with unit tests for normalize.

comment:5 by Nate Bragg, 12 years ago

Patch needs improvement: unset

Very good catch! Thankfully, it has a simple solution - None can be passed in as the variable, which excludes it. With my replacement patch, your test cases now both pass.

Last edited 12 years ago by Nate Bragg (previous) (diff)

comment:6 by Chris Beaven, 12 years ago

Resolution: fixed
Status: assignedclosed

In [17336]:

Fixed #17492 -- Allow reversal of named backreferences. Thanks nate_b

comment:7 by Chris Beaven, 12 years ago

(added nate_b to AUTHORS in r17337)

Note: See TracTickets for help on using tickets.
Back to Top