Closed (fixed)
Project:
Fivestar
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Dec 2011 at 21:38 UTC
Updated:
25 Jan 2012 at 19:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedThis should be a simple matter of removing the following code block:
and then checking whether microdata is set before placing any microdata variables.
Comment #2
Anonymous (not verified) commentedActually, a better way to do this would be to create a
fivestar_get_microdatafunction (or some other name), which then does the test if microdata is enabled, and if not replicates the necessary logic ofentity_get_all_property_info. Then the formatter/preprocess/theme wouldn't need to test whether the microdata variables are available.Comment #3
Anonymous (not verified) commentedI think this should do it.
Comment #4
Anonymous (not verified) commentedSmall change.
Comment #5
ericduran commentedDoesn't apply cleanly :(
Looking into it now.
Comment #6
ericduran commented@linclark Thanks you're awesome :) This is now fixed. Oh and proper commit credit attributions added :)
Thanks again.
--
http://drupalcode.org/project/fivestar.git/commit/558d88f