diff options
author | yakimka <ss.yakim@gmail.com> | 2021-07-15 21:27:59 +0300 |
---|---|---|
committer | Carlton Gibson <carlton.gibson@noumenal.es> | 2021-07-21 11:23:43 +0200 |
commit | 134638176080eba48b9b3a2e484bfee1cb8beaec (patch) | |
tree | 6851d418307e9e62a9a8042aea191d62ba88d6b1 | |
parent | 05e997c40490776b6c0e13806800d6b125e53107 (diff) | |
download | django-134638176080eba48b9b3a2e484bfee1cb8beaec.tar.gz |
[3.2.x] Fixed #32949 -- Restored invalid number handling in DecimalField.validate().
DecimalField must itself validate() values, such as NaN, which cannot be
passed to validators, such as MaxValueValidator, during the
run_validators() phase.
Regression in cc3d24d7d577f174937a0744d886c4c7123cfa85.
Backport of c542d0a07237033225c1d57337ca9474a00648f2 from main
-rw-r--r-- | django/forms/fields.py | 11 | ||||
-rw-r--r-- | docs/releases/3.2.6.txt | 4 | ||||
-rw-r--r-- | tests/forms_tests/field_tests/test_decimalfield.py | 2 |
3 files changed, 15 insertions, 2 deletions
diff --git a/django/forms/fields.py b/django/forms/fields.py index 17c7956b53..0214d60c1c 100644 --- a/django/forms/fields.py +++ b/django/forms/fields.py @@ -350,6 +350,17 @@ class DecimalField(IntegerField): raise ValidationError(self.error_messages['invalid'], code='invalid') return value + def validate(self, value): + super().validate(value) + if value in self.empty_values: + return + if not value.is_finite(): + raise ValidationError( + self.error_messages['invalid'], + code='invalid', + params={'value': value}, + ) + def widget_attrs(self, widget): attrs = super().widget_attrs(widget) if isinstance(widget, NumberInput) and 'step' not in widget.attrs: diff --git a/docs/releases/3.2.6.txt b/docs/releases/3.2.6.txt index 426b9c6797..222e169901 100644 --- a/docs/releases/3.2.6.txt +++ b/docs/releases/3.2.6.txt @@ -9,4 +9,6 @@ Django 3.2.6 fixes several bugs in 3.2.5. Bugfixes ======== -* ... +* Fixed a regression in Django 3.2 that caused a crash validating ``"NaN"`` + input with a ``forms.DecimalField`` that had a specified ``max_value`` + (:ticket:`32949`). diff --git a/tests/forms_tests/field_tests/test_decimalfield.py b/tests/forms_tests/field_tests/test_decimalfield.py index d999dab3f3..df5b9052b2 100644 --- a/tests/forms_tests/field_tests/test_decimalfield.py +++ b/tests/forms_tests/field_tests/test_decimalfield.py @@ -49,7 +49,7 @@ class DecimalFieldTest(FormFieldAssertionsMixin, SimpleTestCase): self.assertIsNone(f.min_value) def test_enter_a_number_error(self): - f = DecimalField(max_digits=4, decimal_places=2) + f = DecimalField(max_value=1, max_digits=4, decimal_places=2) values = ( '-NaN', 'NaN', '+NaN', '-sNaN', 'sNaN', '+sNaN', |