Fixed #27821 -- Clarified docs of the return value of Form.clean_<fieldname>().

Thanks Christian Ullrich for the report and review.
This commit is contained in:
Tim Graham 2017-02-08 08:38:08 -05:00
parent b27166b769
commit 8863c475c5
1 changed files with 5 additions and 4 deletions

View File

@ -69,8 +69,9 @@ overridden:
formfield-specific piece of validation and, possibly, formfield-specific piece of validation and, possibly,
cleaning/normalizing the data. cleaning/normalizing the data.
This method should return the cleaned value obtained from ``cleaned_data``, The return value of this method replaces the existing value in
regardless of whether it changed anything or not. ``cleaned_data``, so it must be the field's value from ``cleaned_data`` (even
if this method didn't change it) or a new cleaned value.
* The form subclass's ``clean()`` method can perform validation that requires * The form subclass's ``clean()`` method can perform validation that requires
access to multiple form fields. This is where you might put in checks such as access to multiple form fields. This is where you might put in checks such as
@ -315,8 +316,8 @@ write a cleaning method that operates on the ``recipients`` field, like so::
if "fred@example.com" not in data: if "fred@example.com" not in data:
raise forms.ValidationError("You have forgotten about Fred!") raise forms.ValidationError("You have forgotten about Fred!")
# Always return the cleaned data, whether you have changed it or # Always return a value to use as the new cleaned data, even if
# not. # this method didn't change it.
return data return data
.. _validating-fields-with-clean: .. _validating-fields-with-clean: