Improved error message for reverse v. reverse_lazy
|Reported by:||Elena Williams||Owned by:||Bas Peschier|
|Cc:||eromijn@…||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
ImproperlyConfigured: The included urlconf %s doesn't have any patterns in it.
Uninformative foremost as %s is top-level urls (problem usually in included urls).
Caused by using reverse rather than reverse_lazy in some view.
No feedback whatsoever is given as to:
- that the error is caused in view rather than url
- which view this might be, or even which app it is in
Context: You get this error starting the developer server. The error is cached (#11712) and fixing it doesn't make the error go away adding to the confusion.
Suggestion: At least more specificity as to where the error lies.
The above added here: https://code.djangoproject.com/wiki/BetterErrorMessages
Appears be reproducible by adding to any view:
from django.core.urlresolvers import reverse class DetailView(generic.DetailView): ... attr = reverse('anything')
Change History (8)
comment:1 Changed 3 years ago by
|Patch needs improvement:||unset|
|Triage Stage:||Unreviewed → Accepted|
comment:2 Changed 3 years ago by
|Component:||Uncategorized → Core (URLs)|
|Type:||Uncategorized → Cleanup/optimization|
comment:3 Changed 3 years ago by
|Owner:||changed from nobody to Bas Peschier|
|Status:||new → assigned|