Hi @hogash,
Thanks for your message.
This is already available within BNFW. The notification you’re looking for is ‘New User Registration – For Admin’.
Hope this helps.
Thread Starter
Hogash
(@hogash)
Hi,
I know this, however, if i have a plugin that rely on that filter, it won’t work with your plugin. I would appreciate if you could add this default WP filter in your plugin.
For the moment, i had to modify your plugin code.
Best regards,
Stefan
Hi @hogash,
That’s true. I’ll add it to my to-do list to look at in the next version. Thanks!
Hi @hogash,
Do you have an example plugin that doesn’t let BNFW currently override this notification? Would like to test with something.
Thread Starter
Hogash
(@hogash)
Hi,
Just published the plugin we are using:
https://wordpress.org/plugins/disable-new-user-registration-admin-emails/
Let me know if you need anything else.
Best regards,
Stefan
Thanks @hogash,
Are you aware of any other plugins that have this issue too? Interested to test with a plugin that has more installs at the moment.
Hi @hogash,
Ok, thanks. I’ll try your plugin and see how I get on.
Hi @hogash,
The filter from 4.9 is now integrated into BNFW and will be available in a release soon however it would appear that your plugin tries to disable the admin emails by setting the ‘to’ field to false. This will be incompatible with BNFW (and possibly other plugins too) as when a user has an admin notification set, we will be changing the ‘to’ field based on the values that we have in the notification. BNFW allows you to disable the New User Registration – For Admin notification as native functionality anyway though so this may not be an issue.
Just wanted to let you know.
Thread Starter
Hogash
(@hogash)
Hi,
Thanks for the info. I am aware of this, however, it was the only way to disable the new user notification email without overriding the wp_new_user_notification pluggable function.
