I’m using BP Registration options to require approval from Admin before a user can access certain areas of the site. This works fine if Admin only issues approval AFTER a user has activated their account, however, Admin approval automatically activates the account which means that if the user attempts to follow the activation link after they have already been approved, it throws an “Invalid Activation Key” error.
I’m wondering if there is any way of editing activate.php to check if an account has already been activated eg. something like bp_account_was_activated but more like bp_account_is_activated_already in order to offer a more useful message eg. “This step is no longer required as your account is already activated so feel to just go ahead and log in” 😉
I guess the other option is simply to make it clear to Admin that they must not approve accounts until the user has activated them but it would still be good to have a way of handling this just in csae they forget.
With many thanks in advance for any thoughts.
Please modify the activate.php template file in yourtheme/buddypress/members and then modify the text to say that you might have already activated the account.
There is a function to check if a user account is active
Since you don’t have access to user id, you won’t be able to use it there.
Thank you as always Brajesh.
And yes, if I understand you correctly, simply modifying the text on the activate.php to advise that if they have problems they may already be activated is probably a good (and the simplest!) solution.
Interestingly, looking at the function bp_core_activate_signup, it LOOKS as if it should be checking and returning a different error if ‘The user is already active’ but it’s really not such a critical issue either way for me to try and figure out why that’s not triggering 😉
Always appreciate your insight and many thanks again.
It is possible to search by key and check if the user activated account. The problem is this behaviour is configurable and depending on your setup, finding by key may not be available.
That’s why I suggested the generic approach.
You must be logged in to reply to this topic.