From 874053edf92c02e1ab155bdf7491b08f9863dce8 Mon Sep 17 00:00:00 2001 From: Tim Graham Date: Mon, 30 Jun 2014 16:30:57 -0400 Subject: [PATCH] Fixed #21942 -- Moved Form.clean() to form API docs. Thanks cjerdonek for the suggestion. --- docs/ref/forms/api.txt | 6 ++++++ docs/ref/forms/validation.txt | 24 +++++++++++++----------- 2 files changed, 19 insertions(+), 11 deletions(-) diff --git a/docs/ref/forms/api.txt b/docs/ref/forms/api.txt index 8909d5c916..954d14e184 100644 --- a/docs/ref/forms/api.txt +++ b/docs/ref/forms/api.txt @@ -71,6 +71,12 @@ should consider its data immutable, whether it has data or not. Using forms to validate data ---------------------------- +.. method:: Form.clean() + +Implement a ``clean()`` method on your ``Form`` when you must add custom +validation for fields that are interdependent. See +:ref:`validating-fields-with-clean` for example usage. + .. method:: Form.is_valid() The primary task of a :class:`Form` object is to validate data. With a bound diff --git a/docs/ref/forms/validation.txt b/docs/ref/forms/validation.txt index 6271749845..676f176910 100644 --- a/docs/ref/forms/validation.txt +++ b/docs/ref/forms/validation.txt @@ -1,3 +1,5 @@ +.. currentmodule:: django.forms + .. _form-and-field-validation: Form and field validation @@ -82,7 +84,7 @@ overridden: called, you also have access to the form's errors attribute which contains all the errors raised by cleaning of individual fields. - Note that any errors raised by your ``Form.clean()`` override will not + Note that any errors raised by your :meth:`Form.clean()` override will not be associated with any field in particular. They go into a special "field" (called ``__all__``), which you can access via the :meth:`~django.forms.Form.non_field_errors` method if you need to. If you @@ -98,8 +100,8 @@ These methods are run in the order given above, one field at a time. That is, for each field in the form (in the order they are declared in the form definition), the ``Field.clean()`` method (or its override) is run, then ``clean_()``. Finally, once those two methods are run for every -field, the ``Form.clean()`` method, or its override, is executed whether or not -the previous methods have raised errors. +field, the `:meth:`Form.clean()` method, or its override, is executed whether +or not the previous methods have raised errors. Examples of each of these methods are provided below. @@ -325,25 +327,25 @@ write a cleaning method that operates on the ``recipients`` field, like so:: return data Sometimes you may want to add an error message to a particular field from the -form's ``clean()`` method, in which case you can use +form's :meth:`~Form.clean()` method, in which case you can use :meth:`~django.forms.Form.add_error()`. Note that this won't always be appropriate and the more typical situation is to raise a ``ValidationError`` from , which is turned into a form-wide error that is available through the :meth:`Form.non_field_errors() ` method. +.. _validating-fields-with-clean: + Cleaning and validating fields that depend on each other ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -.. method:: django.forms.Form.clean() - Suppose we add another requirement to our contact form: if the ``cc_myself`` field is ``True``, the ``subject`` must contain the word ``"help"``. We are performing validation on more than one field at a time, so the form's -``clean()`` method is a good spot to do this. Notice that we are talking about -the ``clean()`` method on the form here, whereas earlier we were writing a -``clean()`` method on a field. It's important to keep the field and form -difference clear when working out where to validate things. Fields are single -data points, forms are a collection of fields. +:meth:`~Form.clean()` method is a good spot to do this. Notice that we are +talking about the ``clean()`` method on the form here, whereas earlier we were +writing a ``clean()`` method on a field. It's important to keep the field and +form difference clear when working out where to validate things. Fields are +single data points, forms are a collection of fields. By the time the form's ``clean()`` method is called, all the individual field clean methods will have been run (the previous two sections), so