Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#12895 closed (fixed)

"SetPasswordForm" should be "PasswordResetForm"

Reported by: kun Owned by: nobody
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: UI/UX:

Description

notice errors with this documentation:
http://docs.djangoproject.com/en/1.1/topics/auth/#topics-auth

search this keyword:
views.password_reset(request[, is_admin_site, template_name, email_template_name, password_reset_form, token_generator, post_reset_redirect])

in "Optional arguments:" -> "password_reset_form" -> "Defaults to SetPasswordForm." <<< it shoud be "PasswordResetForm"

the source code(django\contrib\auth\views.py -> def password_reset...) is "PasswordResetForm", so it shoud be an error.

Attachments (1)

12895.diff (637 bytes) - added by Tim Graham 7 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by Russell Keith-Magee

milestone: 1.2
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

Changed 7 years ago by Tim Graham

Attachment: 12895.diff added

comment:2 Changed 7 years ago by Tim Graham

Has patch: set
Triage Stage: AcceptedReady for checkin

comment:3 Changed 7 years ago by Jannis Leidel

Resolution: fixed
Status: newclosed

(In [12457]) Fixed #12895 - Fixed typo in auth docs. Thanks Timo.

comment:4 Changed 7 years ago by Jannis Leidel

(In [12477]) [1.1.X] Fixed #12895 - Fixed typo in auth docs. Thanks Timo.

Backported from r12457.

comment:5 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

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