abmp-npm 2.0.46 → 2.0.47

package/backend/members-data-methods.js

@@ -410,7 +410,6 @@ async function getMembersWithWixUrl() {
     .eq('showWixUrl', true)
     .ne('action', MEMBER_ACTIONS.DROP)
     .ne('memberships.membertype', MEMBERSHIPS_TYPES.PAC_STAFF)
-    .ne('memberships.membertype', MEMBERSHIPS_TYPES.STUDENT)
     .isNotEmpty('url')
     .limit(1000);
   let currentResults = await membersQuery.find();

package/backend/routers/methods.js

@@ -47,12 +47,10 @@ const createRoutersHandlers = wixRouterMethods => {
         }
         const profileUrl = `${request.baseUrl}/${PAGES_PATHS.PROFILE}/${profileData.url}`;
         const isPrivateMember = profileData.isPrivateMember;
-        const isStudent = profileData.shouldHaveStudentBadge;
-        const shouldNoIndex = isPrivateMember || isStudent;
         const seoData = {
           title: seoTitle,
           description: description,
-          noIndex: shouldNoIndex,
+          noIndex: isPrivateMember,
           metaTags: [
             {
               name: 'description',
@@ -71,7 +69,7 @@ const createRoutersHandlers = wixRouterMethods => {
             },
             {
               name: 'robots',
-              content: shouldNoIndex ? 'noindex, nofollow' : 'index, follow',
+              content: isPrivateMember ? 'noindex, nofollow' : 'index, follow',
             },
             // Open Graph tags
             {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "abmp-npm",
-  "version": "2.0.46",
+  "version": "2.0.47",
   "main": "index.js",
   "scripts": {
     "check-cycles": "madge --circular .",

package/pages/Home.js

@@ -366,7 +366,7 @@ const homePageOnReady = async ({
       isSearchingNearby: _$w('#nearBy').checked,
       preservePagination,
     });
-    // URL is updated inside search() with the filter actually used (avoids rapid select/deselect overwriting with live filter)
+    !preservePagination && (await updateUrlParams(filter, pagination));
     return searchResults;
   }
 

package/pages/personalDetails.js

@@ -358,8 +358,8 @@ async function personalDetailsOnReady({
   }
 
   function handleOptWebsiteCheckboxEnable(showExistingWebsite) {
-    _$w('#optWebsiteCheckbox').checked = !showExistingWebsite;
     if (showExistingWebsite) {
+      _$w('#optWebsiteCheckbox').checked = false;
       _$w('#optWebsiteCheckbox').customClassList.add('disabled-text');
       _$w('#optWebsiteCheckbox').customClassList.add('disabled-checkbox');
       _$w('#optWebsiteCheckbox').disable();

package/public/Utils/homePage.js

@@ -1,24 +1,11 @@
 const { location: wixLocation, queryParams: wixQueryParams } = require('@wix/site-location');
 const { window: wixWindow, rendering } = require('@wix/site-window');
 
-const { DEFAULT_FILTER, DEBOUNCE_DELAY } = require('../consts.js');
+const { DEFAULT_FILTER } = require('../consts.js');
 
-function generateSearchId() {
-  return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, c => {
-    const r = (Math.random() * 16) | 0;
-    const v = c === 'x' ? r : (r & 0x3) | 0x8;
-    return v.toString(16);
-  });
-}
+const { debouncedFunction } = require('./sharedUtils.js');
 
 const createHomepageUtils = (_$w, filterProfiles) => {
-  let currentSearchId = null;
-  let searchDebounceTimer = null;
-  let lastSearchFilter = null;
-  let lastSearchIsSearchingNearby = false;
-  let lastSearchPreservePagination = false;
-  let pendingSearchResolve = null;
-
   const getFiltersSelectors = filterName => ({
     checkBoxContainerSelector: _$w(`#${filterName}CheckBoxContainer`),
     searchTextInputSelector: _$w(`#${filterName}TextInput`),
@@ -668,14 +655,13 @@ const createHomepageUtils = (_$w, filterProfiles) => {
   async function search({
     filter,
     pagination,
-    debounceTimeout: _debounceTimeout,
+    debounceTimeout,
     timeoutType,
     isSearchingNearby,
     preservePagination = false,
   }) {
     const multiStateBoxSelector = _$w('#resultsStateBox');
     const renderingEnv = await rendering.env();
-
     const initSearchResultsUI = () => {
       JSON.stringify(filter) === JSON.stringify(DEFAULT_FILTER)
         ? _$w('#resetFilter').hide()
@@ -686,115 +672,70 @@ const createHomepageUtils = (_$w, filterProfiles) => {
       _$w('#profileRepeater').data = [];
       console.log({ filter });
     };
-
-    const runSearchAndUpdateUI = async (
-      filterToUse,
-      isSearchingNearbyToUse,
-      preservePaginationToUse
-    ) => {
-      if (!isSearchingNearbyToUse) {
+    const runSearchAndUpdateUI = async (filter, isSearchingNearby) => {
+      if (!isSearchingNearby) {
         if (
           JSON.stringify({
-            ...filterToUse,
+            ...filter,
             latitude: 0,
             longitude: 0,
           }) === JSON.stringify(DEFAULT_FILTER)
         ) {
           multiStateBoxSelector.changeState('noSearchCriteria');
-          await updateUrlParams(filterToUse, pagination);
           return [];
         }
       }
-      const thisSearchId = generateSearchId();
-      currentSearchId = thisSearchId;
-
-      let result;
-      try {
-        result = await filterProfiles({
-          filter: filterToUse,
-          isSearchingNearby: isSearchingNearbyToUse,
-        });
-      } catch (error) {
-        if (thisSearchId !== currentSearchId) return [];
+      const nonDebouncedFilterProfiles = async () => {
+        try {
+          const result = await filterProfiles({ filter, isSearchingNearby });
+          return { success: true, response: result };
+        } catch (error) {
+          return { success: false, error };
+        }
+      };
+      //Don't run setTimeout on SSR
+      const funcPromise =
+        renderingEnv === 'backend'
+          ? () => nonDebouncedFilterProfiles()
+          : () =>
+              debouncedFunction({
+                func: filterProfiles,
+                debounceTimeout,
+                timeoutType,
+                args: { filter, isSearchingNearby },
+              });
+      const { success, response, error } = await funcPromise();
+      if (!success) {
         _$w('#numberOfResults').text = '';
         console.error('[search] failed with error:', error);
         multiStateBoxSelector.changeState('errorState');
-        await updateUrlParams(filterToUse, pagination);
         return [];
       }
-
-      if (thisSearchId !== currentSearchId) return [];
-
-      const response = result;
       const totalCount = response.items.length;
       if (!totalCount) {
         _$w('#numberOfResults').text = 'Showing 0 results';
         _$w('#noResultsMessage').text = `${
-          filterToUse.searchText && filterToUse.searchText.length > 0
-            ? `'${filterToUse.searchText}' did not match any search. Please try again.`
+          filter.searchText && filter.searchText.length > 0
+            ? `'${filter.searchText}' did not match any search. Please try again.`
             : 'No results found for the selected filters. Please adjust your filters and try again'
         }`;
         multiStateBoxSelector.changeState('noResultsState');
-        await updateUrlParams(filterToUse, pagination);
         return [];
       }
       console.log({ response });
       handleNumberOfResults(pagination, totalCount);
       _$w('#showingResult').show();
 
-      if (!preservePaginationToUse || pagination.currentPage >= pagination.totalPages) {
+      if (!preservePagination || pagination.currentPage >= pagination.totalPages) {
         pagination.currentPage = 0;
       }
       pagination.totalPages = Math.ceil(totalCount / pagination.pageSize);
       paginateSearchResults(response.items, pagination);
       multiStateBoxSelector.changeState('resultsState');
-      await updateUrlParams(filterToUse, pagination);
       return response.items;
     };
-
-    // Always show loading as soon as user changes input
     initSearchResultsUI();
-
-    // SSR: run immediately, no debounce
-    if (renderingEnv === 'backend') {
-      return await runSearchAndUpdateUI(filter, isSearchingNearby, preservePagination);
-    }
-
-    // Client: debounce the API call; loading is already shown above.
-    // Snapshot the filter so rapid clicks / URL sync can't mutate it before the debounced run.
-    lastSearchFilter = JSON.parse(JSON.stringify(filter));
-    lastSearchIsSearchingNearby = isSearchingNearby;
-    lastSearchPreservePagination = preservePagination;
-
-    if (pendingSearchResolve) {
-      pendingSearchResolve([]);
-      pendingSearchResolve = null;
-    }
-
-    if (searchDebounceTimer) {
-      clearTimeout(searchDebounceTimer);
-      searchDebounceTimer = null;
-    }
-
-    const delay = DEBOUNCE_DELAY[timeoutType] ?? 300;
-    return new Promise(resolve => {
-      pendingSearchResolve = resolve;
-      searchDebounceTimer = setTimeout(async () => {
-        searchDebounceTimer = null;
-        const filterToUse = lastSearchFilter;
-        const isSearchingNearbyToUse = lastSearchIsSearchingNearby;
-        const preservePaginationToUse = lastSearchPreservePagination;
-        const items = await runSearchAndUpdateUI(
-          filterToUse,
-          isSearchingNearbyToUse,
-          preservePaginationToUse
-        );
-        if (pendingSearchResolve) {
-          pendingSearchResolve(items);
-          pendingSearchResolve = null;
-        }
-      }, delay);
-    });
+    return await runSearchAndUpdateUI(filter, isSearchingNearby);
   }
 
   return {

package/public/Utils/sharedUtils.js

@@ -87,11 +87,9 @@ function findMainAddress(addressDisplayOption = [], addresses = []) {
   return '';
 }
 function formatAddress(item) {
-  if (!item) return '';
   let addressParts = [];
-  const limitedPostalCode = (item.postalcode && String(item.postalcode).slice(0, 5)) || ''; //show only 5 digits to not show full user address
-  const status = item.addressStatus;
-  switch (status) {
+  const limitedPostalCode = item.postalcode.slice(0, 5); //show only 5 digits to not show full user address
+  switch (item.addressStatus) {
     case ADDRESS_STATUS_TYPES.FULL_ADDRESS:
       addressParts = [item.line1, item.line2, item.city, item.state, limitedPostalCode];
       break;
@@ -99,10 +97,7 @@ function formatAddress(item) {
       addressParts = [item.city, item.state, limitedPostalCode];
       break;
     default:
-      if (status === ADDRESS_STATUS_TYPES.DONT_SHOW) return '';
-      // Legacy addresses may have no addressStatus; show city/state/zip by default
-      addressParts = [item.city, item.state, limitedPostalCode];
-      break;
+      return '';
   }
   return addressParts.filter(Boolean).join(', ');
 }