Add more info to moderation emails

[jane]

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]"

[nacin]

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

[garyc40]

Patch looks good to me.

[nacin]

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.

[chriscct7]

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.

[DrewAPicture]

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.

[wonderboymusic]

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