fix(CP): ES-156 Fixed the filtering issues with price range filter se…

…lection.
bigcommerce · Apr 10, 2019 · affa3ca · affa3ca
1 parent d5a1b56
commit affa3ca
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 3 deletions.
diff --git a/assets/js/theme/common/faceted-search.js b/assets/js/theme/common/faceted-search.js
@@ -380,10 +380,17 @@ class FacetedSearch {
             return;
         }
 
-        const url = Url.parse(window.location.href);
-        const queryParams = decodeURI($(event.currentTarget).serialize());
+        const url = Url.parse(window.location.href, true);
+        let queryParams = decodeURI($(event.currentTarget).serialize()).split('&');
+        queryParams = urlUtils.parseQueryParams(queryParams);
 
-        urlUtils.goToUrl(Url.format({ pathname: url.pathname, search: `?${queryParams}` }));
+        for (const key in queryParams) {
+            if (queryParams.hasOwnProperty(key)) {
+                url.query[key] = queryParams[key];
+            }
+        }
+
+        urlUtils.goToUrl(Url.format({ pathname: url.pathname, search: urlUtils.buildQueryString(url.query) }));
     }
 
     onStateChange() {

diff --git a/assets/js/theme/common/url-utils.js b/assets/js/theme/common/url-utils.js
@@ -45,6 +45,26 @@ const urlUtils = {
 
         return out.substring(1);
     },
+
+    parseQueryParams: (queryData) => {
+        const params = {};
+
+        for (let i = 0; i < queryData.length; i++) {
+            const temp = queryData[i].split('=');
+
+            if (temp[0] in params) {
+                if (Array.isArray(params[temp[0]])) {
+                    params[temp[0]].push(temp[1]);
+                } else {
+                    params[temp[0]] = [params[temp[0]], temp[1]];
+                }
+            } else {
+                params[temp[0]] = temp[1];
+            }
+        }
+
+        return params;
+    },
 };
 
 export default urlUtils;