#8402: Password change form still assumes oldforms variables --------------------------+------------------------------------------------- Reporter: mtredinnick | Owner: nobody Status: new | Milestone: 1.0 Component: Contrib apps | Version: SVN Keywords: | Stage: Unreviewed Has_patch: 0 | --------------------------+------------------------------------------------- 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. -- Ticket URL: <http://code.djangoproject.com/ticket/8402> Django Code <http://code.djangoproject.com/> The web framework for perfectionists with deadlines --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django updates" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-updates?hl=en -~----------~----~----~----~------~----~------~--~---
