Opened 9 years ago

Closed 8 years ago

#7274 closed (fixed)

Reset password view shouldn't require sites application

Reported by: joshrl <jrookeley@…> Owned by: nobody
Component: Contrib apps Version: master
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:

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@…> 9 years ago.

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by joshrl <jrookeley@…>

comment:1 Changed 8 years ago by Jeff Anderson

Triage Stage: UnreviewedReady for checkin

comment:2 Changed 8 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(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