#35880: Consistency and improvements to the form fields document.
-------------------------------------+-------------------------------------
     Reporter:  Antoliny             |                     Type:
                                     |  Cleanup/optimization
       Status:  new                  |                Component:
                                     |  Documentation
      Version:  5.1                  |                 Severity:  Normal
     Keywords:  Form Fields          |             Triage Stage:
  Document                           |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
 Hello!
 I created an issue by finded the wrong parts and the good parts when
 improved in the Form fields document.

 1.
 
[https://docs.djangoproject.com/en/5.1/ref/forms/fields/#django.forms.Field.required
 Field.requried section ] invalid example.

 {{{
 >>> from django import forms
 >>> f = forms.CharField()
 ...
 >> f.clean(" ")
 ' '
 }}}
 In the example passed a character string with spaces as a factor in the
 clean method to a CharField instance.
 This part is cleared space by CharField's to_python method, result is an
 empty_value so occur validation error.
 {{{
 class CharField(Field):
     ...
     def to_python(self, value):
         """Return a string."""
         if value not in self.empty_values:
             value = str(value)
             if self.strip:
                 value = value.strip() # before value: "  ", after value:
 ""
         if value in self.empty_values:
             return self.empty_value
         return value
     ...
 }}}
 However, in the current document, a string containing spaces seems to pass
 through the clean method.

 ----

 2.
 
[https://docs.djangoproject.com/en/5.1/ref/forms/fields/#django.forms.Field.label
 Field.label section] unclear part -> ("outputting forms as HTML" above)

 It is unclear what " 'Outputting forms as HTML' above" refers to in the
 Field label section.
 This part probably means the
 [https://docs.djangoproject.com/en/5.1/ref/forms/api/#outputting-forms-as-
 html "Outputting forms as HTML" section] of the Forms API Document.

 ----

 3. Consistency "fallback" and link message

 This part is very trivial :)

 The word "fallback" used in the
 
[https://docs.djangoproject.com/en/5.1/ref/forms/fields/#django.forms.Field.initial
 field initial section] are two values "fallback" and "fall back".
 {{{
 "Also note that initial values are not used as "fallback" ...
 ---
 >>> f.is_valid()
 False
 # The form does *not* fall back to using ....
 ...
 }}}

 I think it would be better to include documentation in the message set as
 a link.
 (Like the link message used in
 [https://docs.djangoproject.com/en/5.1/ref/forms/fields/#validators
 validators section].)
 {{{

 ``localize``
 ------------

 .. attribute:: Field.localize

 The ``localize`` argument enables the localization of form data input, as
 well
 as the rendered output.

 See the :doc:`format localization </topics/i18n/formatting>` documentation
 for # --> See the :doc:`format localization </topics/i18n/formatting>
 documentation` for more information.
 }}}
-- 
Ticket URL: <https://code.djangoproject.com/ticket/35880>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion visit 
https://groups.google.com/d/msgid/django-updates/01070192ea823d30-1ec800f6-af74-4046-8121-8f6ba97c6d2c-000000%40eu-central-1.amazonses.com.

Reply via email to