forked from extern/egroupware
Fix an additional case in duplication detection.
Prevents doubled options in some select-account widgets
This commit is contained in:
parent
31d36fb718
commit
dac15dbded
@ -237,9 +237,9 @@ class etemplate_widget_menupopup extends etemplate_widget
|
||||
// so check here, as we re-index
|
||||
// Duplicates might happen if app programmer isn't paying attention and
|
||||
// either uses the same ID in the template, or adds the options twice
|
||||
if(is_numeric($value) && is_array($label) && !array_key_exists('value',$label))
|
||||
if(is_numeric($value) && (!is_array($label) || is_array($label) && !array_key_exists('value',$label)))
|
||||
{
|
||||
$check_value = array_key_exists('value', $label) ? $label['value'] : $value;
|
||||
$check_value = is_array($label) && array_key_exists('value', $label) ? $label['value'] : $value;
|
||||
if($value == $check_value)
|
||||
{
|
||||
foreach($options as $key => $existing)
|
||||
|
Loading…
Reference in New Issue
Block a user