Opened 8 hours ago

Last modified 7 hours ago

#35880 assigned Cleanup/optimization

Consistency and improvements to the form fields document. — at Version 4

Reported by: Antoliny Owned by: Antoliny
Component: Documentation Version: 5.1
Severity: Normal Keywords: Form Fields Document
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Antoliny)

Hello!
I created an issue by finded the wrong parts and the good parts when improved in the Form fields document.

  1. 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.


  1. 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 "Outputting forms as HTML" section of the Forms API Document.


  1. Consistency "fallback" and link message

This part is very trivial :)

The word "fallback" used in the 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 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 documentation</topics/i18n/formatting> ` for more information.

Change History (4)

comment:1 by Antoliny, 8 hours ago

Owner: set to Antoliny
Status: newassigned

comment:2 by Antoliny, 7 hours ago

Has patch: set

comment:3 by Antoliny, 7 hours ago

comment:4 by Antoliny, 7 hours ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top