url tag not working with reverse in 1.0 as it previously did

This looks like a bug, not changed behaviour. If you're passing in a parameter, it must match the first pattern, since that's the only one that matches anything with a parameter. The fact that it isn't doing so is wrong. One of the points behind this rewrite is that similarly named but pattern-wise distinct URL patterns that are differentiated by the parameters they take should be handled correctly.

We used the following code to work around this bug. The code removes items from args and kwargs where the value is empty or None, then calls the real reverse function with the correct args and kwargs.

from django.core.urlresolvers import reverse as django_reverse

def reverse(*args,**kw):
  if 'args' in kw: kw['args'] = tuple((a for a in kw['args'] if a))
  if 'kwargs' in kw: kw['kwargs'] = dict(((k,v) for k,v in kw['kwargs'].items() if v))
  return django_reverse(*args,**kw)

This works fine in the views, but it does not fix the "url" template tags which use Django's reverse function. You could update the "URLNode" class in django/template/defaulttags.py, so the "render" method redefines "reverse" using the code above.

-- Will Reese

Fix for URL reversing with tests

Malcolm: I haven't checked this in because (1) The URL resolver is one of your regular stomping grounds, (2) I wasn't certain if you had a better way of solving this in mind.

My read on the problem - If you deploy the same view method at two different URLs, calling reverse() using the view method name becomes ambiguous. Using a MultiValueDict? rather than a dict means you can find all the places that a given view is deployed.

This looks like the right fix, or, at least, fixing the right problem. The implementation makes me a little nervous because it's essentially introducing an extra level of nesting in the data structure (which is shown up by the extra outer loop when working out the match). I've basically gotten into trouble in the current code by storing the pattern prefix only once for a whole bunch of possible argument combination (the new_matches list in _get_reverse_dict() is the collection of possibilities for the current pattern prefix. So Russell's patch adds extra "groups" of pattern + possibility sets, which looks fine. I want to think about whether there are alternative data structures to use there. MultiValueDict feels shady to me somehow, since I always have to think about how it works. It's not really a natural Python data structure. Maybe lists of lists or something explicit might look more comfortable.

I'll try out alternatives, but it will only be variations on this. Thanks for working this out, Russ. I missed some possibilities in the original code.

(In [9087]) Fixed #9038 -- Correctly handle URL patterns with the same name (or view name), declared independently and that differ only by argument signatures.

Patch from Russell Keith-Magee.

I still face the exact same issue with build 9038. Any suggestions?

What do you mean with 'build 9038'?. If yo meant revision 9038 then that's where your problem lies, you need to be running revision 9087 or newer to see the fix that closed this ticket.

The issue in this ticket has been fixed. If anybody is seeing a different problem, please open a new ticket with a repeatable test case so that we can debug that problem.

Sorry.. I wanted to mean revision 9087. Even with revision 9087 I see the same exception:

TemplateSyntaxError? at /customize/themeeditor/themeoftheday/

Caught an exception while rendering: Reverse for 'navburner.theme_editor' with arguments '()' and keyword arguments '{}' not found.

Original Traceback (most recent call last):

File "C:\workspaces\navburner-02\django-latest\django\template\debug.py", line 71, in render_node

result = node.render(context)

File "C:\workspaces\navburner-02\django-latest\django\template\defaulttags.py", line 378, in render

args=args, kwargs=kwargs)

File "C:\workspaces\navburner-02\django-latest\django\core\urlresolvers.py", line 254, in reverse

*args, **kwargs)))

File "C:\workspaces\navburner-02\django-latest\django\core\urlresolvers.py", line 243, in reverse

"arguments '%s' not found." % (lookup_view, args, kwargs))

NoReverseMatch?: Reverse for 'navburner.theme_editor' with arguments '()' and keyword arguments '{}' not found.

You are not seeing exactly the same problem, since your exception involves no arguments and the initial report had an argument. Also you haven't provided enough information to help figure out what is going wrong, since the exception alone without some information on your url patterns is not very helpful. Finally, this is not the right place to figure out what is going wrong -- as Malcolm said the error in this ticket, which was very specific and not something that covers all "NoReverseMatch" errors -- has been fixed. The best way to figure out what is going on in your case is probably to post on django-users with enough information about your url patters and what you are doing to generate the exception to hopefully identify if the problem is in your setup or the Django code. If Django code, then a new ticket would be appropriate.