Make WordPress Core

Opened 19 years ago

Closed 17 years ago

#1096 closed defect (bug) (fixed)

Resetting admin password results in extra notification e-mail

Reported by: mc_incubus's profile MC_incubus Owned by: rob1n's profile rob1n
Milestone: 2.2 Priority: normal
Severity: trivial Version: 2.1
Component: Administration Keywords: has-patch commit
Focuses: Cc:

Description

When a user resets their password, the administrator is notified.

When the administrator resets their password, they are notified that they have reset their password. This results in a total of 3 e-mails:
1) sending of the "change your password" link
2) sending of the new password
3) telling them that their password was reset

I think the third one can be eliminated for the admin.

Attachments (2)

fix_1096.patch (1.2 KB) - added by MC_incubus 19 years ago.
1096.diff (1.0 KB) - added by rob1n 17 years ago.

Download all attachments as: .zip

Change History (12)

#1 @MC_incubus
19 years ago

  • Patch set to No

#2 @idanso
19 years ago

Trivial fix attached(untested)

#3 @ryan
19 years ago

  • Patch changed from No to Yes

#4 @filosofo
18 years ago

  • Keywords bg|has-patch added
  • Owner changed from anonymous to filosofo
  • Version changed from 1.5 to 2.0.4

Patch works for me.

#5 @rob1n
17 years ago

  • Keywords has-patch commit added; bg|has-patch removed
  • Owner changed from filosofo to rob1n
  • Status changed from new to assigned

Works great for me.

#6 @rob1n
17 years ago

  • Keywords needs-patch added; has-patch commit removed
  • Milestone set to 2.2
  • Version changed from 2.0.4 to 2.1

Never mind. Does *not* apply cleanly for me.

@rob1n
17 years ago

#7 @rob1n
17 years ago

  • Keywords has-patch needs-testing dev-feedback added; needs-patch removed

Okay, I made a new patch on r4879. Devs?

#8 @rob1n
17 years ago

  • Keywords needs-testing removed

#9 @rob1n
17 years ago

  • Keywords commit added; dev-feedback removed

#10 @markjaquith
17 years ago

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

(In [4902]) refrain from notifying the administrator of administrator password changes (it is redundant). Props rob1n for the patch refresh. fixes #1096

Note: See TracTickets for help on using tickets.