pt/superset
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2024-09-12 16:49:40 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #171 from mistercrunch/fix_filter

Browse Source 
Fixing multiple refresh bug in filter_box
This commit is contained in:
Maxime Beauchemin 2016-03-08 07:29:14 -05:00
commit 91833ce16f
2 changed files with 6 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
panoramix/assets/.eslintrc
View File

@ -58,7 +58,7 @@
"max-depth": [2, 5], "max-depth": [2, 5],
"max-len": [0, 80, 4], "max-len": [0, 80, 4],
"max-nested-callbacks": [1, 2], "max-nested-callbacks": [1, 2],
"max-params": [1, 3], "max-params": [1, 4],
"new-parens": [2], "new-parens": [2],
"newline-after-var": [0], "newline-after-var": [0],
"no-bitwise": [0], "no-bitwise": [0],

14
panoramix/assets/visualizations/filter_box.js
View File

@ -7,21 +7,17 @@ var d3 = window.d3 || require('d3');
require('./filter_box.css'); require('./filter_box.css');
require('../javascripts/panoramix-select2.js'); require('../javascripts/panoramix-select2.js');
function filterBox(slice) { function filterBox(slice) {
var filtersObj = {}; var filtersObj = {};
var d3token = d3.select(slice.selector); var d3token = d3.select(slice.selector);
var fltChanged = function () { var fltChanged = function () {
for (var filter in filtersObj) { var val = $(this).val();
var obj = filtersObj[filter]; var vals = [];
var val = obj.val(); if (val !== '') {
var vals = []; vals = val.split(',');
if (val !== '') {
vals = val.split(',');
}
slice.setFilter(filter, vals);
} }
slice.setFilter($(this).attr('name'), vals);
}; };
var refresh = function () { var refresh = function () {