Opened 8 years ago

Closed 8 years ago

#7821 closed (invalid)

Add exception value to NoReverseMatch exception

Reported by: jmohr@… Owned by: nobody
Component: Core (Other) Version: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The NoReverseMatch exception in django.core.urlresolvers.reverse has no string value associated with it. I added a string value to make it display the value of lookup_view, it makes it a little easier to debug URL issues. I've attached a patch, just a quick one-liner.

Attachments (2)

urlresolvers-reverse.diff (658 bytes) - added by jmohr@… 8 years ago.
Add a value to the NoReverseMatch exception raised by django.core.urlresolvers.reverse
urlresolvers-reverse-r8225.patch (664 bytes) - added by Adam Vandenberg 8 years ago.
Updated patch to r8225

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by jmohr@…

Attachment: urlresolvers-reverse.diff added

Add a value to the NoReverseMatch exception raised by django.core.urlresolvers.reverse

comment:1 Changed 8 years ago by Jeff Anderson

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedReady for checkin

Changed 8 years ago by Adam Vandenberg

Updated patch to r8225

comment:2 Changed 8 years ago by Julien Phalip

Resolution: invalid
Status: newclosed

This was fixed in [8272]. Try with a more recent version of trunk.

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