Hi,

As always, great module. I noticed a bit of a bug when testing.

I cleared the webform attached to the sweepstakes I created so there were no submissions. I then entered 2 new submissions as an unauthenticated user to 2 different email address. When I ran cron and checked the winner in the morning, the winner was me (the admin) because I had entered the sweepstakes before clearing the webform. I think this bug could apply to admins and other authenticated users, but I'm not sure. On the sweepstakes page (only accessible by admin) it says that you've already entered the sweepstakes. However, I believe this should be cleared when you clear the webform submissions.

I can continue to test this module if this is helpful.

Cheers,

CarbonPig

Comments

ablondeau’s picture

ablondeau commented

Thanks for the bug report! It looks like I need to look at hooking into webform somehow and catching when entries are deleted. Temporary fix would be to remove all entries with the matching webform node id from table sweepstakes_entries.