‘wp_new_user_notification’ does not support notify = ‘user’

Hi @joroabv,
Thanks for your message.

I haven’t had any reports of double emails for this notification.
Have you experienced this yourself? If so, can you let me know how/when this happened for you?

@voltronik Yes, I get this when using BNFW in combination with Theme My Login (TML) – Moderation plugin and a custom plugin of my own, that tweaks the notifications a bit. I wanted that the mails are sent in a specific order like :

1. Send the activation email (TML moderation plugin) to user and ‘user created’ email to admin on user registration.
2. Send ‘user created’ email to user after acount activation.

To achieve this the notifications go through TML, but at the end it is all about calling ‘wp_new_user_notification’ with $notify=’admin’ on user registration and with $notify=’user’ after the user activates his account. The thing is that since $notify=’user’ is not supported by BNFW, it sends emails to the admin on both calls, hence the double mails. Besides, since ‘user’ is a valid (and documented) option for the original function, I think your override should support it too.

Hi @joroabv,
Thanks for clarifying this – that’s perfect. I agree about the $user part of the notifications should be supported in the override. Let me do some investigation and see if this can be added.