Fix Chart Error Message Display

apache · Aug 14, 2017 · 63ef435 · 63ef435
1 parent c9c6bca
commit 63ef435
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 16 deletions.
diff --git a/superset/assets/javascripts/explore/actions/chartActions.js b/superset/assets/javascripts/explore/actions/chartActions.js
@@ -43,6 +43,11 @@ export function chartRenderingFailed(error) {
   return { type: CHART_RENDERING_FAILED, error };
 }
 
+export const REMOVE_CHART_ALERT = 'REMOVE_CHART_ALERT';
+export function removeChartAlert() {
+  return { type: REMOVE_CHART_ALERT };
+}
+
 export const RUN_QUERY = 'RUN_QUERY';
 export function runQuery(formData, force = false) {
   return function (dispatch, getState) {

diff --git a/superset/assets/javascripts/explore/actions/exploreActions.js b/superset/assets/javascripts/explore/actions/exploreActions.js
@@ -141,11 +141,6 @@ export function removeControlPanelAlert() {
   return { type: REMOVE_CONTROL_PANEL_ALERT };
 }
 
-export const REMOVE_CHART_ALERT = 'REMOVE_CHART_ALERT';
-export function removeChartAlert() {
-  return { type: REMOVE_CHART_ALERT };
-}
-
 export const UPDATE_CHART_TITLE = 'UPDATE_CHART_TITLE';
 export function updateChartTitle(slice_name) {
   return { type: UPDATE_CHART_TITLE, slice_name };

diff --git a/superset/assets/javascripts/explore/components/ChartContainer.jsx b/superset/assets/javascripts/explore/components/ChartContainer.jsx
@@ -325,9 +325,9 @@ ChartContainer.propTypes = propTypes;
 function mapStateToProps({ explore, chart }) {
   const formData = getFormDataFromControls(explore.controls);
   return {
-    alert: explore.chartAlert,
-    can_overwrite: explore.can_overwrite,
-    can_download: explore.can_download,
+    alert: chart.chartAlert,
+    can_overwrite: !!explore.can_overwrite,
+    can_download: !!explore.can_download,
     datasource: explore.datasource,
     column_formats: explore.datasource ? explore.datasource.column_formats : null,
     containerId: explore.slice ? `slice-container-${explore.slice.slice_id}` : 'slice-container',

diff --git a/superset/assets/javascripts/explore/components/SaveModal.jsx b/superset/assets/javascripts/explore/components/SaveModal.jsx
@@ -238,7 +238,7 @@ function mapStateToProps({ explore, saveModal }) {
     can_overwrite: explore.can_overwrite,
     user_id: explore.user_id,
     dashboards: saveModal.dashboards,
-    alert: explore.saveModalAlert,
+    alert: saveModal.saveModalAlert,
   };
 }
 

diff --git a/superset/assets/javascripts/explore/index.jsx b/superset/assets/javascripts/explore/index.jsx
@@ -35,12 +35,12 @@ const bootstrappedState = Object.assign(
     isStarred: false,
     triggerQuery: true,
     triggerRender: false,
-    alert: null,
   },
 );
 
 const initState = {
   chart: {
+    chartAlert: null,
     chartStatus: null,
     chartUpdateEndTime: null,
     chartUpdateStartTime: now(),
@@ -49,6 +49,7 @@ const initState = {
   },
   saveModal: {
     dashboards: [],
+    saveModalAlert: null,
   },
   explore: bootstrappedState,
 };

diff --git a/superset/assets/javascripts/explore/reducers/chartReducer.js b/superset/assets/javascripts/explore/reducers/chartReducer.js
@@ -63,6 +63,12 @@ export default function chartReducer(state = {}, action) {
       }
       return newState;
     },
+    [actions.REMOVE_CHART_ALERT]() {
+      if (state.chartAlert !== null) {
+        return Object.assign({}, state, { chartAlert: null });
+      }
+      return state;
+    },
   };
 
   if (action.type in actionHandlers) {

diff --git a/superset/assets/javascripts/explore/reducers/exploreReducer.js b/superset/assets/javascripts/explore/reducers/exploreReducer.js
@@ -65,12 +65,6 @@ export default function exploreReducer(state = {}, action) {
       const updatedSlice = Object.assign({}, state.slice, { slice_name: action.slice_name });
       return Object.assign({}, state, { slice: updatedSlice });
     },
-    [actions.REMOVE_CHART_ALERT]() {
-      if (state.chartAlert !== null) {
-        return Object.assign({}, state, { chartAlert: null });
-      }
-      return state;
-    },
     [actions.RESET_FIELDS]() {
       const controls = getControlsState(state, getFormDataFromControls(state.controls));
       return Object.assign({}, state, { controls });