BuddyPress.org

@shanebpx: Thanks for that suggestion — it’s beautifully simple. Unfortunately, it appear as though $args['content'] is empty? (But there’s definitely content in the test activity I am deleting!)

For debugging, I tried $message .= print_r($args, true);, which gave:

Array
(
[id] => 31
[action] =>
[content] =>
[component] =>
[type] =>
[primary_link] =>
[user_id] => 52
[item_id] =>
[secondary_item_id] =>
[date_recorded] =>
[hide_sitewide] =>
)

I admit I’m a little bewildered as to why all those values appear empty.

Ah, it’s already been logged and fixed for 1.6.2. I do wish 1.6.2 had been released already! Would have saved me a few frustrating hours.

Revisiting this thread with a new issue!

My function has been working well, except I’ve recently discovered a small flaw: if an administrator deletes a user, then the user receives an email informing him or her that his activity has been removed for each and every thing he or she has ever posted! For a prolific user, that could be hundreds or thousands of emails. Not good!

This is by virtue of the user-delete process — it appears as though BuddyPress deletes each activity individually, and then WordPress deletes the user account.

For reference, my function is:

`function pa_send_notification_email( $args ) {
// If current user deletes his/her own activity, do nothing
// otherwise construct and send email
if ($args[‘user_id’] != bp_loggedin_user_id()) {
$ud = get_userdata($args[‘user_id’]);
$to = $ud->user_email;
// Build the message
if (($to) && ($to != ”)) {
$subject = ‘Your post has been removed’;
$message = ‘A moderator has removed your post. If you want to know why, please forward this message to the moderation team.
‘;
$message .= ‘Removed content:
‘;
global $wpdb;
$message .= wpautop( $wpdb->get_var( $wpdb->prepare( “SELECT content FROM “.$wpdb->prefix.”bp_activity WHERE id = %d;”, $args[‘id’] ) ) );
$message .= ”;

//Send email
wp_mail($to, $subject, $message, ‘content-type: text/html’ );
}
}
}
add_action( ‘bp_before_activity_delete’, ‘pa_send_notification_email’);`

So what I think I need is an extra check in my function, to make sure we are not about to delete the user. I’ve thought about a secondary function to hook into the `delete_user` action to set a global variable, and then checking for this variable in my `pa_send_notification_email()` function.

But I am not sure how to go about this. Does anyone have any thoughts?

Okay, phew, it wasn’t as serious as I first thought. This now won’t send an email if the content is empty (which is, apparently, the case when a user is being deleted):

`function pa_send_notification_email( $args ) {
// If current user deletes his/her own activity, do nothing
if (($args[‘user_id’] != bp_loggedin_user_id())) {
$ud = get_userdata($args[‘user_id’]);
$to = $ud->user_email;
// Build the message
if (($to) && ($to != ”)) {
global $wpdb;
$content = $wpdb->get_var( $wpdb->prepare( “SELECT content FROM “.$wpdb->prefix.”bp_activity WHERE id = %d;”, $args[‘id’] ) );
if ($content != ”) {
$subject = ‘Your post has been removed’;
$message = ‘A moderator has removed your post. If you want to know why, please forward this message to the moderation team.
‘;
$message .= ‘Removed content:
‘;

$message .= wpautop( $content );
$message .= ”;

//Send email
wp_mail($to, $subject, $message, ‘content-type: text/html’ );
}
}
}
}
add_action( ‘bp_before_activity_delete’, ‘pa_send_notification_email’);`