diff options
| author | Tim Graham <timograham@gmail.com> | 2012-08-21 17:32:53 -0400 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2012-08-21 17:36:41 -0400 |
| commit | 42aee6ffe5f16852347e0cf069447950e9d2ef85 (patch) | |
| tree | f7d0f64074ad74983173c63bfd27b72027b384cf /docs/topics | |
| parent | b05d2f51b806da184fb8c56619f4d275a3294bb5 (diff) | |
[1.4.x] Fixed #14885 - Clarified that ModelForm cleaning may not fully complete if the form is invalid.
Thanks Ben Sturmfels for the patch.
Backport of 3fd89d99036696ba08dd2dd7e20a5b375f85d23b from master.
Diffstat (limited to 'docs/topics')
| -rw-r--r-- | docs/topics/forms/modelforms.txt | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/docs/topics/forms/modelforms.txt b/docs/topics/forms/modelforms.txt index f49c9ba6df..a29da70742 100644 --- a/docs/topics/forms/modelforms.txt +++ b/docs/topics/forms/modelforms.txt @@ -200,10 +200,13 @@ The ``is_valid()`` method and ``errors`` ---------------------------------------- The first time you call ``is_valid()`` or access the ``errors`` attribute of a -``ModelForm`` triggers form validation as well as :ref:`model validation -<validating-objects>`. This has the side-effect of cleaning the model you pass -to the ``ModelForm`` constructor. For instance, calling ``is_valid()`` on your -form will convert any date fields on your model to actual date objects. +``ModelForm`` triggers :ref:`form validation <form-and-field-validation>` as +well as :ref:`model validation <validating-objects>`. This has the side-effect +of cleaning the model you pass to the ``ModelForm`` constructor. For instance, +calling ``is_valid()`` on your form will convert any date fields on your model +to actual date objects. If form validation fails, only some of the updates +may be applied. For this reason, you'll probably want to avoid reusing the +model instance. The ``save()`` method |
