Bug: new user notification email cannot be disabled

TimJensen · May 14, 2024, 4:00pm

There is a bug where the new user notification email sends even when the setting is disabled via the ‘wp_send_new_user_notification_to_user’ filter.

github.com

discourse/wp-discourse/blob/7f16f3fa5e97fe2478c07104b1ea6d9678722627/lib/wp-new-user-notification.php#L114-L117


      
                $send_notification_to_user = apply_filters( 'wp_send_new_user_notification_to_user', true, $user );
          
          			// `$deprecated was pre-4.3 `$plaintext_pass`. An empty `$plaintext_pass` didn't sent a user notification.
          			if ( 'admin' === $notify || ( empty( $deprecated ) && empty( $notify ) ) ) {

To fix the bug the $send_notification_to_user variable needs to be included in the condition on line 117.

angus · May 15, 2024, 7:57am

Thanks Tim, looks like that is indeed a bug. I’ll add a fix for it in the next version.

Topic		Replies	Views
Notification Emails Not Being Sent for New Messages Bug needs-repro	1	24	August 12, 2024
Troubleshooting user who changed email via wordpress sso, not getting notifications since then WordPress	0	658	February 11, 2019
Disable email notifications Feature	7	998	July 11, 2023
How can notifications be configured? Support	7	554	September 10, 2022
How can I send email notifications to users from discourse? Support	4	666	February 13, 2022

Bug: new user notification email cannot be disabled

Related topics