Currently using the "Current user" option while doing joins to VotingAPI data tables does not take into account the user's IP address. Since there can be many votes for the 0 UID, only the first anonymous user's vote ever shows up in generated Views.

I haven't tested this patch, but it seems like it's the general approach needed to correct this for anonymous users.

A better explanation of the problem and how to reproduce it is at http://drupal.org/node/592266.

Comments

ChrisRut’s picture

subscribe

ChrisRut’s picture

StatusFileSize
new904 bytes

Attached is a non-dev patch for 6.x-2.3
Patch worked for me BTW, Thanks quicksketch :-)

mmilo’s picture

BUMP!

I'm guessing that #798790: Option for anonymous votes to use session id in vote_source field might be a longer term solution, but this patch works for me!

YK85’s picture

subscribing

geerlingguy’s picture

Subscribe - just ran into this problem. I applied the patch in the first post to the latest stable release of Voting API, and it works fine for me.

geerlingguy’s picture

Status: Needs review » Reviewed & tested by the community

Changing status... maybe can also get into the 7.x branch?

asb’s picture

Could we please get a new release with this patch, after is "reviewed & tested by the community" for several months and the last stable/recommended release 6.x-1.19 was published 2009-Oct-22 (almost two years ago)?

Thank you very much!!

RoastBeats’s picture

Hey ASB... I made the same mistake at first. I think you're looking at Fivestar, but this is a patch for VotingAPI (which corrects the problem in Fivestar). So the 6.X-1.19 version from Oct 09 is referencing Fivestar.

Also, confirming that the patch worked for me (though, just in case others get hit on the same snag... don't forget to renew your IP address in order to test!).

asb’s picture

The latest release tagged as "supported" and "recommended" is votingapi 6.x-2.3 and was published 2009-Aug-15. For 'Voting API', for more than two years no "supported" and "recommended" release has been published.

torotil’s picture

Version: 6.x-2.x-dev » 7.x-2.x-dev
Status: Reviewed & tested by the community » Needs work

I'm not convinced that this is or should be in the scope of votingapi additionally the proposed patch has whitespace errors.

elshae’s picture

Issue summary: View changes

I used that patch on 6.x-2.3 and selected anonymous rollover to be immediately and now as anonymous I see every item as voted for! sigh....