WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 4 years ago

#15534 closed enhancement (maybelater)

Add more info to moderation emails

Reported by: jane Owned by:
Milestone: Priority: normal
Severity: minor Version: 3.0
Component: Mail Keywords: has-patch needs-testing dev-feedback close
Focuses: Cc:
PR Number:

Description

When comments have been approved, include who approved it in the email that gets sent to admin saying there's a new comment. I.e.:

"Ryan approved this comment:
[current comment info in email]"

Attachments (1)

wordpress-email-mod-update2.patch (4.1 KB) - added by chalet16 9 years ago.

Download all attachments as: .zip

Change History (8)

#1 @nacin
9 years ago

We should store rudimentary history (or at least this one data point) in commentmeta as well.

#2 @scribu
9 years ago

  • Cc scribu added

#3 @garyc40
9 years ago

  • Keywords has-patch needs-testing added

Patch looks good to me.

#4 @nacin
9 years ago

I haven't looked at how they're doing it yet, but Akismet is now storing comment history, so maybe we should make it compatible with how they're structuring the data.

#5 @chriscct7
5 years ago

  • Keywords dev-feedback close added; 3.2-early removed

This would probably be better left to a plugin. If core wants to store who approved a comment, I would suggest also including a new db column in the column table to store when it was approved, or last modified, or both. Seems those would go hand in hand.

#6 @DrewAPicture
5 years ago

FWIW, it looks like Akismet is storing their history as an array of data in comment meta:

if ( is_object( $current_user ) && isset( $current_user->user_login ) )
        $user = $current_user->user_login;

$event = array(
        'time'    => self::_get_microtime(),
        'message' => $message,
        'event'   => $event,
        'user'    => $user,
);

// $unique = false so as to allow multiple values per comment
$r = add_comment_meta( $comment_id, 'akismet_history', $event, false );

where the value of $event passed to the array is any number of things, including 'check-spam|ham|error', 'wp-blacklisted', "status-changed-$approved", 'report-spam|ham', 'cron-retry', etc.

#7 @wonderboymusic
4 years ago

  • Milestone Future Release deleted
  • Resolution set to maybelater
  • Status changed from new to closed

This could be done in a plugin like Akismet (which is doing it), I don't see us jumping to add the functionality to core right now

Note: See TracTickets for help on using tickets.