Now you can only vote once on a node when you are logged in. I'd like to make it possible that a user can vote multiple times on a node. Basicly I'm making the anonymous vote interval available for users with the current behaviour (Never) as default.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | fivestar.module.txt | 58.44 KB | Anonymous (not verified) |
| #12 | fivestar.module.txt | 57.42 KB | webpoga |
| #7 | drupal5.patch | 2.6 KB | Uruviel |
| #7 | drupal6.patch | 1.83 KB | Uruviel |
| #1 | fivestar.patch | 2.79 KB | Uruviel |
Comments
Comment #1
Uruviel commentedComment #2
jax commentedsubscribing
Comment #3
captcha commentedIf a user wants to vote a multiple number of times, s/he need not log in and go about voting as an anonymous user, till satisfied.
I am not sure why a user should be allowed more than one vote, it's not logical unless you are forming some sort of hierarchy of users, where some are given more powers than others :)
Comment #4
jax commentedWell actually it's a feature we need. Any user, anonymous or not, is presented with random nodes to vote on. If a node is shown for a second time the user can just vote again. Since the behavior is optional, why not include it and spare us the need to maintain a fork of the module ourselves?
Comment #5
quicksketchSeems like a reasonable request, but I don't feel that I can commit this patch now that there is a Drupal 6 version of the module where this functionality has been moved to voting API. If someone can provide the patch necessary to add this to the Drupal 6 version of Fivestar and VotingAPI, I'd be glad to add it.
The Drupal 5 version looks great however. Nice work Uruviel.
Comment #6
Uruviel commentedThank you. I'd like to see it commited for Drupal 5 however as we are using that version for all our websites. As previously said, the behaviour is optional.
Comment #7
Uruviel commentedI updated the patch for the new version of fivestar. I also created a patch for drupal 6. It needs a good review though as I'm very new to drupal 6.
Comment #8
quicksketchUruviel, thanks for the patch for Drupal 6, but the functionality needs to be added to VotingAPI in 6, not Fivestar. Because the voting interval has been moved to VotingAPI directly, it makes more sense to add this option in VotingAPI rather than Fivestar to keep everything in the same place. Also, be sure to include spaces instead of tabs in your patch.
Comment #9
Uruviel commentedIf I make a patch for voting API (drupal 6 version), will you patch the drupal 5 version of fivestar?
Comment #10
quicksketchYes, if you make the patch for VotingAPI (and Eaton is cool with it), I'll apply the patch to 5.x Fivestar. I just want to make sure that the features are still available in 6.x and in the right place. No one would want to upgrade to 6.x if it caused them to loose features. Thanks for your perseverance in this.
Comment #11
webpoga commentedcan someone make me a PATCH(ED) file for this. Im having trouble patching files. PLEASE.
Comment #12
webpoga commentedI cannot really use that file. Can some patch this file for me. I need the drupal5.patch
Comment #13
Anonymous (not verified) commentedHello webpoga,
I attached a patched version of fivestar for d5. What's missing in the patch is the setting of a variable. The patched version is using a variable "fivestar_user_vote_interval" with a value of "-1" but it is not set in the patch file. So you have to add it yourself to the variables table in your Drupal installation.
Best regards
Mike
hailstorm
Comment #14
quicksketchThis issue has languished. Like I said above, this needs to be a feature of VotingAPI before it can be added to Fivestar. Given the lack of updates, I don't think this will be completed. Please reopen if interested in carrying the torch.