Module also sends mails to mail addresses that are already in by other users
yaph - May 22, 2007 - 12:49
| Project: | Temporary Invitation |
| Version: | 5.x-1.8 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | yaph |
| Status: | closed |
Jump to:
Description
Hi jpetso
great module! One issue I just noticed is that I can send invitations to an e-mail address that is already used by another user on the same site. When the invited user logs in with the code he has the same e-mail address as the previously existing user.
When the previously existing user tries to change his account settings and submits the form he gets an error message that his e-mail address is already used by another users.
I fixed this by checking if the entered value for the e-mail already exits in the temporary_invitation_manage_validate() function. See attached patch file.
| Attachment | Size |
|---|---|
| temporary_invitation_0.patch | 968 bytes |
#1
Thanks for the bug report and the patch! I committed it in a minimally modified form - linebreaks at 80 characters, and using COUNT(*) instead of db_num_rows(), because the latter is going to be deprecated soon. Fixed in the upcoming 5.x-1.9 release.
#2