fix(gatsby-theme-store): complex plp navigation (#749)

vtex · Jul 1, 2021 · 849f079 · 849f079
1 parent 299baba
commit 849f079
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/packages/gatsby-theme-store/src/sdk/search/converter/useSearchParamsFromURL.ts b/packages/gatsby-theme-store/src/sdk/search/converter/useSearchParamsFromURL.ts
@@ -1,4 +1,3 @@
-import { useLocation } from '@reach/router'
 import { parseSearchParamsState } from '@vtex/store-sdk'
 import { useMemo } from 'react'
 import type { SearchParamsState } from '@vtex/store-sdk'
@@ -11,8 +10,5 @@ export interface SelectedFacets {
 /**
  * @description: Hydrates search context for dynamic search pages.
  */
-export const useSearchParamsFromUrl = (): SearchParamsState => {
-  const { href } = useLocation()
-
-  return useMemo(() => parseSearchParamsState(new URL(href)), [href])
-}
+export const useSearchParamsFromUrl = ({ href }: Location): SearchParamsState =>
+  useMemo(() => parseSearchParamsState(new URL(href)), [href])
diff --git a/packages/gatsby-theme-store/src/sdk/searchBar/controller/index.ts b/packages/gatsby-theme-store/src/sdk/searchBar/controller/index.ts
@@ -59,6 +59,7 @@ export const search = async (term: string) => {
   const path = `/${slugified}`
   const exists = await pathExists(path)
 
+  params.delete('map')
   if (exists) {
     // The page /slugified exists, let's navigate to this page
     pathname = `/${slugified}`

diff --git a/packages/gatsby-theme-store/src/templates/search.browser.tsx b/packages/gatsby-theme-store/src/templates/search.browser.tsx
@@ -21,9 +21,10 @@ const SearchPage: FC<PageProps> = (props) => {
   const {
     pageContext: { hideUnavailableItems },
     pageContext,
+    location,
   } = props
 
-  const searchParams = useSearchParamsFromUrl()
+  const searchParams = useSearchParamsFromUrl(location)
   const variables = useQueryVariablesFromSearchParams(searchParams)
   const redirecting = usePersonalizedSearchRedirect(searchParams)