From 4d4b371fd3fa217eb892c4822a238bcb9f37542d Mon Sep 17 00:00:00 2001
From: Simcha Shats <simcha.shats@nielsen.com>
Date: Tue, 6 Apr 2021 12:29:20 +0300
Subject: [PATCH] refactor: fix CR notes

---
 .../FiltersConfigModal/FiltersConfigForm/DefaultValue.tsx | 4 ++--
 .../src/filters/components/Select/SelectFilterPlugin.tsx  | 8 ++++----
 .../src/filters/components/Select/transformProps.ts       | 4 ++--
 superset-frontend/src/filters/components/Select/types.ts  | 4 ++--
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/DefaultValue.tsx b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/DefaultValue.tsx
index 4101f1e7bb700..34fd7c0d8e40b 100644
--- a/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/DefaultValue.tsx
+++ b/superset-frontend/src/dashboard/components/nativeFilters/FiltersConfigModal/FiltersConfigForm/DefaultValue.tsx
@@ -21,7 +21,7 @@ import {
   Behavior,
   SetDataMaskHook,
   SuperChart,
-  Place,
+  AppSection,
 } from '@superset-ui/core';
 import { FormInstance } from 'antd/lib/form';
 import Loading from 'src/components/Loading';
@@ -61,7 +61,7 @@ const DefaultValue: FC<DefaultValueProps> = ({
     <SuperChart
       height={25}
       width={250}
-      place={Place.FILTER_CONFIG_MODAL}
+      appSection={AppSection.FILTER_CONFIG_MODAL}
       behaviors={[Behavior.NATIVE_FILTER]}
       formData={formData}
       // For charts that don't have datasource we need workaround for empty placeholder
diff --git a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
index e992d9eeeb365..26dde5d03ad66 100644
--- a/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
+++ b/superset-frontend/src/filters/components/Select/SelectFilterPlugin.tsx
@@ -22,7 +22,7 @@ import {
   DataMask,
   ensureIsArray,
   GenericDataType,
-  Place,
+  AppSection,
   t,
   tn,
 } from '@superset-ui/core';
@@ -58,7 +58,7 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
     width,
     behaviors,
     setDataMask,
-    place,
+    appSection,
   } = props;
   const {
     defaultValue,
@@ -72,7 +72,7 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
   } = formData;
 
   const isDefaultToFirstItemInFilterConfigModal =
-    place === Place.FILTER_CONFIG_MODAL && defaultToFirstItem;
+    appSection === AppSection.FILTER_CONFIG_MODAL && defaultToFirstItem;
 
   const groupby = ensureIsArray<string>(formData.groupby);
   // Correct initial value for Ant Select
@@ -86,7 +86,7 @@ export default function PluginFilterSelect(props: PluginFilterSelectProps) {
 
   // If we are in config modal we always need show empty select for `defaultToFirstItem`
   const [values, setValues] = useState<SelectValue>(
-    defaultToFirstItem && place !== Place.FILTER_CONFIG_MODAL
+    defaultToFirstItem && appSection !== AppSection.FILTER_CONFIG_MODAL
       ? firstItem
       : initSelectValue,
   );
diff --git a/superset-frontend/src/filters/components/Select/transformProps.ts b/superset-frontend/src/filters/components/Select/transformProps.ts
index 069c14e26fb80..4bf27d008b426 100644
--- a/superset-frontend/src/filters/components/Select/transformProps.ts
+++ b/superset-frontend/src/filters/components/Select/transformProps.ts
@@ -29,7 +29,7 @@ export default function transformProps(
     queriesData,
     width,
     behaviors,
-    place,
+    appSection,
   } = chartProps;
   const newFormData = { ...DEFAULT_FORM_DATA, ...formData };
   const { setDataMask = () => {} } = hooks;
@@ -42,7 +42,7 @@ export default function transformProps(
 
   return {
     coltypeMap,
-    place,
+    appSection,
     width,
     behaviors,
     height,
diff --git a/superset-frontend/src/filters/components/Select/types.ts b/superset-frontend/src/filters/components/Select/types.ts
index 83f26144001ce..8542e988fa579 100644
--- a/superset-frontend/src/filters/components/Select/types.ts
+++ b/superset-frontend/src/filters/components/Select/types.ts
@@ -17,7 +17,7 @@
  * under the License.
  */
 import {
-  Place,
+  AppSection,
   ChartProps,
   Behavior,
   DataRecord,
@@ -56,7 +56,7 @@ export type PluginFilterSelectProps = PluginFilterStylesProps & {
   data: DataRecord[];
   setDataMask: SetDataMaskHook;
   behaviors: Behavior[];
-  place: Place;
+  appSection: AppSection;
   formData: PluginFilterSelectQueryFormData;
 };