Closed (fixed)
Project:
Fivestar
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Jul 2011 at 17:40 UTC
Updated:
28 Sep 2011 at 23:51 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
james.elliott commentedComment #2
ericduran commentedHere's a re-rolled of this patch. But I don't think this patch is ever going to get in like this.
We can't just remove the votingapi hooks because they provide a little more than stars display. Right now the fields work as you would expected them to work, but all the logic is done on the rendering of the widget.
If we apply this patch, we loose the option of doing relationships with the votingapi results. This will not allow people to do something like Top rated content. We'll they'll be able to do top rated content they just wont be able to display them as stars.
So this issue should be Clean up votingapi hooks, because it's still relying on all the old content type settings stuff.
Comment #3
ericduran commentedComment #4
ericduran commentedComment #5
ericduran commentedJust bumping this issue for my self. This will ideally be the next patch to get in.
Also note the above patch just remove the support. This is not a viable solution because we still needs the stars to be a widget when display voting api data such as Top rated content, etc.
Comment #6
rodrigoaguilerawell, subscribe to this issue then
Comment #7
brunorios1 commentedsubscribing
Comment #8
jabbamonkey commentedsubscribing...
Comment #9
BeaPower commentedAlso bring back the ability to change colors.
Comment #10
tneo commentedAnd how do I get my own widget instead of stars, will that be fixed by this commit?
Comment #11
dagleesWhat's the proposed solution? We may be able to help if we know what direction you want this to take.
Comment #12
ann_meredith commentedsub
Comment #13
ericduran commented@BeaPower, that feature was remove on purpose.
@tneo, You can declare your own widget using hook_fivestar_widgets.
@Daglees, I don't quiet have a proposed solution just yet.
Comment #14
SNaKeMe commentedCan't change the style of fivestar on my panel site.
Get an AJAX Error:
Fatal error: Unsupported operand types in /sites/all/modules/fivestar/includes/fivestar.field.inc on line 353
I think it's the same error you have in views just in panels.
Sub.
Comment #15
ericduran commentedThis commit -- http://drupal.org/commitlog/commit/2490/d8d5c086f0c75ecc743f07a2e1be7a0a...
Removed the views support for now. Until we have a fix in place. I'm doing this to prevent users from trying to use the non-workings views formatter.
Please don't comment on this issue asking for views support. We will add it back.
Comment #16
tneo commentedThanks Eric, found where to change it. :-)
Comment #17
Canadaka commentedsubscribe
Comment #18
ericduran commentedOk, so I though of this for a while.
I think the best option is to drop the custom views formatters. We should remove all the custom views_formatter code out of the fivestar module.
You really don't need the custom formatters anymore because you could just select the field for display. The only thing you needs is the voting api relationships to do the sorts which you still have available to you.
So essentially strip all the functionality and this should be good to go.
Comment #19
ericduran commentedIt looks like @james.elliott was a month ahead of me. ;)
Comment #20
ericduran commentedComment #21
ericduran commentedOk, this is done -- http://drupal.org/commitlog/commit/2490/fe54202c94f747d2724b8b4324926fd9...
Comment #22
Canadaka commentedI just installed the newest DEV and I can still add my fivestar "field_rating" as a field and it produces the JS error here: http://drupal.org/node/1268972
So how can I go about showing a nodes current rating represented by stars in a view?
Comment #23
ericduran commented@Canadaka is not instance. It takes time for the new file to be available with the changes.
I'll have a look at the other issue and report back there.
Comment #24
rodrigoaguileraThe example view should be updated to make an example on how to show the stars in wiews ¿Am I right?
Comment #25
ericduran commented@rodrigoaguilera We don't provide an example views. I'll create a patch for votingapi to remove there fivestar example.
Comment #26
Canadaka commentedIts working now in views! Sometimes when before the view is saved the widget selection contains no images though, just the text representation
But when I save view seems to be all good.
Comment #27
brunorios1 commentedbut is possible to configure the display options of the widget using the field?
i.e.: i want to choose the stars style, show stars only without the vote feature and without the text...
thanks!
Comment #28
Canadaka commentedyes you can do all of that in the field settings in views. But you have to do it for each view, would be nice if one could set a site-wide default widget selection.
Comment #29
ericduran commented@brunorios1 Yes its possible, just unchecked the exposed this widget checkbox.
Comment #30
ericduran commented@Canadaka, glad is all working now.
Comment #32
marcoka commentedit is pretty nice cleaning things up. the problem with this it that if you upate now, your site will be broken because now formatters are missing. also ratings in blocks will not appear anymore.