Multiple Notification Emails being sent

  • I’ve been working with a client using this plugin and they reported multiple emails were being sent. It turns out, after approving a user they were clicking Refresh to check for new entries, which sent another email (and generated a new password).

    I’m posting this in case anyone else has the same issue.

    Also, I’ve made some modifications to rpr-admin-menu.php, which should correct this issue. It checks that a user has the role rpr_unverified before approving them.
    A diff of the changes is available at
    https://www.dropbox.com/s/cagifd3zoks9699/rpr-admin-menu-diff.txt

    If it would be helpful, I’m happy to make these changes available to you in another format.

    Thanks.

    https://www.ads-software.com/plugins/register-plus-redux/

Viewing 2 replies - 1 through 2 (of 2 total)

  • Hi Salcode,

    Thanks for this fix, I’m having the same problem as well. My problem with your code is that I’m not sure where to put it right now, as it kind of abruptly ends!
    You’d help me a lot showing the entire file if you could.

    Thanks in advance!

    Thread Starter Sal Ferrarello

    (@salcode)

    Disclaimer first, this patch was based on a previous version. I’ve applied the changes to the latest version of the plugin and it seems to work but it has not been tested thoroughly. This is unsupported and you can use it at your own risk.

    Replacing rpr-admin-menu.php with the revised version at
    https://gist.github.com/salcode/eb541ee54ce840287730 should solve the problem.

    What the code is doing – I’m adding a check for the current role to ensure the user has not already been approved.

    If the authors want to get in contact with me, I’d be happy to help them integrate these changes into the plugin.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Multiple Notification Emails being sent’ is closed to new replies.