Closed (fixed)
Project:
Fivestar
Version:
6.x-1.12
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Aug 2008 at 16:31 UTC
Updated:
11 Sep 2008 at 05:32 UTC
Jump to comment: Most recent file
In the fivestar_form() function, there is a call to set
$settings['text'] => variable_get('fivestar_text_' . $node->type, 'combo')
I believe that string 'combo' should instead be 'dual', based upon the values that are defined for the "Text display style" found in fivestar_form_alter(). There is one other instance similar to this in fivestar_views_widget_hander(), but I'm not as confident in my assessment there since I haven't really been looking at that code.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | fivestar294870.patch | 1.3 KB | vladimir.dolgopolov |
Comments
Comment #1
vladimir.dolgopolov commentedI agree with you. There is a confusion between "dual" and "combo".
"Combo" used in CSS and XML here so I think "combo" should be replaced totally.
But the patch fixes only fivestar_text_* variable.
Comment #2
quicksketchThanks again vladimir! I committed your fix, much appreciated!
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.