fixed default value for selectbox with optgroups does not show label

This commit is contained in:
Ralf Becker 2014-01-10 09:08:40 +00:00
parent 6927f61614
commit 8c44986e7b

View File

@ -387,37 +387,10 @@ class preferences_settings
$GLOBALS['egw']->preferences->group[$appname][$setting['name']] : $GLOBALS['egw']->preferences->group[$appname][$setting['name']] :
$GLOBALS['egw']->preferences->default[$appname][$setting['name']]; $GLOBALS['egw']->preferences->default[$appname][$setting['name']];
if (isset($setting['values']) && !is_array($default) && // replace default value(s) for selectboxes with selectbox labels
(string)$setting['values'][$default] !== '' && (string)$default !== '') if (isset($setting['values']) && is_array($setting['values']))
{ {
if(is_array($setting['values'][$default])) $default = self::get_default_label($default, $setting['values']);
{
foreach($setting['values'] as $key => $value)
{
if($value['value'] == $default && $value['label'])
{
$default = $value['label'];
break;
}
}
}
else
{
$default = $setting['values'][$default];
}
}
else if (strpos($default, ',') !== false)
{
$default = explode(',',$default);
}
if(is_array($default))
{
$values = array();
foreach($default as $value)
{
if (isset($setting['values'][$value])) $values[] = $setting['values'][$value];
}
if ($values) $default = implode(', ', $values);
} }
if (is_array($types[$setting['name']])) // translate the substitution names if (is_array($types[$setting['name']])) // translate the substitution names
{ {
@ -487,6 +460,49 @@ class preferences_settings
return $content; return $content;
} }
/**
* Get label for given default value(s)
*
* @param string|array $default default value(s) to get label for
* @param array $values values optional including optgroups
* @param boolean $lang=true
* @return string comma-separated and translated labels
*/
protected static function get_default_label($default, array $values, $lang=true)
{
// explode comma-separated multiple default values
if (!is_array($default) && !isset($values[$default]) && strpos($default, ',') !== false)
{
$labels = explode(',', $default);
}
else
{
$labels = (array)$default;
}
foreach($labels as &$def)
{
if (isset($values[$def]))
{
$def = is_array($values[$def]) ? $values[$def]['label'] : $values[$def];
}
else // value could be in an optgroup
{
foreach($values as $value)
{
if (is_array($value) && !isset($value['label']) && isset($value[$def]))
{
$def = is_array($value[$def]) ? $value[$def]['label'] : $value[$def];
break;
}
}
}
if ($lang) $def = lang($def);
}
$label = implode(', ', $labels);
//error_log(__METHOD__."(".array2string($default).', '.array2string($values).") returning $label");
return $label;
}
/** /**
* Get preferences by calling various hooks to supply them * Get preferences by calling various hooks to supply them
* *