Exclude Current user email and send notification

Question

I am currently building a wordpress plugin to send email notification to list of users roles who comments on a post.

For Example:

If a user with "subscriber" role comments on a post all the other users who have "subscriber" role gets email alert.Currently i have achieved this through the following code

if( current_user_can('subscriber'))
{
$user_query = new WP_User_Query( array( 'role' => 'subscriber');
                foreach ( $user_query->results as $user ) 
                {
                    $mail=$user->user_email;


                    $subject  ="this is example";
                    $message  = "";
                    $message .= "$post_permalink\n";

                    wp_mail($user->user_email,  $subject,$message);         
}
}

The above code sends email to all the users who have "subscriber" role.

Output:

It gets even the current user who comments on the post and send the email notification.

Problem to be solved:

How can i exclude the current user email and send only notification to other users those who have "subscriber" role .

Answer 1

First, you should know that current_user_can() only accepts capabilities, not roles, so you are using it wrong and you can end up with unexpected results.

That being said, to exlude users form WP_User_Query you can use the exclude parameter:

// Get current user data
$user = wp_get_current_user();

// Check if current user is a subscriber
if( ! empty( $user ) && in_array( "subscriber", (array) $user->roles ) ) {

    $user_query = new WP_User_Query( array(
                                           'role'   => 'subscriber',
                                           // exlude current user
                                           'exclude' => array( $user->ID ),
                                     )
    );

    $subscribers = $user_query->get_results();

    if( ! empty( $subscribers ) ) {
        foreach ( $subscribers as $user )  {

            $mail = $user->user_email;
            $subject  ="this is example";
            $message  = "";
            $message .= "$post_permalink\n";

            wp_mail( $mail, $subject, $message);

        }
    }

}

Answer 2

Following should works for you:-

 if( current_user_can('read')) { 
    $exclude_user = array();
    $exclude_user[] =get_current_user_id();
    $args = array(
    'role' => 'subscriber',
    'exclude' => $exclude_user
    );
    $user_query = new WP_User_Query( $args); 
    foreach ( $user_query->results as $user ) { $mail=$user->user_email;

                    $subject  ="this is example";
                    $message  = "";
                    $message .= "$post_permalink\n";

                    wp_mail($user->user_email,  $subject,$message);         
    }


}