From 012462729db705c19675e9df8b408db8fc90537a Mon Sep 17 00:00:00 2001
From: Hannu Lyytikainen <hannu@sharetribe.com>
Date: Thu, 15 Feb 2018 17:12:35 +0200
Subject: [PATCH] Remove unnecessary listing query params

---
 src/containers/SearchPage/SearchPage.js        | 12 ++++++------
 .../__snapshots__/SearchPage.test.js.snap      | 18 ++----------------
 2 files changed, 8 insertions(+), 22 deletions(-)

diff --git a/src/containers/SearchPage/SearchPage.js b/src/containers/SearchPage/SearchPage.js
index 4c3c3a94..8955b37b 100644
--- a/src/containers/SearchPage/SearchPage.js
+++ b/src/containers/SearchPage/SearchPage.js
@@ -75,10 +75,8 @@ const validURLParamForExtendedData = (paramKey, urlParams) => {
 const pickSearchParamsOnly = params => {
   const { address, origin, bounds, country, ...rest } = params || {};
   return {
-    address,
     origin,
     bounds,
-    country,
     ...validURLParamForExtendedData(CATEGORY_URL_PARAM, rest),
     ...validURLParamForExtendedData(AMENITIES_URL_PARAM, rest),
   };
@@ -174,9 +172,11 @@ export class SearchPageComponent extends Component {
       latlng: ['origin'],
       latlngBounds: ['bounds'],
     });
+
     const perPage = SHARETRIBE_API_MAX_PAGE_SIZE;
     const page = 1;
-    const searchParamsForMapResults = { ...searchInURL, include: ['images'], page, perPage };
+    const { address, country, ...rest } = searchInURL;
+    const searchParamsForMapResults = { ...rest, include: ['images'], page, perPage };
     this.searchMapListingsInProgress = true;
 
     // Search more listings for map
@@ -333,7 +333,7 @@ export class SearchPageComponent extends Component {
           <div className={css.searchResultContainer}>
             <SearchFilters
               className={css.searchFilters}
-              urlQueryParams={urlQueryParams}
+              urlQueryParams={searchInURL}
               listingsAreLoaded={listingsAreLoaded}
               resultsCount={totalItems}
               searchInProgress={searchInProgress}
@@ -345,7 +345,7 @@ export class SearchPageComponent extends Component {
             />
             <SearchFiltersMobile
               className={css.searchFiltersMobile}
-              urlQueryParams={urlQueryParams}
+              urlQueryParams={searchInURL}
               listingsAreLoaded={listingsAreLoaded}
               resultsCount={totalItems}
               searchInProgress={searchInProgress}
@@ -476,7 +476,7 @@ SearchPage.loadData = (params, search) => {
     latlng: ['origin'],
     latlngBounds: ['bounds'],
   });
-  const { page = 1, ...rest } = queryParams;
+  const { page = 1, address, country, ...rest } = queryParams;
   return searchListings({
     ...rest,
     page,
diff --git a/src/containers/SearchPage/__snapshots__/SearchPage.test.js.snap b/src/containers/SearchPage/__snapshots__/SearchPage.test.js.snap
index 339ea655..82d5c319 100644
--- a/src/containers/SearchPage/__snapshots__/SearchPage.test.js.snap
+++ b/src/containers/SearchPage/__snapshots__/SearchPage.test.js.snap
@@ -51,14 +51,7 @@ exports[`SearchPageComponent matches snapshot 1`] = `
         resultsCount={22}
         searchInProgress={false}
         searchListingsError={null}
-        urlQueryParams={
-          Object {
-            "address": undefined,
-            "bounds": undefined,
-            "country": undefined,
-            "origin": undefined,
-          }
-        }
+        urlQueryParams={Object {}}
       />
       <InjectIntl(withRouter(SearchFiltersMobileComponent))
         amenities={
@@ -94,14 +87,7 @@ exports[`SearchPageComponent matches snapshot 1`] = `
         searchInProgress={false}
         searchListingsError={null}
         showAsModalMaxWidth={768}
-        urlQueryParams={
-          Object {
-            "address": undefined,
-            "bounds": undefined,
-            "country": undefined,
-            "origin": undefined,
-          }
-        }
+        urlQueryParams={Object {}}
       />
       <div
         className=""