switch ($element['#settings']['style']) {
      case 'average':
        $default_value = $element['#values']['average'];
      case 'user':
        $default_value = $element['#values']['user'];
      case 'smart':
        $default_value = (!empty($element['#values']['user']) ? $element['#values']['user'] : $element['#values']['average']);
      case 'dual':
        $default_value = $element['#values']['user'];
    }

The default value in my case (case-smart) was always overridden by the user value (case 'dual').
I suggest adding breaks to all case statements in _fivestar_get_element_default_value.

Comments

ericduran’s picture

ericduran commented
Priority: Normal » Critical

damn, I could swear I added those case statements already :-/

wouters_f’s picture

wouters_f commented
Status: Active » Fixed

Is fixed in recent version.

ericduran’s picture

ericduran commented
Status: Fixed » Closed (duplicate)

Ah ok Thanks. Marking it as dupe instead of fixed.