summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdrian Holovaty <adrian@holovaty.com>2008-09-10 06:06:04 +0000
committerAdrian Holovaty <adrian@holovaty.com>2008-09-10 06:06:04 +0000
commit32db67e6f9c739b265f2860ffda42092bfa6b594 (patch)
treedce4ae97ff36a89d4a55c05c53b90d88e094cfdf
parent40cb11a55495c187bfd713b6e85dda72f076346b (diff)
Fixed #8957 -- Fixed incorrect error message when Admin prepopulated_fields refers to a field that does not exist. Thanks, charmless
git-svn-id: http://code.djangoproject.com/svn/django/trunk@9002 bcc190cf-cafb-0310-a4f2-bffc1f526a37
-rw-r--r--django/contrib/admin/validation.py13
1 files changed, 4 insertions, 9 deletions
diff --git a/django/contrib/admin/validation.py b/django/contrib/admin/validation.py
index 13d3a9d181..63be20f504 100644
--- a/django/contrib/admin/validation.py
+++ b/django/contrib/admin/validation.py
@@ -120,8 +120,7 @@ def validate(cls, model):
def validate_inline(cls):
# model is already verified to exist and be a Model
if cls.fk_name: # default value is None
- f = get_field(cls, cls.model, cls.model._meta,
- 'fk_name', cls.fk_name)
+ f = get_field(cls, cls.model, cls.model._meta, 'fk_name', cls.fk_name)
if not isinstance(f, models.ForeignKey):
raise ImproperlyConfigured("'%s.fk_name is not an instance of "
"models.ForeignKey." % cls.__name__)
@@ -229,19 +228,15 @@ def validate_base(cls, model):
% (cls.__name__, field))
check_isseq(cls, "prepopulated_fields['%s']" % field, val)
for idx, f in enumerate(val):
- get_field(cls, model,
- opts, "prepopulated_fields['%s'][%d]"
- % (f, idx), f)
+ get_field(cls, model, opts, "prepopulated_fields['%s'][%d]" % (field, idx), f)
def check_isseq(cls, label, obj):
if not isinstance(obj, (list, tuple)):
- raise ImproperlyConfigured("'%s.%s' must be a list or tuple."
- % (cls.__name__, label))
+ raise ImproperlyConfigured("'%s.%s' must be a list or tuple." % (cls.__name__, label))
def check_isdict(cls, label, obj):
if not isinstance(obj, dict):
- raise ImproperlyConfigured("'%s.%s' must be a dictionary."
- % (cls.__name__, label))
+ raise ImproperlyConfigured("'%s.%s' must be a dictionary." % (cls.__name__, label))
def get_field(cls, model, opts, label, field):
try: