When using email verification, the first time a user verifies their email everything is OK. If they verify a second time, though, they get what looks like an error message, when in fact they are just unnecessarily re-verifying their email. This was confusing our users and causing complaints.

This patch changes the message, but still doesn't log the user in automatically from the verification email. It also displays the message as informational instead of a warning.

If the user is already logged in as the user who this email verifies, it just shows a message that the user has already verified their email address.

If the user is not logged in, it additionally suggests they log in.

If the user is logged in as another user, it additionally logs them out.

CommentFileSizeAuthor
logintoboggan-reverify.patch890 bytesscottgifford

Comments

hunmonk’s picture

hunmonk commented
Status: Needs review » Closed (won't fix)

i'm happy with the current implementation. i personally think an error message is appropriate in this circumstance, and the instructions in that message seem pretty clear to me.

so most likely, this patch won't be committed to the official version of the module. however, you can still run it locally on your site -- that's the beauty of open source... :)

scottgifford’s picture

scottgifford commented

OK. We developed this patch in response to complaints from our mobile users, who would sometimes click the link twice, probably accidentally, then call us to complain that our site was broken. Perhaps the users you are targeting are more savvy, or have a steadier finger on their iPhone screens. :-)