Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed #7571 -- Fixed parameter matching in include()'d urlpattern #2375

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

afuna commented Feb 27, 2014

Fixed URL resolving in the case where an outer regex includes an inner
regex and both regexes use positional parameters instead of named
groups, causing the outer regex's parameters to override the inner
regex's.

Modified the regex url resolver so that it will concatenates and then
normalizes, instead of normalizing and then concatenating.

Fixed #7571 -- Fixed parameter matching in include()'d urlpattern
Fixed URL resolving in the case where an outer regex includes an inner
regex and both regexes use positional parameters instead of named
groups, causing the outer regex's parameters to override the inner
regex's.

Modified the regex url resolver so that it will concatenates and then
normalizes, instead of normalizing and then concatenating.
Member

bmispelon commented Mar 4, 2014

Merged in 5d568bc.

Thanks!

@bmispelon bmispelon closed this Mar 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment