From 7a2886f876787a1e1ab66060e1eeb40849bea600 Mon Sep 17 00:00:00 2001
From: Jeff Sieu <jeffsy00@gmail.com>
Date: Tue, 25 Oct 2022 01:17:18 +0800
Subject: [PATCH] [questions][fix] fix search params loading wrongly

---
 .../components/questions/LandingComponent.tsx |  2 +-
 apps/portal/src/pages/questions/browse.tsx    | 92 +++++++++----------
 .../src/utils/questions/useSearchParam.ts     | 22 ++---
 3 files changed, 58 insertions(+), 58 deletions(-)

diff --git a/apps/portal/src/components/questions/LandingComponent.tsx b/apps/portal/src/components/questions/LandingComponent.tsx
index 1007f1e4..cd2ace15 100644
--- a/apps/portal/src/components/questions/LandingComponent.tsx
+++ b/apps/portal/src/components/questions/LandingComponent.tsx
@@ -118,7 +118,7 @@ export default function LandingComponent({
             onClick={() => {
               if (company !== undefined && location !== undefined) {
                 return handleLandingQuery({
-                  company: company.value,
+                  company: company.label,
                   location: location.value,
                   questionType,
                 });
diff --git a/apps/portal/src/pages/questions/browse.tsx b/apps/portal/src/pages/questions/browse.tsx
index cb895f27..37193635 100644
--- a/apps/portal/src/pages/questions/browse.tsx
+++ b/apps/portal/src/pages/questions/browse.tsx
@@ -9,7 +9,6 @@ import { Button, SlideOut } from '@tih/ui';
 
 import QuestionOverviewCard from '~/components/questions/card/question/QuestionOverviewCard';
 import ContributeQuestionCard from '~/components/questions/ContributeQuestionCard';
-import type { FilterOption } from '~/components/questions/filter/FilterSection';
 import FilterSection from '~/components/questions/filter/FilterSection';
 import QuestionSearchBar from '~/components/questions/QuestionSearchBar';
 import CompanyTypeahead from '~/components/questions/typeahead/CompanyTypeahead';
@@ -195,17 +194,17 @@ export default function QuestionsBrowsePage() {
   const [loaded, setLoaded] = useState(false);
   const [filterDrawerOpen, setFilterDrawerOpen] = useState(false);
 
-  const [selectedCompanyOptions, setSelectedCompanyOptions] = useState<
-    Array<FilterOption>
-  >([]);
+  // Const [selectedCompanyOptions, setSelectedCompanyOptions] = useState<
+  //   Array<FilterOption>
+  // >([]);
 
-  const [selectedRoleOptions, setSelectedRoleOptions] = useState<
-    Array<FilterOption>
-  >([]);
+  // const [selectedRoleOptions, setSelectedRoleOptions] = useState<
+  //   Array<FilterOption>
+  // >([]);
 
-  const [selectedLocationOptions, setSelectedLocationOptions] = useState<
-    Array<FilterOption>
-  >([]);
+  // const [selectedLocationOptions, setSelectedLocationOptions] = useState<
+  //   Array<FilterOption>
+  // >([]);
 
   const questionTypeFilterOptions = useMemo(() => {
     return QUESTION_TYPES.map((questionType) => ({
@@ -275,9 +274,37 @@ export default function QuestionsBrowsePage() {
     sortType,
   ]);
 
+  const selectedCompanyOptions = useMemo(() => {
+    return selectedCompanies.map((company) => ({
+      checked: true,
+      id: company,
+      label: company,
+      value: company,
+    }));
+  }, [selectedCompanies]);
+
+  const selectedRoleOptions = useMemo(() => {
+    return selectedRoles.map((role) => ({
+      checked: true,
+      id: role,
+      label: role,
+      value: role,
+    }));
+  }, [selectedRoles]);
+
+  const selectedLocationOptions = useMemo(() => {
+    return selectedLocations.map((location) => ({
+      checked: true,
+      id: location,
+      label: location,
+      value: location,
+    }));
+  }, [selectedLocations]);
+
   if (!loaded) {
     return null;
   }
+
   const filterSidebar = (
     <div className="divide-y divide-slate-200 px-4">
       <Button
@@ -293,9 +320,9 @@ export default function QuestionsBrowsePage() {
           setSelectedQuestionAge('all');
           setSelectedRoles([]);
           setSelectedLocations([]);
-          setSelectedCompanyOptions([]);
-          setSelectedRoleOptions([]);
-          setSelectedLocationOptions([]);
+          // SetSelectedCompanyOptions([]);
+          // setSelectedRoleOptions([]);
+          // setSelectedLocationOptions([]);
         }}
       />
       <FilterSection
@@ -306,8 +333,8 @@ export default function QuestionsBrowsePage() {
             {...field}
             clearOnSelect={true}
             filterOption={(option) => {
-              return !selectedCompanyOptions.some((selectedOption) => {
-                return selectedOption.value === option.value;
+              return !selectedCompanies.some((company) => {
+                return company === option.value;
               });
             }}
             isLabelHidden={true}
@@ -323,19 +350,10 @@ export default function QuestionsBrowsePage() {
         onOptionChange={(option) => {
           if (option.checked) {
             setSelectedCompanies([...selectedCompanies, option.label]);
-            setSelectedCompanyOptions((prevOptions) => [
-              ...prevOptions,
-              { ...option, checked: true },
-            ]);
           } else {
             setSelectedCompanies(
               selectedCompanies.filter((company) => company !== option.label),
             );
-            setSelectedCompanyOptions((prevOptions) =>
-              prevOptions.filter(
-                (prevOption) => prevOption.label !== option.label,
-              ),
-            );
           }
         }}
       />
@@ -347,8 +365,8 @@ export default function QuestionsBrowsePage() {
             {...field}
             clearOnSelect={true}
             filterOption={(option) => {
-              return !selectedRoleOptions.some((selectedOption) => {
-                return selectedOption.value === option.value;
+              return !selectedRoles.some((role) => {
+                return role === option.value;
               });
             }}
             isLabelHidden={true}
@@ -364,19 +382,10 @@ export default function QuestionsBrowsePage() {
         onOptionChange={(option) => {
           if (option.checked) {
             setSelectedRoles([...selectedRoles, option.value]);
-            setSelectedRoleOptions((prevOptions) => [
-              ...prevOptions,
-              { ...option, checked: true },
-            ]);
           } else {
             setSelectedRoles(
               selectedCompanies.filter((role) => role !== option.value),
             );
-            setSelectedRoleOptions((prevOptions) =>
-              prevOptions.filter(
-                (prevOption) => prevOption.value !== option.value,
-              ),
-            );
           }
         }}
       />
@@ -413,8 +422,8 @@ export default function QuestionsBrowsePage() {
             {...field}
             clearOnSelect={true}
             filterOption={(option) => {
-              return !selectedLocationOptions.some((selectedOption) => {
-                return selectedOption.value === option.value;
+              return !selectedLocations.some((location) => {
+                return location === option.value;
               });
             }}
             isLabelHidden={true}
@@ -430,19 +439,10 @@ export default function QuestionsBrowsePage() {
         onOptionChange={(option) => {
           if (option.checked) {
             setSelectedLocations([...selectedLocations, option.value]);
-            setSelectedLocationOptions((prevOptions) => [
-              ...prevOptions,
-              { ...option, checked: true },
-            ]);
           } else {
             setSelectedLocations(
               selectedLocations.filter((role) => role !== option.value),
             );
-            setSelectedLocationOptions((prevOptions) =>
-              prevOptions.filter(
-                (prevOption) => prevOption.value !== option.value,
-              ),
-            );
           }
         }}
       />
diff --git a/apps/portal/src/utils/questions/useSearchParam.ts b/apps/portal/src/utils/questions/useSearchParam.ts
index 3895c5b4..25208e6e 100644
--- a/apps/portal/src/utils/questions/useSearchParam.ts
+++ b/apps/portal/src/utils/questions/useSearchParam.ts
@@ -25,7 +25,7 @@ export const useSearchParam = <Value = string>(
   const [isInitialized, setIsInitialized] = useState(false);
   const router = useRouter();
 
-  const [filters, setFilters] = useState<Array<Value>>(defaultValues || []);
+  const [params, setParams] = useState<Array<Value>>(defaultValues || []);
 
   useEffect(() => {
     if (router.isReady && !isInitialized) {
@@ -33,7 +33,7 @@ export const useSearchParam = <Value = string>(
       const query = router.query[name];
       if (query) {
         const queryValues = Array.isArray(query) ? query : [query];
-        setFilters(
+        setParams(
           queryValues
             .map(stringToParam)
             .filter((value) => value !== null) as Array<Value>,
@@ -43,27 +43,27 @@ export const useSearchParam = <Value = string>(
         const localStorageValue = localStorage.getItem(name);
         if (localStorageValue !== null) {
           const loadedFilters = JSON.parse(localStorageValue);
-          setFilters(loadedFilters);
+          setParams(loadedFilters);
         }
       }
       setIsInitialized(true);
     }
   }, [isInitialized, name, stringToParam, router]);
 
-  const setFiltersCallback = useCallback(
-    (newFilters: Array<Value>) => {
-      setFilters(newFilters);
+  const setParamsCallback = useCallback(
+    (newParams: Array<Value>) => {
+      setParams(newParams);
       localStorage.setItem(
         name,
         JSON.stringify(
-          newFilters.map(valueToQueryParam).filter((param) => param !== null),
+          newParams.map(valueToQueryParam).filter((param) => param !== null),
         ),
       );
     },
     [name, valueToQueryParam],
   );
 
-  return [filters, setFiltersCallback, isInitialized] as const;
+  return [params, setParamsCallback, isInitialized] as const;
 };
 
 export const useSearchParamSingle = <Value = string>(
@@ -73,14 +73,14 @@ export const useSearchParamSingle = <Value = string>(
   },
 ) => {
   const { defaultValue, ...restOpts } = opts ?? {};
-  const [filters, setFilters, isInitialized] = useSearchParam<Value>(name, {
+  const [params, setParams, isInitialized] = useSearchParam<Value>(name, {
     defaultValues: defaultValue !== undefined ? [defaultValue] : undefined,
     ...restOpts,
   } as SearchParamOptions<Value>);
 
   return [
-    filters[0],
-    (value: Value) => setFilters([value]),
+    params[0],
+    (value: Value) => setParams([value]),
     isInitialized,
   ] as const;
 };