Browse files

Fixed #2578 -- Give a more accurate error message for admin.list_dis…


at model validation time. Patch from Christopher Lenz.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
malcolmt committed Sep 22, 2006
1 parent 9e05fc1 commit dc39762fde7c9a20f3bcf2025ca2cb46e5c73433
Showing with 2 additions and 1 deletion.
  1. +2 −1 django/core/
@@ -958,7 +958,8 @@ def get_validation_errors(outfile, app=None):
f = opts.get_field(fn)
except models.FieldDoesNotExist:
- e.add(opts, '"admin.list_filter" refers to %r, which isn\'t a field.' % fn)
+ if not hasattr(cls, fn):
+ e.add(opts, '"admin.list_display_links" refers to %r, which isn\'t an attribute, method or property.' % fn)
if fn not in opts.admin.list_display:
e.add(opts, '"admin.list_display_links" refers to %r, which is not defined in "admin.list_display".' % fn)
# list_filter

0 comments on commit dc39762

Please sign in to comment.