I think this was added when adding microdata support. This essentially causes a call to undefined function being that the function is part of entity api.

We don't required entity api. :-/

Comments

Anonymous’s picture

This should be a simple matter of removing the following code block:

  // If microdata is not enabled, add empty arrays for each property so we
  // don't have to check later in the theme functions.
  else {
    $summary_options['microdata'] = array(
      '#attributes' => array(),
    );
    $entity_info = entity_get_all_property_info($entity_type);
    foreach ($entity_info[$field['field_name']]['property info'] as $property_name => $property) {
      $summary_options['microdata'][$property_name]['#attributes'] = array();
    }
  }

and then checking whether microdata is set before placing any microdata variables.

Anonymous’s picture

Actually, a better way to do this would be to create a fivestar_get_microdata function (or some other name), which then does the test if microdata is enabled, and if not replicates the necessary logic of entity_get_all_property_info. Then the formatter/preprocess/theme wouldn't need to test whether the microdata variables are available.

Anonymous’s picture

Status: Active » Needs review
StatusFileSize
new3 KB

I think this should do it.

Anonymous’s picture

StatusFileSize
new3.02 KB

Small change.

ericduran’s picture

Doesn't apply cleanly :(

Looking into it now.

ericduran’s picture

Status: Needs review » Fixed

@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

Status: Fixed » Closed (fixed)

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