maxmustermann/kibana
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix issue with adding negating scripted filter

Browse source
This commit is contained in:
Lukas Olson 2015-09-21 12:44:06 -07:00
parent 77078ec818
commit bfc03ddd13
2 changed files with 20 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/ui/public/filter_manager/__tests__/filter_manager.js
View file

@ -111,6 +111,22 @@ describe('Filter Manager', function () {
filterManager.add('_exists_', 'myField', '-', 'myIndex');
checkAddFilters(0, null, 3);
expect(appState.filters).to.have.length(2);
var scriptedField = {name: 'scriptedField', scripted: true, script: 1};
filterManager.add(scriptedField, 1, '+', 'myIndex');
checkAddFilters(1, [{
meta: {index: 'myIndex', negate: false, field: 'scriptedField'},
script: {
script: '(' + scriptedField.script + ') == value',
lang: scriptedField.lang,
params: {value: 1}
}
}], 4);
expect(appState.filters).to.have.length(3);
filterManager.add(scriptedField, 1, '-', 'myIndex');
checkAddFilters(0, null, 5);
expect(appState.filters).to.have.length(3);
});
it('should enable matching filters being changed', function () {

4
src/ui/public/filter_manager/filter_manager.js
View file

@ -26,6 +26,10 @@ define(function (require) {
if (filter.query) {
return filter.query.match[fieldName] && filter.query.match[fieldName].query === value;
}
if (filter.script) {
return filter.meta.field === fieldName && filter.script.params.value === value;
}
});
if (existing) {