Opened 19 years ago
Closed 19 years ago
#1818 closed enhancement (fixed)
[patch] Slightly better error message for FieldDoesNotExist error
Reported by: | Owned by: | Adrian Holovaty | |
---|---|---|---|
Component: | Core (Other) | Version: | dev |
Severity: | minor | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Sometimes you get a FieldDoesNotExist
error that does not provide enough information because it doesn't indicate which model class lacks a certain field. The following change would improve the error:
Index: django/db/models/options.py =================================================================== --- django/db/models/options.py (revision 413) +++ django/db/models/options.py (working copy) @@ -94,7 +94,8 @@ for f in to_search: if f.name == name: return f - raise FieldDoesNotExist, "name=%s" % name + raise FieldDoesNotExist('%s has no field named "%s"' + % (self.module_name, name)) def get_order_sql(self, table_prefix=''): "Returns the full 'ORDER BY' clause for this object, according to self.ordering."
Note:
See TracTickets
for help on using tickets.
(In [2870]) Fixed #1818 -- Added better error message for FieldDoesNotExist error. Thanks, Christopher Lenz