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

FilterQueryString: fix issues with not 

NOT is a "chain" with just one subfilter and therefore behaves a little
bit different. Should work fine now.

fixes #6554
This commit is contained in:
Thomas Gelf 2014-06-25 10:39:48 +02:00
parent 1dcf44b9cd
commit 9359f527db
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
library/Icinga/Data/Filter/FilterQueryString.php
View File

@ -122,12 +122,16 @@ class FilterQueryString
if ($next === '!') {
$not = $this->readFilters($nestingLevel + 1, '!');
$filters[] = $not;
continue;
if (in_array($this->nextChar(), array('|', '&', ')'))) {
$next = $this->readChar();
} else {
break;
}
}
if ($op === null && count($filters > 0) && ($next === '&' || $next === '|')) {
$op = $next;
$next = $this->readChar();
continue;
}
if ($next === false) {
@ -180,10 +184,15 @@ class FilterQueryString
}
}
if ($nestingLevel === 0 && count($filters) === 1) {
if ($nestingLevel === 0 && $this->pos < $this->length) {
$this->parseError($op, 'Did not read full filter');
}
if ($nestingLevel === 0 && count($filters) === 1 && $op !== '!') {
// There is only one filter expression, no chain
return $filters[0];
}
if ($op === null && count($filters) === 1) {
$op = '&';
}
@ -204,6 +213,10 @@ class FilterQueryString
$this->string = $string;
$this->length = strlen($string);
if ($this->length === 0) {
return Filter::matchAll();
}
return $this->readFilters();
}