CODEWITHSUNDEEP
phpwordpresswordpress-themingwordpress-plugin-creationemail-delivery
Æthelstan
Æthelstan

Reputation: 1043

WordPress: I need to send users a notification when their role is changed, my code snippet doesn't work

I'm building a members style site using WordPress. When a user registers their default role is "subscriber" once we manually approve their account we change the user role to "private_event_member", we need to send the user an email to tell them we have changed their role. I found the following code snippet and added it to the functions.php file

function user_role_update( $user_id, $new_role ) {
        $site_url = get_bloginfo('wpurl');
        $user_info = get_userdata( $user_id );
        $to = $user_info->user_email;
        $subject = "Role changed: ".$site_url."";
        $message = "Hello " .$user_info->display_name . " your role has changed on ".$site_url.", congratulations you are now an " . $new_role;
        wp_mail($to, $subject, $message);
}
add_action( 'set_user_role', 'user_role_update', 10, 2);

This failed to send the email as expected. So to be sure I decided to install a plugin called WP Mail Log and then also WP Mail SMTP, I configured the Sendblue SMTP option. I've tested this and all other emails like for example user registration notifications and new orders are being sent and recorded successfully in the logs, these are being received. The above mentioned code however seems to do nothing.

This seems to be a widely used piece of code that should work so can anyone explain to me why this snippet behaves differently from other mail on the server? It doesn't even appear in the sending logs so as far as I can see it's not doing anything at all. Has the set_user_role action I'm hooking into changed? What could be the cause?

Any help much appreciate!

Upvotes: 0

Views: 1746

Answers (2)

Jewel Wildmoon
Jewel Wildmoon

Reputation: 1

I know this is late, but I figured I'll add if anyone else is getting the following issue. I don't have enough rep to comment on MartinMirchev's Answer, so I'll have to post a separate one.

Adding onto their answer, it wasn't placing the name of the role in the email, however, I got it to work now by changing:

$message = "Hello, your user role is changed to ".$userdata['role']."";

to:

$message = "Hello, your user role is changed to ".$role."";

Upvotes: 0

Snuffy
Snuffy

Reputation: 2096

You should use profile_update hook for this. Read more here - https://developer.wordpress.org/reference/hooks/profile_update/

function notify_user_on_role_change($user_id,$old_user_data,$userdata) {
    // Getting role before update
    foreach($old_user_data->roles as $role):
        $old_role = $role;
    endforeach;

    // error_log(print_r($userdata,true)); // debug 

    //If we change role send email
    if($old_role != $userdata['role']):
        $user_info = get_userdata( $user_id );
        $to = $user_info->user_email; 
        $subject = "Profile Updated";
        $message = "Hello, your user role is changed to ".$userdata['role']."";
        wp_mail( $to, $subject, $message);
    endif;
}
add_action('profile_update','notify_user_on_role_change',10,3);

Send notification only if you change to specific user role

function notify_user_on_role_change($user_id,$old_user_data,$userdata) {
    // Getting role before update
    foreach($old_user_data->roles as $role):
        $old_role = $role;
    endforeach;

    // error_log(print_r($old_role,true)); // debug 

    //If we change role send email
    if($old_role != 'private_event_member' && $userdata['role'] == 'private_event_member'):
        $user_info = get_userdata( $user_id );
        $to = $user_info->user_email; 
        $subject = "Profile Updated";
        $message = "Hello, your user role is changed to ".$userdata['role']."";
        wp_mail( $to, $subject, $message);
    endif;
}
add_action('profile_update','notify_user_on_role_change',10,3);

Upvotes: 2

Related Questions