diff --git a/django/forms/forms.py b/django/forms/forms.py index e2ed1a4ac13..25e081e324d 100644 --- a/django/forms/forms.py +++ b/django/forms/forms.py @@ -373,14 +373,13 @@ class BaseForm(object): def _clean_fields(self): for name, field in self.fields.items(): - if field.disabled: - # Initial values are supposed to be clean - self.cleaned_data[name] = self.initial.get(name, field.initial) - continue # value_from_datadict() gets the data from the data dictionaries. # Each widget type knows how to retrieve its own data, because some # widgets split data over several HTML fields. - value = field.widget.value_from_datadict(self.data, self.files, self.add_prefix(name)) + if field.disabled: + value = self.initial.get(name, field.initial) + else: + value = field.widget.value_from_datadict(self.data, self.files, self.add_prefix(name)) try: if isinstance(field, FileField): initial = self.initial.get(name, field.initial) diff --git a/tests/model_forms/tests.py b/tests/model_forms/tests.py index 9de1a7f24b5..daf36c8fc2f 100644 --- a/tests/model_forms/tests.py +++ b/tests/model_forms/tests.py @@ -1519,6 +1519,21 @@ class ModelChoiceFieldTests(TestCase): '' ) + def test_disabled_modelchoicefield(self): + class ModelChoiceForm(forms.ModelForm): + author = forms.ModelChoiceField(Author.objects.all(), disabled=True) + + class Meta: + model = Book + fields = ['author'] + + book = Book.objects.create(author=Writer.objects.create(name='Test writer')) + form = ModelChoiceForm({}, instance=book) + self.assertEqual( + form.errors['author'], + ['Select a valid choice. That choice is not one of the available choices.'] + ) + def test_modelchoicefield_iterator(self): """ Iterator defaults to ModelChoiceIterator and can be overridden with diff --git a/tests/postgres_tests/test_json.py b/tests/postgres_tests/test_json.py index b88d103761f..0165cabfd92 100644 --- a/tests/postgres_tests/test_json.py +++ b/tests/postgres_tests/test_json.py @@ -260,14 +260,6 @@ class TestFormField(PostgreSQLTestCase): form_field = model_field.formfield() self.assertIsInstance(form_field, forms.JSONField) - def test_formfield_disabled(self): - class JsonForm(Form): - name = CharField() - jfield = forms.JSONField(disabled=True) - - form = JsonForm({'name': 'xyz', 'jfield': '["bar"]'}, initial={'jfield': ['foo']}) - self.assertIn('["foo"]', form.as_p()) - def test_prepare_value(self): field = forms.JSONField() self.assertEqual(field.prepare_value({'a': 'b'}), '{"a": "b"}')