From e767e4d9ef7b996f8701b8eccd2f0c1ae8ab591f Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Wed, 3 Apr 2019 13:25:49 -0700
Subject: [PATCH] Fix <Select> clearable (#7215)

closes https://github.com/apache/incubator-superset/issues/7213
related to
https://github.com/apache/incubator-superset/pull/6722#issuecomment-479366738
---
 .../components/controls/SelectControl.jsx     | 33 +++++++++----------
 1 file changed, 16 insertions(+), 17 deletions(-)

diff --git a/superset/assets/src/explore/components/controls/SelectControl.jsx b/superset/assets/src/explore/components/controls/SelectControl.jsx
index e15bf7ab0..906d867ea 100644
--- a/superset/assets/src/explore/components/controls/SelectControl.jsx
+++ b/superset/assets/src/explore/components/controls/SelectControl.jsx
@@ -92,25 +92,24 @@ export default class SelectControl extends React.PureComponent {
 
   onChange(opt) {
     let optionValue = null;
-    if (!opt) {
-      return;
-    }
-    if (this.props.multi) {
-      optionValue = [];
-      for (const o of opt) {
-        if (o.meta === true) {
-          optionValue = this.getOptions(this.props)
-                            .filter(x => !x.meta)
-                            .map(x => x[this.props.valueKey]);
-          break;
-        } else {
-          optionValue.push(o[this.props.valueKey]);
+    if (opt) {
+      if (this.props.multi) {
+        optionValue = [];
+        for (const o of opt) {
+          if (o.meta === true) {
+            optionValue = this.getOptions(this.props)
+                              .filter(x => !x.meta)
+                              .map(x => x[this.props.valueKey]);
+            break;
+          } else {
+            optionValue.push(o[this.props.valueKey]);
+          }
         }
+      } else if (opt.meta === true) {
+        return;
+      } else {
+        optionValue = opt[this.props.valueKey];
       }
-    } else if (opt.meta === true) {
-      return;
-    } else {
-      optionValue = opt[this.props.valueKey];
     }
     this.props.onChange(optionValue);
   }