From ebed50fd12c78e523c102abccc38a8d89554f863 Mon Sep 17 00:00:00 2001
From: "Michael S. Molina" <70410625+michael-s-molina@users.noreply.github.com>
Date: Wed, 1 Feb 2023 10:33:42 -0500
Subject: [PATCH] fix: Remove button is broken for metrics on Explore (#22940)

---
 .../controls/FilterControl/AdhocFilterControl/index.jsx   | 8 ++++++--
 .../explore/components/controls/OptionControls/index.tsx  | 5 +----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/superset-frontend/src/explore/components/controls/FilterControl/AdhocFilterControl/index.jsx b/superset-frontend/src/explore/components/controls/FilterControl/AdhocFilterControl/index.jsx
index 2114a9bd45..44f4c469e6 100644
--- a/superset-frontend/src/explore/components/controls/FilterControl/AdhocFilterControl/index.jsx
+++ b/superset-frontend/src/explore/components/controls/FilterControl/AdhocFilterControl/index.jsx
@@ -121,7 +121,10 @@ class AdhocFilterControl extends React.Component {
         sections={this.props.sections}
         operators={this.props.operators}
         datasource={this.props.datasource}
-        onRemoveFilter={() => this.onRemoveFilter(index)}
+        onRemoveFilter={e => {
+          e.stopPropagation();
+          this.onRemoveFilter(index);
+        }}
         onMoveLabel={this.moveLabel}
         onDropLabel={() => this.props.onChange(this.state.values)}
         partitionColumn={this.state.partitionColumn}
@@ -195,6 +198,7 @@ class AdhocFilterControl extends React.Component {
   onRemoveFilter(index) {
     const { confirmDeletion } = this.props;
     const { values } = this.state;
+    const { removeFilter } = this;
     if (confirmDeletion) {
       const { confirmationText, confirmationTitle, triggerCondition } =
         confirmDeletion;
@@ -203,7 +207,7 @@ class AdhocFilterControl extends React.Component {
           title: confirmationTitle,
           content: confirmationText,
           onOk() {
-            this.removeFilter(index);
+            removeFilter(index);
           },
         });
         return;
diff --git a/superset-frontend/src/explore/components/controls/OptionControls/index.tsx b/superset-frontend/src/explore/components/controls/OptionControls/index.tsx
index 495fab71e5..90ae73c637 100644
--- a/superset-frontend/src/explore/components/controls/OptionControls/index.tsx
+++ b/superset-frontend/src/explore/components/controls/OptionControls/index.tsx
@@ -308,10 +308,7 @@ export const OptionControlLabel = ({
       <CloseContainer
         role="button"
         data-test="remove-control-button"
-        onClick={e => {
-          e.stopPropagation();
-          onRemove();
-        }}
+        onClick={onRemove}
       >
         <Icons.XSmall iconColor={theme.colors.grayscale.light1} />
       </CloseContainer>