holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

* Tracker: fix slow rendering of edit popup for hugh number of tracker-user

Browse Source 
doublicate check was iterating through all options for each option, causing it to take eg. 25s for 7000 options
This commit is contained in:
Ralf Becker 2015-05-27 12:18:12 +00:00
parent 3156134a31
commit 82f9190c32
1 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
etemplate/inc/class.etemplate_widget_menupopup.inc.php
View File

@ -333,27 +333,21 @@ class etemplate_widget_menupopup extends etemplate_widget
{
$backup_options = $options;
$values = array();
foreach($options as $value => &$label)
{
// Of course once we re-index the options, we can't detect duplicates
// 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_array($label) || is_array($label) && !array_key_exists('value',$label))
$check_value = (string)(is_array($label) && array_key_exists('value', $label) ? $label['value'] : $value);
if (isset($values[$check_value]))
{
$check_value = (string)(is_array($label) && array_key_exists('value', $label) ? $label['value'] : $value);
if((string)$value === $check_value)
{
foreach($options as $key => $existing)
{
if(is_array($existing) && isset($existing['value']) && (string)$existing['value'] === $check_value && $key != $value)
{
unset($options[$value]);
continue 2;
}
}
}
unset($options[$value]);
continue;
}
$values[$check_value] = $label;
if (is_null($use_array_of_objects) && is_numeric($value) && (!is_array($label) || !isset($label['value'])))
{
$options = $backup_options;