No need to bicker over Fivestar vs. Node Review. Let them work together!

This patch adds conditional support for Fivestar module, easily swapping out the current select lists and display for a pretty Fivestar display. This patch changes less than 50 lines of code (FAPI is a beautiful thing). Check the screenshots.

CommentFileSizeAuthor
#3 nodereviews_fivestar.patch4.16 KBquicksketch
#2 nodereviews_fivestar.patch4.14 KBquicksketch
nodereviews_fivestar.patch4.16 KBquicksketch
FirefoxScreenSnapz001.png28.18 KBquicksketch
Picture 1.png23.26 KBquicksketch

Comments

JohnForsythe’s picture

JohnForsythe commented

screenshot looks good :)

will test..

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new nodereviews_fivestar.patch4.14 KB

Oops, let's make it so that we don't try to display Fivestar even if it's not enabled.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new nodereviews_fivestar.patch4.16 KB

Another one that references the constant NODEREVIEW_FIVESTAR_ENABLE correctly.

JohnForsythe’s picture

JohnForsythe commented

I've been trying to make this work on my test box, but it doesn't seem to show the fivestar fields in place of the normal dropdown. Is there something I need to configure in fivestar first? I've enabled fivestar for the content type, but that didn't help. I have fivestar 5.x-1.12 installed.

quicksketch’s picture

quicksketch
he/him
commented

Hmm, there shouldn't be any special tricks. Are the fivestar.js and fivestar.css files being added to the page and does Fivestar work when you visit admin/settings/fivestar?

JohnForsythe’s picture

JohnForsythe commented
Assigned: Unassigned » JohnForsythe
Status: Needs review » Fixed

Somehow I missed the new "use fivestar" checkbox on the "review types" screen. Anyway, after testing, it doesn't appear to break easily, and even handles half-stars properly. Good patch :)

quicksketch’s picture

quicksketch
he/him
commented

Great! Get ready for the Fivestar users. Feel free to kick them back if Fivestar starts acting up. ;)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

SamRose’s picture

SamRose commented

Adding score info would be awesome. Such as the way fivestar tallies amount of votes, and "average", or more preferrably a total.

eforth’s picture

eforth commented

Could someone really help me on this? I tried to patch Nodereviews but my nodereview_node_nodereview.inc generates a "not a valid working copy".

I'd really appreciate a hand, thanks in advance

EDIT
NEVERMIND! I learned how to patch manually and this worked like a charm! Excelent Patch, thank you very much