Ticket #29929: 0001-Tests-that-demonstrate-breakage.patch

tests/admin_views/admin.py

@@ -35 +35 @@
     MainPrepopulated, ModelWithStringPrimaryKey, NotReferenced, OldSubscriber,
     OtherStory, Paper, Parent, ParentWithDependentChildren, ParentWithUUIDPK,
     Person, Persona, Picture, Pizza, Plot, PlotDetails, PlotProxy,
-    PluggableSearchPerson, Podcast, Post, PrePopulatedPost,
+    PluggableSearchPerson, Podcast, Post, PrePopulatedPost, PrepopulatedModel,
     PrePopulatedPostLargeSlug, PrePopulatedSubPost, Promo, Question,
     ReadablePizza, ReadOnlyPizza, Recipe, Recommendation, Recommender,
     ReferencedByGenRel, ReferencedByInline, ReferencedByParent,
@@ -949 +949 @@
             raise Exception("'obj' passed to get_formsets_with_inlines was None during change_view")
         return super().get_formsets_with_inlines(request, obj)
 
+# Admin for #29929
+class PrepopulatedReadOnlyAdmin(admin.ModelAdmin):
+    prepopulated_fields = {'to_be_prepopulated': ('input_value',)}
+
+    def has_change_permission(self, *args, **kwargs):
+        return False
 
 site = admin.AdminSite(name="admin")
 site.site_url = '/my-site-url/'
@@ -1021 +1027 @@
 site.register(ParentWithUUIDPK)
 site.register(RelatedPrepopulated, search_fields=['name'])
 site.register(RelatedWithUUIDPKModel)
+site.register(PrepopulatedModel, PrepopulatedReadOnlyAdmin)
 
 # We intentionally register Promo and ChapterXtra1 but not Chapter nor ChapterXtra2.
 # That way we cover all four cases:

tests/admin_views/models.py

@@ -979 +979 @@
 class Authorship(models.Model):
     book = models.ForeignKey(Book, models.CASCADE)
     author = models.ForeignKey(Author, models.CASCADE)
+
+
+# Model for 29929
+class PrepopulatedModel(models.Model):
+    input_value = models.TextField()
+    to_be_prepopulated = models.TextField()

tests/admin_views/tests.py

@@ -46 +46 @@
     MainPrepopulated, Media, ModelWithStringPrimaryKey, OtherStory, Paper,
     Parent, ParentWithDependentChildren, ParentWithUUIDPK, Person, Persona,
     Picture, Pizza, Plot, PlotDetails, PluggableSearchPerson, Podcast, Post,
-    PrePopulatedPost, Promo, Question, ReadablePizza, ReadOnlyPizza,
+    PrePopulatedPost, PrepopulatedModel, Promo, Question, ReadablePizza, ReadOnlyPizza,
     Recommendation, Recommender, RelatedPrepopulated, RelatedWithUUIDPKModel,
     Report, Restaurant, RowLevelChangePermissionModel, SecretHideout, Section,
     ShortMessage, Simple, State, Story, SuperSecretHideout, SuperVillain,
@@ -972 +972 @@
         self.assertContains(response, '<th scope="col"  class="column-value">')
         self.assertNotContains(response, '<th scope="col"  class="sortable column')
 
+    def test_prepopulatedfields_readonly(self):
+        """
+        Check if prepopulated fields don't break change views when they're read only.
+
+        See #29929
+        """
+        PrepopulatedModel.objects.create(pk=1, input_value='foo', to_be_prepopulated='foo')
+        self.client.get(reverse('admin:admin_views_prepopulatedmodel_change', args=(1,)))
+
 
 @override_settings(TEMPLATES=[{
     'BACKEND': 'django.template.backends.django.DjangoTemplates',