diff options
| author | Adrian Holovaty <adrian@holovaty.com> | 2008-09-10 06:06:04 +0000 |
|---|---|---|
| committer | Adrian Holovaty <adrian@holovaty.com> | 2008-09-10 06:06:04 +0000 |
| commit | 32db67e6f9c739b265f2860ffda42092bfa6b594 (patch) | |
| tree | dce4ae97ff36a89d4a55c05c53b90d88e094cfdf | |
| parent | 40cb11a55495c187bfd713b6e85dda72f076346b (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.py | 13 |
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: |
