Opened 5 years ago

Closed 5 years ago

#32215 closed Cleanup/optimization (invalid)

documentation possible error - minor

Reported by: JD Owned by: nobody
Component: Documentation Version: 3.1
Severity: Normal Keywords: possible documentation typo
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Mariusz Felisiak)

On ​https://docs.djangoproject.com/en/3.1/topics/forms/modelforms/#topics-forms-modelforms
Move comma in widget descriptor?
it says:

class AuthorForm(forms.Form):
    name = forms.CharField(max_length=100)
    title = forms.CharField(
        max_length=3,
        widget=forms.Select(choices=TITLE_CHOICES),
    )
    birth_date = forms.DateField(required=False)

and I THINK should be:

class AuthorForm(forms.Form):
    name = forms.CharField(max_length=100)
    title = forms.CharField(
        max_length=3,
        widget=forms.Select(choices=TITLE_CHOICES) 
    ), 
    birth_date = forms.DateField(required=False)

Change History (1)

comment:1 by Mariusz Felisiak, 5 years ago

Description: modified (diff)
Resolution: → invalid
Status: new → closed

No, it's not a typo. title is a form field, with your patch it will be a tuple.

Note: See TracTickets for help on using tickets.
Back to Top