From bc2849898b230bf808b8716e8d9e629db2a76f52 Mon Sep 17 00:00:00 2001
From: Ville Brofeldt <ville.v.brofeldt@gmail.com>
Date: Thu, 11 Mar 2021 10:54:57 +0200
Subject: [PATCH] remove from useEffect

---
 .../filters/components/Select/SelectFilterPlugin.tsx |  3 ---
 .../src/filters/components/Select/types.ts           | 12 ++----------
 2 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
index 641d9e9100ae6..7238c2faa8125 100644
--- a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
+++ b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
@@ -35,7 +35,6 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
     currentValue,
     inverseSelection,
     inputRef,
-    sortAscending,
   } = formData;
 
   const [values, setValues] = useState<(string | number)[]>(defaultValue ?? []);
@@ -83,7 +82,6 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
     handleChange(currentValue ?? []);
   }, [
     JSON.stringify(currentValue),
-    sortAscending,
     multiSelect,
     enableEmptyFilter,
     inverseSelection,
@@ -93,7 +91,6 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
     handleChange(defaultValue ?? []);
   }, [
     JSON.stringify(defaultValue),
-    sortAscending,
     multiSelect,
     enableEmptyFilter,
     inverseSelection,
diff --git a/superset-frontend/src/filters/components/Select/types.ts b/superset-frontend/src/filters/components/Select/types.ts
index c5186782c778b..7ab08b5c3cbb8 100644
--- a/superset-frontend/src/filters/components/Select/types.ts
+++ b/superset-frontend/src/filters/components/Select/types.ts
@@ -18,9 +18,7 @@
  */
 import {
   Behavior,
-  ChartDataResponseResult,
-  ChartProps,
-  DataRecord, GenericDataType,
+  DataRecord,
   QueryFormData,
   SetDataMaskHook,
 } from '@superset-ui/core';
@@ -41,17 +39,11 @@ export type PluginFilterSelectQueryFormData = QueryFormData &
   PluginFilterStylesProps &
   PluginFilterSelectCustomizeProps;
 
-export interface PluginFilterSelectChartProps extends ChartProps {
-  queriesData: ChartDataResponseResult[];
-}
-
 export type PluginFilterSelectProps = PluginFilterStylesProps & {
-  coltypeMap: Record<string, GenericDataType>;
   data: DataRecord[];
   setDataMask: SetDataMaskHook;
   behaviors: Behavior[];
   formData: PluginFilterSelectQueryFormData;
-  queriesData: ChartDataResponseResult[];
 };
 
 export const DEFAULT_FORM_DATA: PluginFilterSelectCustomizeProps = {
@@ -59,6 +51,6 @@ export const DEFAULT_FORM_DATA: PluginFilterSelectCustomizeProps = {
   currentValue: null,
   enableEmptyFilter: false,
   inverseSelection: false,
-  multiSelect: false,
+  multiSelect: true,
   sortAscending: true,
 };