Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reverse_named_url fails on Django 1.5 if using url arguments #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SpectralAngel
Copy link

While migrating an app to Django 1.5 one of my menus handled by treemenus started to fail, it was looking for an url named this way:

"my_awesome_url its_awesome_argument"

All as a single url, this commit fixes that, but i owe you a test case.

@jphalip
Copy link
Owner

jphalip commented Apr 7, 2013

Hello and thanks for the patch. If you could provide a test, that would be great. That would also clarify the issue you've run into. Cheers!

@leliocampanile
Copy link

I'm trying this patch and it works for me with django 1.5.1.

@cary929
Copy link

cary929 commented May 4, 2015

https://docs.djangoproject.com/en/1.8/ref/templates/builtins/#std:templatetag-url

try to use 'url' to replace 'reverse_named_url' :)

just like :

  • {{ menu_item.caption }}
  • Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    None yet
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    4 participants