homeflowjs 0.12.13 → 0.12.15

package/actions/properties.actions.js

@@ -14,7 +14,18 @@ export const setPropertyLinksAsync = () => (dispatch) => {
   const searchHistory = localStorage.getItem('searchHistory');
   if (!searchHistory) return null;
 
-  const lastSearch = JSON.parse(searchHistory)[0];
+  const parsedSearchHistory = JSON.parse(searchHistory);
+  if (!Array.isArray(parsedSearchHistory)) return null;
+
+  const validateSearch = (search) =>
+    (!search.isQuerySearch && search?.q)
+    || search?.place
+    || (search.isQuerySearch && search?.placeId)
+    || (search.isQuerySearch && search?.poly);
+
+  const filteredSearchHistory = parsedSearchHistory.filter(validateSearch)
+  if (!filteredSearchHistory?.length) return null;
+  const lastSearch = filteredSearchHistory[0];
   const { place, ...restOfSearch } = lastSearch;
   const searchURL = `/search.ljson?${buildQueryString(restOfSearch)}`;
 
@@ -23,6 +34,8 @@ export const setPropertyLinksAsync = () => (dispatch) => {
     prev: '',
   };
 
+  const getPropertyUrl = (property) => property?.property_url || property?.url;
+
   fetch(searchURL)
     .then((response) => response.json())
     .then(({ properties }) => {
@@ -31,8 +44,8 @@ export const setPropertyLinksAsync = () => (dispatch) => {
       const index = properties.findIndex((prop) => prop.property_id === Homeflow.get('property').property_id);
       // if there is a next and previous property in page, set those
       if (properties[index + 1] && properties[index - 1]) {
-        propertyLinks.next = properties[index + 1].property_url;
-        propertyLinks.prev = properties[index - 1].property_url;
+        propertyLinks.next = getPropertyUrl(properties[index + 1]);
+        propertyLinks.prev = getPropertyUrl(properties[index - 1]);
 
         return dispatch(setPropertyLinks(propertyLinks));
       }
@@ -40,7 +53,7 @@ export const setPropertyLinksAsync = () => (dispatch) => {
       // if there is no next but there is a previous, conduct search for next page
       // but use previous
       if (!properties[index + 1] && properties[index - 1]) {
-        propertyLinks.prev = properties[index - 1].property_url;
+        propertyLinks.prev = getPropertyUrl(properties[index - 1]);
 
         const nextSearch = {
           ...restOfSearch,
@@ -54,7 +67,7 @@ export const setPropertyLinksAsync = () => (dispatch) => {
             if (!properties) {
               propertyLinks.next = '';
             } else {
-              propertyLinks.next = properties[0].property_url;
+              propertyLinks.next = getPropertyUrl(properties[0]);
             }
 
             return dispatch(setPropertyLinks(propertyLinks));
@@ -64,7 +77,7 @@ export const setPropertyLinksAsync = () => (dispatch) => {
       // if there is no previous but there is a next, conduct search for previous page
       // but use next
       if (!properties[index - 1] && properties[index + 1]) {
-        propertyLinks.next = properties[index + 1].property_url;
+        propertyLinks.next = getPropertyUrl(properties[index + 1]);
 
         const nextSearch = {
           ...restOfSearch,
@@ -85,7 +98,7 @@ export const setPropertyLinksAsync = () => (dispatch) => {
             if (!properties) {
               propertyLinks.prev = '';
             } else {
-              propertyLinks.prev = properties[properties.length - 1].property_url;
+              propertyLinks.prev = getPropertyUrl(properties[properties.length - 1]);
             }
 
             return dispatch(setPropertyLinks(propertyLinks));

package/cookie-consent/cookie-consent.js

@@ -3,7 +3,7 @@ const initCookieConsent = () => {
     const defaultMessage = 'This website uses cookies to improve your experience.';
     // appends the 'decline' and 'learn more' links to the message
     const ccFullscreenMessage = (message = defaultMessage, href) => {
-      let newMessage = `${message}<a aria-label="learn more about cookies" tabindex="0" class="cc-link" href="${href}" target="_blank">Learn more</a>`;
+      let newMessage = `${message}<a aria-label="learn more about cookies" tabindex="0" class="cc-link" href="${href}" target="_blank">View cookie policy</a>`;
       newMessage += ' or <a aria-label="deny cookies" role="button" tabindex="0" class="cc-btn cc-deny cc-deny-link">decline</a>';
       return newMessage;
     };
@@ -56,8 +56,8 @@ const initCookieConsent = () => {
             message: ccFullscreenMessage(message, href),
           },
           elements: {
-            deny: '<a style="display: none;"></a>',
-            link: '<a style="display: none;"></a>',
+            deny: '<a style="display: none;" href="#"></a>',
+            link: '<a style="display: none;" href="#"></a>',
           },
         };
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homeflowjs",
-  "version": "0.12.13",
+  "version": "0.12.15",
   "sideEffects": [
     "modal/**/*",
     "user/default-profile/**/*",