WordPress.org
Get WordPress

Make WordPress Core

Opened 21 years ago

Closed 19 years ago

#1096 closed defect (bug) (fixed)

Resetting admin password results in extra notification e-mail

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

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 21 years ago.
1096.diff​ (1.0 KB ) - added by rob1n 19 years ago.

Download all attachments as: .zip

Change History (12)

#1 @MC_incubus
21 years ago

  • Patch → No

#2 @idanso
21 years ago

Trivial fix attached(untested)

#3 @ryan
21 years ago

  • Patch No → Yes

@MC_incubus
21 years ago

#4 @filosofo
20 years ago

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

Patch works for me.

#5 @rob1n
19 years ago

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

Works great for me.

#6 @rob1n
19 years ago

  • Keywords needs-patch added; has-patch commit removed
  • Milestone → 2.2
  • Version 2.0.4 → 2.1

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

@rob1n
19 years ago

#7 @rob1n
19 years ago

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

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

#8 @rob1n
19 years ago

  • Keywords needs-testing removed

#9 @rob1n
19 years ago

  • Keywords commit added; dev-feedback removed

#10 @markjaquith
19 years ago

  • Resolution → fixed
  • Status assigned → 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.