Closes #2212 - Ignoring both the source and target attributes

elastic · Dec 10, 2014 · 2570759 · 2570759
1 parent e47adef
commit 2570759
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/kibana/components/filter_bar/lib/dedupFilters.js b/src/kibana/components/filter_bar/lib/dedupFilters.js
@@ -4,7 +4,7 @@ define(function (require) {
   return function (existing, filters) {
     filters = _.filter(filters, function (item) {
       return !_.find(existing, function (existingFilter) {
-        return _.isEqual(_.omit(existingFilter, excludedAttributes), item);
+        return _.isEqual(_.omit(existingFilter, excludedAttributes), _.omit(item, excludedAttributes));
       });
     });
     return filters;

diff --git a/test/unit/specs/components/filter_bar/dedupFilters.js b/test/unit/specs/components/filter_bar/dedupFilters.js
@@ -25,7 +25,7 @@ define(function (require) {
         ];
         var filters = [
           { range: { bytes: { from: 1024, to: 2048 } } },
-          { query: { match: { _term: { query: 'apache', type: 'phrase' } } } }
+          { meta: { negate: false }, query: { match: { _term: { query: 'apache', type: 'phrase' } } } }
         ];
         var results = dedupFilters(existing, filters);
         expect(results).to.contain(filters[0]);