Closed (won't fix)
Project:
Fivestar
Version:
6.x-1.20
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Jan 2013 at 19:16 UTC
Updated:
4 Mar 2014 at 08:27 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
organicwire commentedHere's the exported view for testing.
Comment #2
organicwire commentedHere's a patch fixing the issue. The problem is that in fivestar.module, the custom widget is built with the form_id
fivestar_custom_widget(fivestar_form(), line 1091). Later, when processing the submit data, form-data with this form_id is refused (fivestar_form_submit(), line 1299). Therefore, the form data is simply ignored.The form fixes this by adding an alternative if condition
$form_state['values']['form_id'] == 'fivestar_custom_widget'Comment #3
organicwire commentedOops, removed debug comment from patch.
Comment #4
organicwire commentedPlease review.
Comment #6
organicwire commentedTry this patch. The module path wasn't standard.
Comment #7
organicwire commentedSorry, this was the wrong (original) patch again. Now the right one.
Comment #9
organicwire commentedRequeue for testing because testbot expects module in sites/default/modules.
Comment #11
organicwire commentedIt's still a wrong path in the patch. Another try.
Comment #12
organicwire commentedComment #13
organicwire commentedAnyone there for reviewing? The fix is there, we just need a little testing ;)
Comment #13.0
organicwire commentedSuggestion for content types for easier testing.
Comment #14
whiteph commentedWe can no longer support the Drupal 6 version of Fivestar. It is in security maintenance mode only. When the Drupal 8 version of Fivestar is released, the Drupal 6 version will be officially deprecated.
Comment #15
whiteph commentedSee Help testing Drupal 6 patches.