Opened 4 years ago

Closed 4 years ago

#8695 closed feature request (fixed)

Send email for approved comments once

Reported by: Denis-de-Bernardy Owned by: anonymous
Priority: normal Milestone: 2.8
Component: Administration Version: 2.7
Severity: minor Keywords: has-patch
Cc:

Description

Shouldn't this be the default behavior in WP?

http://wordpress.org/extend/plugins/filter-email-notifications/

Attachments (2)

8695.diff (673 bytes) - added by Denis-de-Bernardy 4 years ago.
wp-includes--pluggable.php.diff (742 bytes) - added by josephscott 4 years ago.

Download all attachments as: .zip

Change History (12)

  • Milestone changed from 2.7.1 to 2.8
  • Type changed from defect (bug) to feature request

Not really a bug, as the current behaviour could be useful in some cases (I.E. multiple people sharing a login/email.)

Could be added as an option I suppose...

Honestly, I doubt that sending an email would make sense in the use-case you mention. If I've an editor who uses my account, I probably don't care to receive an email when he approves a comment.

The case I was considering was

  1. Email is sent to say comment is awaiting moderation (this goes to a list or shared mailbox)
  2. Someone logs on to approve said comment.
  3. A further mail is sent out, the other people on that list then know that they don't need to moderate that comment.

I'm not fussed either way. As I said, the idea has potential as an option.

  • Milestone changed from 2.8 to Future Release
  • Keywords has-patch added
  • Milestone changed from Future Release to 2.8

comment:6   ryan4 years ago

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

(In [11057]) Don't notify post author of own comments. Props Denis-de-Bernardy. fixes #8695

comment:7   ryan4 years ago

(In [11057]) Don't notify post author of own comments. Props Denis-de-Bernardy. fixes #8695

  • Resolution fixed deleted
  • Status changed from closed to reopened

The initial change for this was incorrect. The check should be explicitly for the post author user ID and the comment user ID. I've included a simple patch to make this change.

Currently in the case where the post author is moderating comments the current check always matches because it's looking at the current user.

very correct.

/me bangs his head on a wall.

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

(In [11387]) Don't notify post author of own comments. PrProps josephscott. fixes #8695

Note: See TracTickets for help on using tickets.