fix: Fix sorting on mobile (#145)

mirumee · Feb 3, 2025 · adc3129 · adc3129
1 parent 869f37c
commit adc3129
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/apps/storefront/src/app/[locale]/(main)/search/actions.ts b/apps/storefront/src/app/[locale]/(main)/search/actions.ts
@@ -27,24 +27,16 @@ export const handleFiltersFormSubmit = async (
   });
 
   passThroughParams.forEach((param) => {
-    const paramsValue = searchParams[param];
     const formValue = formData.get(param) as string;
+    const value = formValue ?? searchParams[param];
 
     // Sorting can be changed either independently (on desktop)
     // or together with filters (on mobile)
     // that is why it needs some custom logic
-    if (param === "sortBy") {
-      const value = formValue ?? paramsValue;
-
-      if (value !== DEFAULT_SORT_BY) {
-        params.set(param, value);
-      } else {
-        params.delete(param);
-      }
-    }
-
-    if (searchParams[param]) {
-      params.set(param, searchParams[param]);
+    if (param === "sortBy" && value === DEFAULT_SORT_BY) {
+      params.delete(param);
+    } else if (value) {
+      params.set(param, value);
     }
   });