tyler.durden
/
icingaweb2
mirror of https://github.com/Icinga/icingaweb2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FilterEditor: fix buggy dropdown boxes 

There were some issues with option names. Also cleaned up HTML a
little bit and added autosubmission.
This commit is contained in:
Thomas Gelf 2014-06-25 10:37:06 +02:00
parent b40d0f6a1e
commit 1dcf44b9cd
1 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
library/Icinga/Web/Widget/FilterEditor.php
View File

@ -48,31 +48,22 @@ class FilterEditor extends AbstractWidget
} else {
$attributes = $view->propertiesToString($attributes);
}
$html = '<select name="' . $view->escape($name) . '"' . $attributes . '>';
$beenActive = false;
sort($list);
$html = '<select name="' . $view->escape($name) . '"' . $attributes . ' class="autosubmit">' . "\n";
asort($list);
foreach ($list as $k => $v) {
$active = '';
if ($k === $selected) {
$active = ' selected="selected"';
$beenActive = true;
}
$v = str_replace('_', ' ', ucfirst($v));
$html .= sprintf(
'<option value="%s"%s>%s</option>',
' <option value="%s"%s>%s</option>' . "\n",
$view->escape($k),
$active,
$view->escape($v)
);
}
if (! $beenActive && $selected) {
$html .= sprintf(
'<option value="%s" selected="selected">%s</option>',
$view->escape($selected),
$view->escape(str_replace('_', ' ', ucfirst(ltrim($selected, '_'))))
);
}
$html .= '</select>';
$html .= '</select>' . "\n\n";
return $html;
}
@ -145,7 +136,7 @@ class FilterEditor extends AbstractWidget
. '</li></ul>';
}
} else {
$html .= $op . '<ul><li>' . implode('</li><li>', $parts) . '</li></ul>';
$html .= $op . "<ul>\n <li>\n" . implode("</li>\n <li>", $parts) . "</li>\n</ul>\n";
}
return $html;
}
@ -205,6 +196,19 @@ class FilterEditor extends AbstractWidget
protected function selectColumn($filter)
{
$name = 'column_' . $filter->getId();
$cols = $this->arrayForSelect($this->query->getColumns());
$active = $filter->getColumn();
$seen = false;
foreach ($cols as $k => & $v) {
$v = str_replace('_', ' ', ucfirst($v));
if ($k === $active) {
$seen = true;
}
}
if (!$seen) {
$cols[$active] = str_replace('_', ' ', ucfirst(ltrim($active, '_')));
}
if ($this->query === null) {
return sprintf(
@ -215,8 +219,8 @@ class FilterEditor extends AbstractWidget
} else {
return $this->select(
$name,
$this->arrayForSelect($this->query->getColumns()),
$filter->getColumn()
$cols,
$active
);
}
}