Closed (fixed)
Project:
Fivestar
Version:
7.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
6 Nov 2013 at 07:41 UTC
Updated:
20 Feb 2014 at 08:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
marcoka commentedactually i can confirm this with alpha2
ok this happens if you do the following
add a field "rate while editing" and chose tag "vote"add a field "rate while viewing" and chose tag "vote".
somehow in my install all fields seem to be reset and have "vote" as tag and thats why this is.but i am pretty sure i created a tag for each field, they still exist but are not assigned anymore. maybe something/update broke it. i dont know.
tested again and the above statement is invalid because i added another field with a tag "foo". it is also reset.
that happens if you vote your own content (in my case admin voted both times)
does happen too if voting user not the user who voted while editing
Comment #2
whiteph commentedI also came across this, while looking into Star Display Options Missing from field.
I agree that this is a critical issue.
Comment #3
whiteph commentedPlease test this patch.
Comment #5
whiteph commented3: fivestar-ratingIsReset-2128711-3.patch queued for re-testing.
Comment #7
whiteph commentedThe patch I posted in #3 passes the tests in my local version, and the failure details seem to have very little to do with my change. The error report shows a MySQL environment failure. Maybe there's a problem on Drupal.org.
Comment #8
whiteph commented3: fivestar-ratingIsReset-2128711-3.patch queued for re-testing.
Comment #10
whiteph commented3: fivestar-ratingIsReset-2128711-3.patch queued for re-testing.
Comment #12
jthorson commentedComment #13
jthorson commented3: fivestar-ratingIsReset-2128711-3.patch queued for re-testing.
Comment #14
jthorson commentedTest issue was due to the issue version, testing against a release instead of the latest -dev.
Comment #15
whiteph commented@jthorson - thanks!
Comment #16
whiteph commented@hksuineg and @marcoka - are you able to test this patch? Given that it's a critical issue, I'd like to commit this asap. Thanks.
Comment #17
wouser commentedI have also confirmed that Patch 3 works successfully. Thank you whiteph!
Comment #18
whiteph commentedChanged status after comment #17
Comment #19
whiteph commentedThanks @wouser