#12097 closed (fixed)
ModelChoiceField does not document required queryset parameter properly
| Reported by: | Preston Holmes | Owned by: | Jason Yosinski |
|---|---|---|---|
| Component: | Documentation | Version: | 1.1 |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
http://docs.djangoproject.com/en/dev/ref/forms/fields/#modelchoicefield
The reference to ModelChoiceField.queryset comes before the section on ModelChoiceField and it should come after - also it should be more explicit that queryset is a required parameter
Attachments (1)
Change History (9)
comment:1 by , 14 years ago
| milestone: | → 1.2 |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 14 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 14 years ago
| Has patch: | set |
|---|
by , 14 years ago
| Attachment: | ModelChoiceField_doc.diff added |
|---|
follow-up: 5 comment:4 by , 14 years ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
on re-reading the original version it is technically complete. However I think the order in the included patch is clearer.
comment:5 by , 14 years ago
Replying to anonymous:
on re-reading the original version it is technically complete. However I think the order in the included patch is clearer.
Sorry - this comment was by me - the original reporter of this item - didn't see I was no longer logged in
comment:6 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:7 by , 14 years ago
Note:
See TracTickets
for help on using tickets.
Patch submitted and ready for review.