Fivestar module is allowing to revote eventhough the options "Allow the users to revote" and "Allow the users to cancel ratings" are turned off. Actually it is giving the user an impression that it is allowing to revote, anyway it is not updating in the database. Also it is not displaying the average vote after voting in ajax. It will display the average only after reloading the page.

Please review my patch.

CommentFileSizeAuthor
#7 fivestar_dont_allow_revote_ajax-2686357-7.patch669 bytescriz
#2 fivestar_rating_average_not_showing-2686357-1.patch677 bytesvishnu358
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vishnu358 created an issue. See original summary.

vishnu358’s picture

vishnu358 CreditAttribution: vishnu358 commented
FileSize
fivestar_rating_average_not_showing-2686357-1.patch677 bytes
vishnu358’s picture

vishnu358 CreditAttribution: vishnu358 commented
Issue summary: View changes
vishnu358’s picture

vishnu358 CreditAttribution: vishnu358 commented
Status: Active » Needs review
rakesh.nimje84@gmail.com’s picture

rakesh.nimje84@gmail.com CreditAttribution: rakesh.nimje84@gmail.com commented

#2 Working in my case.

vishnu358’s picture

vishnu358 CreditAttribution: vishnu358 commented
Issue tags: +Fivestar
criz’s picture

criz
he/him
Location Vienna/Austria
CreditAttribution: criz at acolono GmbH commented
Title: Fivestar is allowing to revote and not displaying average. » Fivestar is allowing to revote on ajax despite settings
FileSize
fivestar_dont_allow_revote_ajax-2686357-7.patch669 bytes

Thx, patch works for me.
Just corrected some coding style issues.

criz’s picture

criz
he/him
Location Vienna/Austria
CreditAttribution: criz at acolono GmbH commented
Version: 7.x-2.1 » 7.x-2.x-dev
Status: Needs review » Reviewed & tested by the community
jafacakes2011’s picture

jafacakes2011 CreditAttribution: jafacakes2011 at Upbeat Productions commented

Hi, #7 worked nicely for me. Think it should be added when possible :)

dbt102’s picture

dbt102 CreditAttribution: dbt102 commented

Thanks for the patches and reviews

  • dbt102 committed cd62eb2 on 7.x-2.x authored by vishnu358 
    Issue #2686357 by vishnu358, criz, dbt102: Fivestar is allowing to...
dbt102’s picture

dbt102 CreditAttribution: dbt102 commented
dbt102’s picture

dbt102 CreditAttribution: dbt102 commented
Status: Reviewed & tested by the community » Fixed
dbt102’s picture

dbt102 CreditAttribution: dbt102 commented

Status: Fixed » Closed (fixed)

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