Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#31906 closed Bug (fixed)

Possible ModelChoiceIterator docs typo

Reported by: Claude Paroz Owned by: Ninad Kulkrni
Component: Documentation Version: 3.1
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

In https://docs.djangoproject.com/fr/3.1/ref/forms/fields/#modelchoiceiterator, under field, it is written:

The instance of ModelChoiceField or ModelMultipleChoiceField to iterator and yield choice.

Should iterator be iterate, or did I miss something? And choice or choices?

Change History (10)

comment:1 by Carlton Gibson, 4 years ago

Triage Stage: UnreviewedAccepted

Yes, should be to iterate and yield choices. I’d say.

comment:2 by Mariusz Felisiak, 4 years ago

Easy pickings: set

comment:3 by KESHAV KUMAR, 4 years ago

Can I take this issue ?

comment:4 by Claude Paroz, 4 years ago

Don't ask, just take it when it is not already assigned.

comment:5 by KESHAV KUMAR, 4 years ago

Owner: changed from nobody to KESHAV KUMAR
Status: newassigned

comment:6 by Ninad Kulkrni, 4 years ago

Owner: changed from KESHAV KUMAR to Ninad Kulkrni

Documentation issue is fixed in the below pull request.

https://github.com/django/django/pull/13335

comment:7 by Ninad Kulkrni, 4 years ago

The documentation issue is fixed in the below pull request.

https://github.com/django/django/pull/13336

comment:8 by Mariusz Felisiak, 4 years ago

Has patch: set
Triage Stage: AcceptedReady for checkin

comment:9 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

Resolution: fixed
Status: assignedclosed

In 927bd888:

Fixed #31906 -- Fixed typo in docs/ref/forms/fields.txt.

comment:10 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

In 8fb24a51:

[3.1.x] Fixed #31906 -- Fixed typo in docs/ref/forms/fields.txt.

Backport of 927bd888ddea8bf37aec5515a7367fe8aeb3d1b4 from master

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