Firstly, I'd like to thank you for this great module.

I've found a problem while implementing a fivestar-vote on my nodes.

My situation:
drupal 6.13 => clean default install
fivestar 6.x-2.17
votingapi 6.x-2.1

and, NO any other contribute module.

I enabled fivestart for the default content type "story", and then the fivestar shows up, it's cool.
And then I added a user "test1", and configured the permission that "test1" can vote now.

Then the problems came out, fivestar can only save the "admin" vote.
I use the "admin" user in Firefox, and use the "test1" in IE.( because I want to avoid a cookie conflict ? )
I use "test1" to vote, is shows "saving your vote", and then "your vote have been saved", and then "Your rating: 2 Average: 2 (1 vote)". But after I reload the page, my vote disappeared...The text under the stars is "Your rating: None Average: 2 (1 vote)", but the star is still 2 star, that is right.

I've tried several times, every user can vote correctly, but only the display of "admin" is right.
And, only the last vote value is saved.

Is anybody know something about this? Thanks!

Files: 
CommentFileSizeAuthor
#9 fivestar_single_vote.patch606 bytesquicksketch
1.JPG3.42 KBytt

Comments

altparty’s picture

Similar problem here. All new votes are stored in the database with uid=1.
Same configuration.
Problem appeared after update to fivestar 6.x-2.17

splint’s picture

exact same problem here.

splint’s picture

just installed 6.x-2.x-dev and it fixed the problem.

ytt’s picture

do you mean this is a bug of votingapi?

hitfactory’s picture

Changing line 701 of fivestar.module to the following works for me.

<?php
 _fivestar_cast_vote
($type, $cid, $value, $tag, NULL, TRUE);
?>
Abbademdon’s picture

Seemed it was always setting the current uid to a TRUE value...?

duozersk’s picture

it seems to me that it always records a vote from the uid=1...

| 1144 | node | 2865 | 20 | percent | vote | 1 | 1246635288 | 24.6.228.37 |
| 1160 | node | 2865 | 80 | percent | vote | 1 | 1246714429 | 91.188.6.113 |
| 1187 | node | 2865 | 100 | percent | vote | 1 | 1246915371 | 89.178.157.86 |

duozersk’s picture

fix from #5 helps to fix the issue... don't have skills to validate if it is the fully qualified fix that will not cause other issues.

quicksketch’s picture

Status:Active» Fixed
StatusFileSize
new606 bytes

kidrobot nailed it in #5. Apologies for all the breakage. I've committed his suggested change and I'll roll a new release immediately.

chilledoutbeardedman’s picture

Thanks. Works great.

Status:Fixed» Closed (fixed)

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