search

prymitive / bootstrap-breadcrumbs

Django template tags for easy breadcrumbs using twitter bootstrap css classes or custom template
django-bootstrap-breadcrumbs.readthedocs.org
MIT License
93 stars 48 forks source link

Pass 'current_app' when rendering view names #8

Closed edavis closed 11 years ago

edavis commented 11 years ago

Hey,

Currently the breadcrumbs don't utilize current_app when reversing view names. This can lead to unexpected results when using namespaced URLs.

With this commit, current_app is figured out from the request object and passed to reverse.

Tested and works on Django 1.5. I didn't have a pre-1.5 codebase handy, but that AttributeError exception handler should cover them. Feel free to test/tweak as needed.

Thanks.

prymitive commented 11 years ago

released in 0.5.3

edavis commented 11 years ago

Thanks.