From 7a8460191eabcbe3b63deeebcde7c7097f62bb08 Mon Sep 17 00:00:00 2001 From: Alasdair Nicol Date: Sat, 1 Aug 2015 22:32:51 +0100 Subject: [PATCH] [1.8.x] Fixed #25206 -- Fixed error message when checking a ModelAdmin fieldset's fields. Backport of 8972818289452d611d97fac0f4a6d24625987b31 from master --- 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 d8f7fe57e4..663138ced0 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 647a3d0eb4..e4e09d795b 100644 --- a/tests/admin_checks/tests.py +++ b/tests/admin_checks/tests.py @@ -126,7 +126,7 @@ class SystemChecksTestCase(TestCase): 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"] @@ -140,7 +140,7 @@ class SystemChecksTestCase(TestCase): 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', @@ -150,7 +150,7 @@ class SystemChecksTestCase(TestCase): 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 = [ @@ -165,7 +165,7 @@ class SystemChecksTestCase(TestCase): 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',