From 8972818289452d611d97fac0f4a6d24625987b31 Mon Sep 17 00:00:00 2001 From: Alasdair Nicol Date: Sat, 1 Aug 2015 22:32:51 +0100 Subject: [PATCH] Fixed #25206 -- Fixed error message when checking a ModelAdmin fieldset's fields. --- django/contrib/admin/checks.py | 2 +- tests/admin_checks/tests.py | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/django/contrib/admin/checks.py b/django/contrib/admin/checks.py index 15a2719777..c74e8b96c2 100644 --- a/django/contrib/admin/checks.py +++ b/django/contrib/admin/checks.py @@ -135,7 +135,7 @@ class BaseModelAdminChecks(object): ) ] elif not isinstance(fieldset[1]['fields'], (list, tuple)): - return must_be('a list or tuple', option="fieldsets[1]['fields']", obj=cls, id='admin.E008') + return must_be('a list or tuple', option="%s[1]['fields']" % label, obj=cls, id='admin.E008') fields = flatten(fieldset[1]['fields']) if len(fields) != len(set(fields)): diff --git a/tests/admin_checks/tests.py b/tests/admin_checks/tests.py index d988ade920..0b4d905c1b 100644 --- a/tests/admin_checks/tests.py +++ b/tests/admin_checks/tests.py @@ -125,7 +125,7 @@ class SystemChecksTestCase(SimpleTestCase): def test_fieldsets_fields_non_tuple(self): """ - Tests for a tuple/list within fieldsets[1]['fields']. + Tests for a tuple/list for the first fieldset's fields. """ class NotATupleAdmin(admin.ModelAdmin): list_display = ["pk", "title"] @@ -139,7 +139,7 @@ class SystemChecksTestCase(SimpleTestCase): errors = NotATupleAdmin.check(model=Song) expected = [ checks.Error( - "The value of 'fieldsets[1]['fields']' must be a list or tuple.", + "The value of 'fieldsets[0][1]['fields']' must be a list or tuple.", hint=None, obj=NotATupleAdmin, id='admin.E008', @@ -149,7 +149,7 @@ class SystemChecksTestCase(SimpleTestCase): def test_nonfirst_fieldset(self): """ - Tests for a tuple/list within the second fieldsets[2]['fields']. + Tests for a tuple/list for the second fieldset's fields. """ class NotATupleAdmin(admin.ModelAdmin): fieldsets = [ @@ -164,7 +164,7 @@ class SystemChecksTestCase(SimpleTestCase): errors = NotATupleAdmin.check(model=Song) expected = [ checks.Error( - "The value of 'fieldsets[1]['fields']' must be a list or tuple.", + "The value of 'fieldsets[1][1]['fields']' must be a list or tuple.", hint=None, obj=NotATupleAdmin, id='admin.E008',