Fix: Allow to set the filter via URL query parameter

Drop `usePageFilter` `locationQueryFilterString` because the query param
filter gathered from `useSearchParams` is the only valid value.

Drop `FilterProvider` `locationQuery` prop because a string was passed
instead of the expected query object and because usePageFilter already
handles its use case internally via `useSearchParams`.

src/web/entities/filterprovider.jsx

@@ -13,11 +13,9 @@ const FilterProvider = ({
   fallbackFilter,
   gmpname,
   pageName = gmpname,
-  locationQuery = {},
 }) => {
   const [returnedFilter, isLoadingFilter] = usePageFilter(pageName, gmpname, {
     fallbackFilter,
-    locationQueryFilterString: locationQuery?.filter,
   });
   return (
     <React.Fragment>
@@ -29,9 +27,6 @@ const FilterProvider = ({
 FilterProvider.propTypes = {
   fallbackFilter: PropTypes.filter,
   gmpname: PropTypes.string,
-  locationQuery: PropTypes.shape({
-    filter: PropTypes.string,
-  }),
   pageName: PropTypes.string,
 };
 

src/web/entities/withEntitiesContainer.jsx

@@ -98,7 +98,6 @@ const withEntitiesContainer =
             <FilterProvider
               fallbackFilter={fallbackFilter}
               gmpname={gmpname}
-              locationQuery={searchParams.toString()}
             >
               {({filter}) => (
                 <EntitiesContainerWrapper

src/web/hooks/usePageFilter.js

@@ -46,14 +46,7 @@ const useDefaultFilter = pageName =>
  *          filter and function to reset the filter
  */
 
-const usePageFilter = (
-  pageName,
-  gmpName,
-  {
-    fallbackFilter,
-    locationQueryFilterString: initialLocationQueryFilterString,
-  } = {},
-) => {
+const usePageFilter = (pageName, gmpName, {fallbackFilter} = {}) => {
   const gmp = useGmp();
   const dispatch = useDispatch();
   const [searchParams, setSearchParams] = useSearchParams();
@@ -73,9 +66,7 @@ const usePageFilter = (
 
   // use null as value for not set at all
   let returnedFilter;
-  // only use searchParams directly if locationQueryFilterString is undefined
-  const locationQueryFilterString =
-    initialLocationQueryFilterString || searchParams.get('filter');
+  const locationQueryFilterString = searchParams.get('filter');
 
   const [locationQueryFilter, setLocationQueryFilter] = useState(
     hasValue(locationQueryFilterString)