Nits and refactor

apache · Dec 1, 2023 · 03c80bd · 03c80bd
1 parent 58e0a4e
commit 03c80bd
Showing 1 changed file with 4 additions and 15 deletions.
diff --git a/airflow/www/static/js/dag/nav/FilterBar.tsx b/airflow/www/static/js/dag/nav/FilterBar.tsx
@@ -76,17 +76,6 @@ const FilterBar = () => {
     },
   });
 
-  const runTypeOptionsDeserialized = transformCsvToMultiSelectOptions(
-    filters.runTypeOptions
-  );
-  const runTypeDeserialized = transformCsvToMultiSelectOptions(filters.runType);
-  const runStateOptionsDeserialized = transformCsvToMultiSelectOptions(
-    filters.runStateOptions
-  );
-  const runStateDeserialized = transformCsvToMultiSelectOptions(
-    filters.runState
-  );
-
   return (
     <Flex
       backgroundColor="blackAlpha.200"
@@ -120,7 +109,7 @@ const FilterBar = () => {
         <Box px={2} style={multiSelectBoxStyle}>
           <MultiSelect
             {...multiSelectStyles}
-            value={runTypeDeserialized}
+            value={transformCsvToMultiSelectOptions(filters.runType)}
             onChange={(typeOptions) => {
               if (
                 Array.isArray(typeOptions) &&
@@ -131,15 +120,15 @@ const FilterBar = () => {
                 );
               }
             }}
-            options={runTypeOptionsDeserialized}
+            options={transformCsvToMultiSelectOptions(filters.runTypeOptions)}
             placeholder="All Run Types"
           />
         </Box>
         <Box />
         <Box px={2} style={multiSelectBoxStyle}>
           <MultiSelect
             {...multiSelectStyles}
-            value={runStateDeserialized}
+            value={transformCsvToMultiSelectOptions(filters.runState)}
             onChange={(stateOptions) => {
               if (
                 Array.isArray(stateOptions) &&
@@ -150,7 +139,7 @@ const FilterBar = () => {
                 );
               }
             }}
-            options={runStateOptionsDeserialized}
+            options={transformCsvToMultiSelectOptions(filters.runStateOptions)}
             placeholder="All Run States"
           />
         </Box>