From d311124be59df64278f3149d68e79ce45b8a6c64 Mon Sep 17 00:00:00 2001
From: Tim Graham <timograham@gmail.com>
Date: Sat, 18 Aug 2018 16:15:18 -0400
Subject: Fixed #29682 -- Fixed admin change form crash if a view-only model's
 form has an extra field.

---
 tests/admin_views/admin.py | 11 ++++++++++-
 tests/admin_views/tests.py |  1 +
 2 files changed, 11 insertions(+), 1 deletion(-)

(limited to 'tests/admin_views')

diff --git a/tests/admin_views/admin.py b/tests/admin_views/admin.py
index 8565d04a05..04cc6c79e7 100644
--- a/tests/admin_views/admin.py
+++ b/tests/admin_views/admin.py
@@ -91,6 +91,14 @@ class ChapterXtra1Admin(admin.ModelAdmin):
     )
 
 
+class ArticleForm(forms.ModelForm):
+    extra_form_field = forms.BooleanField(required=False)
+
+    class Meta:
+        fields = '__all__'
+        model = Article
+
+
 class ArticleAdmin(admin.ModelAdmin):
     list_display = (
         'content', 'date', callable_year, 'model_year', 'modeladmin_year',
@@ -101,10 +109,11 @@ class ArticleAdmin(admin.ModelAdmin):
     list_filter = ('date', 'section')
     autocomplete_fields = ('section',)
     view_on_site = False
+    form = ArticleForm
     fieldsets = (
         ('Some fields', {
             'classes': ('collapse',),
-            'fields': ('title', 'content')
+            'fields': ('title', 'content', 'extra_form_field'),
         }),
         ('Some other fields', {
             'classes': ('wide',),
diff --git a/tests/admin_views/tests.py b/tests/admin_views/tests.py
index 297d625376..df1936aa90 100644
--- a/tests/admin_views/tests.py
+++ b/tests/admin_views/tests.py
@@ -1768,6 +1768,7 @@ class AdminViewPermissionsTest(TestCase):
         response = self.client.get(article_change_url)
         self.assertEqual(response.status_code, 200)
         self.assertEqual(response.context['title'], 'View article')
+        self.assertContains(response, '<label>Extra form field:</label>')
         self.assertContains(response, '<a href="/test_admin/admin/admin_views/article/" class="closelink">Close</a>')
         post = self.client.post(article_change_url, change_dict)
         self.assertEqual(post.status_code, 302)
-- 
cgit v1.3