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 :
- Send the activation email (TML moderation plugin) to user and ‘user created’ email to admin on user registration.
- 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.