Closed (fixed)
Project:
Temporary Invitation
Version:
5.x-1.8
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
22 May 2007 at 12:49 UTC
Updated:
6 Jun 2007 at 13:17 UTC
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.
| Comment | File | Size | Author |
|---|---|---|---|
| temporary_invitation_0.patch | 968 bytes | yaph |
Comments
Comment #1
jpetso commentedThanks 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.
Comment #2
(not verified) commented