From d0bad96b1ab9065a5e1d313793da35089ee1f07c Mon Sep 17 00:00:00 2001 From: "Hugh A. Miles II" Date: Fri, 29 Oct 2021 17:19:37 -0400 Subject: [PATCH] fix: Allow users to update database in Dataset Edit Modal (#17265) * add condition to fix save * remove console.log --- .../src/components/Datasource/DatasourceEditor.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/superset-frontend/src/components/Datasource/DatasourceEditor.jsx b/superset-frontend/src/components/Datasource/DatasourceEditor.jsx index ebd2ebfcb7..6c6ae9854c 100644 --- a/superset-frontend/src/components/Datasource/DatasourceEditor.jsx +++ b/superset-frontend/src/components/Datasource/DatasourceEditor.jsx @@ -471,7 +471,6 @@ class DatasourceEditor extends React.PureComponent { const { datasourceType, datasource } = this.state; const sql = datasourceType === DATASOURCE_TYPES.physical.key ? '' : datasource.sql; - const newDatasource = { ...this.state.datasource, sql, @@ -489,6 +488,7 @@ class DatasourceEditor extends React.PureComponent { } onDatasourcePropChange(attr, value) { + if (value === undefined) return; // if value is undefined do not update state const datasource = { ...this.state.datasource, [attr]: value }; this.setState( prevState => ({