First of all thanks for getting this ready for D6. :)

There appears to be a problem with anonymous votes overwriting user votes when the same IP is used. It looks as though the problem is in votingapi_current_user_identifier() exclusively using IP when $user->uid is zero. This will then cause authenticated user votes to be returned when fed directly into votingapi_select_votes().

I've attached a patch fixing the above. Please review.

--
Sammy Spets
Synerger
http://synerger.com

CommentFileSizeAuthor
votingapi.module.200803171614.patch667 bytessammys

Comments

eaton’s picture

eaton commented
Status: Needs review » Fixed

This fix has been integrated in the newer betas. Thanks!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.