only fetch tables if we have a schema, otherwise reset options. (#2490)

apache · Mar 27, 2017 · 93551a6 · 93551a6
1 parent 7eafbab
commit 93551a6
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/superset/assets/javascripts/SqlLab/components/SqlEditorLeftBar.jsx b/superset/assets/javascripts/SqlLab/components/SqlEditorLeftBar.jsx
@@ -65,7 +65,7 @@ ${this.props.queryEditor.schema}/${input}`;
   }
   // TODO: move fetching methods to the actions.
   fetchTables(dbId, schema, substr) {
-    if (dbId) {
+    if (dbId && schema) {
       this.setState({ tableLoading: true, tableOptions: [] });
       const url = `/superset/tables/${dbId}/${schema}/${substr}/`;
       $.get(url, (data) => {
@@ -75,6 +75,8 @@ ${this.props.queryEditor.schema}/${input}`;
           tableLength: data.tableLength,
         });
       });
+    } else {
+      this.setState({ tableLoading: false, tableOptions: [] });
     }
   }
   changeTable(tableOpt) {