Code

Opened 6 years ago

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

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by joshrl <jrookeley@…>

comment:1 Changed 6 years ago by programmerq

  • Triage Stage changed from Unreviewed to Ready for checkin

comment:2 Changed 6 years ago by adrian

  • Resolution set to fixed
  • Status changed from new to closed

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

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.