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!
Comment | File | Size | Author |
---|---|---|---|
#9 | fivestar_single_vote.patch | 606 bytes | quicksketch |
1.JPG | 3.42 KB | ytt |
Comments
Comment #1
altparty CreditAttribution: altparty commentedSimilar 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
Comment #2
splint CreditAttribution: splint commentedexact same problem here.
Comment #3
splint CreditAttribution: splint commentedjust installed 6.x-2.x-dev and it fixed the problem.
Comment #4
ytt CreditAttribution: ytt commenteddo you mean this is a bug of votingapi?
Comment #5
hitfactory CreditAttribution: hitfactory commentedChanging line 701 of fivestar.module to the following works for me.
Comment #6
Abbademdon CreditAttribution: Abbademdon commentedSeemed it was always setting the current uid to a TRUE value...?
Comment #7
duozerskit 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 |
Comment #8
duozerskfix 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.
Comment #9
quicksketchkidrobot nailed it in #5. Apologies for all the breakage. I've committed his suggested change and I'll roll a new release immediately.
Comment #10
chilledoutbeardedman CreditAttribution: chilledoutbeardedman commentedThanks. Works great.