Opened 16 years ago

Closed 16 years ago

Last modified 12 years ago

#8402 closed (fixed)

Password change form still assumes oldforms variables

Reported by: Malcolm Tredinnick Owned by:
Component: Contrib apps Version: dev
Severity: Keywords: aug22sprint
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This is a django.contrib.auth bug, but the relevant template is "django/contrib/admin/templates/registrations/password_change_form.html": this template uses the oldforms-style html_error_list instead of .errors to display error messsages.

The reason this isn't a simple trivial change is because a test should actually be written to test that form and ensure the problem doesn't occur again. To do that:

  • put a new unittest class in django.contrib.auth.tests.views
  • modify django.contrib.auth.tests.__init__ to include the class
  • hook up a /login/ URL in django.contrib.auth.urls, since you can't change the password without logging in.
  • work out a password for the testuser in the fixture data and update the fixture data's password hash. Then document what this password is for other test users somehow (in the fixture directory maybe?). It might be possible to guess the current password; I haven't tried. But just creating the hash for a know password (e.g. "test" would be fine.
  • At least test that trying to change a password and failing displays the error messages.

Attachments (1)

8402.add_password_change_tests_and_update_default_template_to_newforms.diff (6.2 KB ) - added by mtrichardson 16 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by Malcolm Tredinnick, 16 years ago

Triage Stage: UnreviewedAccepted

comment:2 by mtrichardson, 16 years ago

Owner: changed from nobody to mtrichardson

comment:3 by mtrichardson, 16 years ago

The previous diff adds tests in addition to fixing the problem.

comment:4 by mtrichardson, 16 years ago

Owner: mtrichardson removed

comment:5 by mtrichardson, 16 years ago

Keywords: aug22sprint added

comment:6 by Malcolm Tredinnick, 16 years ago

(In [8496]) Updated password change template (in admin templates) to use current form
variables (rather than oldforms stuff). Patch from Mike Richardson. Refs #8402.

comment:7 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: newclosed

Tests were committed in [8497], which is the last piece of this.

comment:8 by Jacob, 12 years ago

milestone: 1.0

Milestone 1.0 deleted

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