Opened 7 years ago

Closed 7 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@… 7 years ago.
Add a value to the NoReverseMatch exception raised by django.core.urlresolvers.reverse
urlresolvers-reverse-r8225.patch (664 bytes) - added by adamv 7 years ago.
Updated patch to r8225

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by jmohr@…

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

comment:1 Changed 7 years ago by programmerq

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

Changed 7 years ago by adamv

Updated patch to r8225

comment:2 Changed 7 years ago by julien

  • Resolution set to invalid
  • Status changed from new to closed

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

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