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:


notice errors with this documentation:

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\ -> def password_reset...) is "PasswordResetForm", so it shoud be an error.

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by russellm

  • milestone set to 1.2
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

Changed 7 years ago by timo

comment:2 Changed 7 years ago by timo

  • Has patch set
  • Triage Stage changed from Accepted to Ready for checkin

comment:3 Changed 7 years ago by jezdez

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:4 Changed 7 years ago by jezdez

(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 deleted

Milestone 1.2 deleted

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