#20841: NotImplementedErrors should provide exception messages
--------------------------------------+---------------------------------
Reporter: joseph@… | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Uncategorized | Version: 1.5
Severity: Normal | Keywords: NotImplementedError
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 1 | UI/UX: 0
--------------------------------------+---------------------------------
The Django source has many occurrences of NotImplementedErrors raised
without a message. For example, if a dev creates a class inheriting from
contrib.admin.SimpleListFilter and does not provide a lookups() method,
all they see from the Debug template is "NotImplementedError at
/path/to/file/ No exception supplied". This, in my opinion, makes seeking
help difficult for those not comfortable enough to dig through the Django
source. A simple message such as "instances of SimpleListFilter must
provide a lookups() method" provides instant feedback and a non-ambiguous
means to seeking help if needed.
In short, my request is to have all NotImplementedError exceptions raised
accompanied by a message.
I realize this change would be boring to work on, so I am more than
willing to read through the patch procedures and submit a patch myself if
you all find this a worthy cause. (The nature of this request may even be
simple enough to send through Github)
--
Ticket URL: <https://code.djangoproject.com/ticket/20841>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/065.9fc050ebc86469d545f4c0fd8468b76e%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.
