diff --git a/django/contrib/admin/templatetags/admin_list.py b/django/contrib/admin/templatetags/admin_list.py index 7cd3f0a3af..b1ab7d543c 100644 --- a/django/contrib/admin/templatetags/admin_list.py +++ b/django/contrib/admin/templatetags/admin_list.py @@ -179,7 +179,7 @@ def items_for_result(cl, result, form): result_repr = conditional_escape(result_repr) yield mark_safe(u'%s' % (row_class, result_repr)) if form: - yield mark_safe(force_unicode(form[cl.model._meta.pk.name])) + yield mark_safe(u'%s' % force_unicode(form[cl.model._meta.pk.name])) def results(cl): if cl.formset: diff --git a/tests/regressiontests/admin_changelist/tests.py b/tests/regressiontests/admin_changelist/tests.py index 2164653b75..b70d7c51f4 100644 --- a/tests/regressiontests/admin_changelist/tests.py +++ b/tests/regressiontests/admin_changelist/tests.py @@ -1,7 +1,8 @@ import unittest from django.contrib import admin from django.contrib.admin.views.main import ChangeList -from regressiontests.admin_changelist.models import Child +from django.template import Context, Template +from regressiontests.admin_changelist.models import Child, Parent class ChangeListTests(unittest.TestCase): def test_select_related_preserved(self): @@ -11,10 +12,51 @@ class ChangeListTests(unittest.TestCase): """ m = ChildAdmin(Child, admin.site) cl = ChangeList(MockRequest(), Child, m.list_display, m.list_display_links, - m.list_filter,m.date_hierarchy, m.search_fields, + m.list_filter, m.date_hierarchy, m.search_fields, m.list_select_related, m.list_per_page, m.list_editable, m) self.assertEqual(cl.query_set.query.select_related, {'parent': {'name': {}}}) + def test_result_list_html(self): + """ + Regression test for #11791: Inclusion tag result_list generates a + table and this checks that the items are nested within the table + element tags. + """ + new_parent = Parent.objects.create(name='parent') + new_child = Child.objects.create(name='name', parent=new_parent) + request = MockRequest() + m = ChildAdmin(Child, admin.site) + cl = ChangeList(request, Child, m.list_display, m.list_display_links, + m.list_filter, m.date_hierarchy, m.search_fields, + m.list_select_related, m.list_per_page, m.list_editable, m) + FormSet = m.get_changelist_formset(request) + cl.formset = FormSet(queryset=cl.result_list) + template = Template('{% load admin_list %}{% spaceless %}{% result_list cl %}{% endspaceless %}') + context = Context({'cl': cl}) + table_output = template.render(context) + hidden_input_elem = '' + self.failIf(table_output.find(hidden_input_elem) == -1, + 'Failed to find expected hidden input element in: %s' % table_output) + self.failIf(table_output.find('%s' % hidden_input_elem) == -1, + 'Hidden input element is not enclosed in element.') + + # Test with list_editable fields + m.list_display = ['id', 'name', 'parent'] + m.list_display_links = ['id'] + m.list_editable = ['name'] + cl = ChangeList(request, Child, m.list_display, m.list_display_links, + m.list_filter, m.date_hierarchy, m.search_fields, + m.list_select_related, m.list_per_page, m.list_editable, m) + FormSet = m.get_changelist_formset(request) + cl.formset = FormSet(queryset=cl.result_list) + template = Template('{% load admin_list %}{% spaceless %}{% result_list cl %}{% endspaceless %}') + context = Context({'cl': cl}) + table_output = template.render(context) + self.failIf(table_output.find(hidden_input_elem) == -1, + 'Failed to find expected hidden input element in: %s' % table_output) + self.failIf(table_output.find('%s' % hidden_input_elem) == -1, + 'Hidden input element is not enclosed in element.') + class ChildAdmin(admin.ModelAdmin): list_display = ['name', 'parent'] def queryset(self, request):