Opened 16 years ago

Closed 16 years ago

#7274 closed (fixed)

Reset password view shouldn't require sites application

Reported by: joshrl <jrookeley@…> Owned by: nobody
Component: Contrib apps Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The reset password view (django.contrib.auth.views.password_reset) requires that the django.contrib.sites application is installed. Perhaps there is a good reason for this, but I couldn't think of one so I created a patch to use RequestSite, if sites isn't installed.

Attachments (1)

reset_password_site_not_required_r7540.diff​ (914 bytes ) - added by joshrl <jrookeley@…> 16 years ago.

Download all attachments as: .zip

Change History (3)

by joshrl <jrookeley@…>, 16 years ago

Attachment: reset_password_site_not_required_r7540.diff​ added

comment:1 by Jeff Anderson, 16 years ago

Triage Stage: Unreviewed → Ready for checkin

comment:2 by Adrian Holovaty, 16 years ago

Resolution: → fixed
Status: new → closed

(In [7748]) Fixed #7274 -- Improved the django.contrib.auth password_reset() view not to require the Sites framework. Thanks, joshrl

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