Non-editable FieldError raised when using a GenericForeignKey on a form.

[James Pic]

https://code.djangoproject.com/ticket/35224 made GFK a Field with hard-coded editable=False

This breaks this very nice feature in Django 5.1 ​https://django-autocomplete-light.readthedocs.io/en/master/gfk.html

With exception: cannot be specified for TModel model form as it is a non-editable field. Check fields/fieldsets/exclude attributes of class TestAdmin

While I understand this is an interesting default protection for users, it would also be nice if Django would keep on allowing users to create their form fields for GFK if they want to, as they could until Django 5.0

[James Pic]

This fixes the problem enough ​https://github.com/django/django/pull/19110

If you want agree to any fix of this kind I'll gladly add a test.

TYIA for your feedback

[James Pic]

In case any dal users stumbles on this, there's a documented monkey patch meanwhile this is fixed, or in case this won't be fixed ​https://github.com/yourlabs/django-autocomplete-light/commit/2e8933d2473409a19c624610800e077d3fbf1074

[Sarah Boyce]

This looks fiddly, but the following test fails on main, but was passing on 5.0 and is a regression in 6002df06713cb0a7050432263527a25754190c27

On main, the error is django.core.exceptions.FieldError: 'question' cannot be specified for Answer model form as it is a non-editable field.

tests/contenttypes_tests/test_fields.py

@@ -1 +1 @@
 import json
 
+from django import forms
 from django.contrib.contenttypes.fields import GenericForeignKey
 from django.contrib.contenttypes.prefetch import GenericPrefetch
 from django.db import models
@@ -65 +66 @@
         answer.refresh_from_db()
         self.assertIsNot(answer.question, old_question_obj)
 
+    def test_form_field(self):
+        class AnswerForm(forms.ModelForm):
+            question = forms.CharField()
+
+            class Meta:
+                model = Answer
+                fields = ["question"]
+
+        form = AnswerForm()
+        self.assertIn("question", form.fields)
+
 
 class GenericRelationTests(TestCase):
     def test_value_to_string(self):

However, note that question was excluded by default on the form.

Looks like this was also raised here: ​https://forum.djangoproject.com/t/non-editable-error-for-a-genericforeignkey-in-an-modelform-when-updating-to-version-5-1/35033

[James Pic]

pushed test and docs, keeping the current behaviour and allowing to go back to the previous one with explicit editable=True
we can also restore the previous behaviour with editable=True by default, but I'm not convinced it was intended, given the hardcoding on editable=False in the said commit
although, perhaps the author just reproduced what he saw: a class attribute of editable=False ​https://github.com/django/django/commit/6002df06713cb0a7050432263527a25754190c27#diff-76b33afd71c30afb749a0de312cfe4a4578d78abae84a9d5db06388a4b1bc454L39
I'm willing to update the patch whenever a design decision is taken by authority though