You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ch...@apache.org on 2019/02/19 20:51:14 UTC

[incubator-superset] branch release--0.31 updated: Fix extra_filters in multi line viz (#6868)

This is an automated email from the ASF dual-hosted git repository.

christine pushed a commit to branch release--0.31
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/release--0.31 by this push:
     new c0eaa5f  Fix extra_filters in multi line viz (#6868)
c0eaa5f is described below

commit c0eaa5f62d9c78e3f0ccce296f59de61da78295d
Author: Beto Dealmeida <ro...@dealmeida.net>
AuthorDate: Tue Feb 19 12:51:08 2019 -0800

    Fix extra_filters in multi line viz (#6868)
    
    (cherry picked from commit b035185b1c5c4f4332bfc8c9f748166f8d43151f)
---
 superset/assets/src/visualizations/nvd3/LineMulti/LineMulti.jsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/superset/assets/src/visualizations/nvd3/LineMulti/LineMulti.jsx b/superset/assets/src/visualizations/nvd3/LineMulti/LineMulti.jsx
index aa95d54..cbd6eb1 100644
--- a/superset/assets/src/visualizations/nvd3/LineMulti/LineMulti.jsx
+++ b/superset/assets/src/visualizations/nvd3/LineMulti/LineMulti.jsx
@@ -94,9 +94,9 @@ class LineMulti extends React.Component {
       const combinedFormData = {
         ...subslice.form_data,
         filters: (subsliceFormData.filters || [])
-          .concat(filters || [])
-          .concat(extraFilters || []),
+          .concat(filters || []),
         time_range: timeRange,
+        extra_filters: extraFilters || [],
       };
       const addPrefix = prefixMetricWithSliceName;
       return getJson(getExploreLongUrl(combinedFormData, 'json'))