Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#24250 closed Bug (fixed)

Class FormMixin does not provide "form" variable in centext.

Reported by: 4544fa8d Owned by: nobody
Component: Documentation Version: 1.7
Severity: Normal Keywords: FormMixin form context
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

Class FormMixin does not provide "form" variable in centext. Documentation states that it should

https://docs.djangoproject.com/en/1.7/ref/class-based-views/mixins-editing/#formmixin

Attachments (1)

24250.diff (726 bytes) - added by Tim Graham 7 years ago.

Download all attachments as: .zip

Change History (12)

Changed 7 years ago by Tim Graham

Attachment: 24250.diff added

comment:1 Changed 7 years ago by Tim Graham

Component: Generic viewsDocumentation
Has patch: set
Triage Stage: UnreviewedAccepted

Added a patch to correct the documentation.

Other errors corrected:

  • get_context_data() is not implemented on the mixin.
  • The sentence "Views mixing FormMixin must provide an implementation of form_valid and form_invalid." doesn't make sense as these methods have default implementations on the mixin.

comment:2 Changed 7 years ago by Claude Paroz

get_context_data() is inherited from ContextMixin (which only adds the view in the context). Does this warrants its presence in FormMixin docs?

comment:3 Changed 7 years ago by Tim Graham

Triage Stage: AcceptedReady for checkin

Yes, I'll add that. Thanks for the review.

comment:4 Changed 7 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: newclosed

In 5d193d042a3cc49033f0e8b5125913650d779496:

Fixed #24250 -- Corrected mistakes in FormMixin docs.

comment:5 Changed 7 years ago by Tim Graham <timograham@…>

In 9316a04a9cb8e2f57068c1655c18510f943120e4:

[1.6.x] Fixed #24250 -- Corrected mistakes in FormMixin docs.

Backport of 5d193d042a3cc49033f0e8b5125913650d779496 from master

comment:6 Changed 7 years ago by Tim Graham <timograham@…>

In 6454716264163b4693c03168ec56ea894ea511f2:

[1.8.x] Fixed #24250 -- Corrected mistakes in FormMixin docs.

Backport of 5d193d042a3cc49033f0e8b5125913650d779496 from master

comment:7 Changed 7 years ago by Tim Graham <timograham@…>

In 9c456ea8ff49eb8323f0b776935444197b6d75f2:

[1.7.x] Fixed #24250 -- Corrected mistakes in FormMixin docs.

Backport of 5d193d042a3cc49033f0e8b5125913650d779496 from master

comment:8 Changed 7 years ago by Tim Graham <timograham@…>

In 9a391fbd6102176d576aa96ed3aeee9e9661b455:

Fixed broken links in class based views flattend index; refs #24250.

comment:9 Changed 7 years ago by Tim Graham <timograham@…>

In 5fdd74f4be4aa6ce54021492e1b05ebf3770463d:

[1.8.x] Fixed broken links in class based views flattend index; refs #24250.

Backport of 9a391fbd6102176d576aa96ed3aeee9e9661b455 from master

comment:10 Changed 7 years ago by Tim Graham <timograham@…>

In 346451164e56b01d8174bb21e3354f47db2764ae:

[1.7.x] Fixed broken links in class based views flattend index; refs #24250.

Backport of 9a391fbd6102176d576aa96ed3aeee9e9661b455 from master

comment:11 Changed 7 years ago by Tim Graham <timograham@…>

In 5104d935045c85d6e3a7354dcb74b5009e5794f2:

[1.6.x] Fixed broken links in class based views flattend index; refs #24250.

Backport of 9a391fbd6102176d576aa96ed3aeee9e9661b455 from master

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