Merge pull request #189 from jpmorganchase/filters-fix

Fixing filters when setting multiple filters per column
finos · Aug 13, 2018 · 6b67811 · 6b67811
2 parents 3edaeb1 + b16fffd
commit 6b67811
Showing 1 changed file with 1 addition and 9 deletions.
diff --git a/packages/perspective-viewer/src/js/view.js b/packages/perspective-viewer/src/js/view.js
@@ -258,17 +258,9 @@ function column_aggregate_clicked() {
 
 
 function column_filter_clicked() {
-    let filters = JSON.parse(this.getAttribute('filters'));
     let new_filters = this._get_view_filters();
-    for (let filter of filters) {
-        let updated_filter = new_filters.find(x => x[0] === filter[0]);
-        if (updated_filter) {
-            filter[1] = updated_filter[1];
-            filter[2] = updated_filter[2];
-        }
-    }
     this._updating_filter = true;
-    this.setAttribute('filters', JSON.stringify(filters));
+    this.setAttribute('filters', JSON.stringify(new_filters));
     this._updating_filter = false;
     this._debounce_update();
 }