Opened 17 years ago

Closed 17 years ago

#7902 closed (fixed)

/password_change/ is 404

Reported by: Thomas Kerpe Owned by: nobody
Component: *.djangoproject.com Version:
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

After a password reset via: ​http://www.djangoproject.com/accounts/password/reset/ I got an e-mail which contains a link to ​http://www.djangoproject.com/password_change/ which returns 404.
A working link would be ​http://www.djangoproject.com/accounts/password/change/

Attachments (2)

password_email_link.diff​ (579 bytes ) - added by Michael Newman 17 years ago.
password_email_url.diff​ (611 bytes ) - added by Bob Thomas <bthomas@…> 17 years ago.
Replace hardcoded link with url tag

Download all attachments as: .zip

Change History (8)

comment:1 by Michael Newman, 17 years ago

Component: Django Web site → Admin interface
Has patch: set
milestone: → 1.0
Triage Stage: Unreviewed → Accepted

This is pretty straight forward, I added a patch for simplicity. This hard coded link should be replaced with a url tag, once that is possible.

by Michael Newman, 17 years ago

Attachment: password_email_link.diff​ added

comment:2 by Bob Thomas <bthomas@…>, 17 years ago

Cc: bthomas@… added

{% url django.contrib.auth.views.password_change %} works on my site.

by Bob Thomas <bthomas@…>, 17 years ago

Attachment: password_email_url.diff​ added

Replace hardcoded link with url tag

comment:3 by Thomas Kerpe, 17 years ago

Component: Admin interface → Django Web site
milestone: 1.0

The bug was reported against Django Web site not against admin, also admin is definitively wrong. Setting it back to Django Web site.

The template-problem is not in admin but just a missing password_reset_email.txt template in djangoproject website.

Bob Thomas approach using {% url django.contrib.auth.views.password_change %} in the contrib app template is ok and more generic.
An alternative would be to just add the corrected template to django_website/templates/registration/password_reset_email.txt

comment:4 by Bob Thomas <bthomas@…>, 17 years ago

This may be obsolete now, see ticket:8129

comment:5 by Bob Thomas, 17 years ago

Cc: bthomas@… removed

comment:6 by Jacob, 17 years ago

Resolution: → fixed
Status: new → closed

(In [8925]) [djangoproject.com] a bunch of fixes to the website. Fixes #7271, #7459, #7902, #8271, #7949, #8845.

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