abmp-npm 1.1.84 → 1.1.85

package/backend/routers/utils.js

@@ -1,7 +1,7 @@
 const { getMainAddress } = require('../../public/Utils/sharedUtils');
 const { getMemberBySlug } = require('../members-data-methods');
 const {
-  getAddressesByStatus,
+  getMoreAddressesToDisplay,
   formatDateToMonthYear,
   hasStudentMembership,
   isPAC_STAFF,
@@ -48,7 +48,10 @@ function transformMemberToProfileData(member, siteAssociation) {
     ?.map(val => val.license)
     .filter(Boolean)
     .join(', ');
-  const processedAddresses = getAddressesByStatus(member.addresses, member.addressDisplayOption);
+  const moreAddressesToDisplay = getMoreAddressesToDisplay(
+    member.addresses,
+    member.addressDisplayOption
+  );
 
   const memberships = member.memberships || [];
   const siteAssociationMembership = memberships.find(m => m.association === siteAssociation);
@@ -67,7 +70,7 @@ function transformMemberToProfileData(member, siteAssociation) {
     mainAddress,
     testimonials: member.testimonial || [],
     licenceNo,
-    processedAddresses,
+    moreAddressesToDisplay,
     memberSince:
       (member.showABMP &&
         siteAssociationMembership &&

package/backend/utils.js

@@ -4,7 +4,7 @@ const { site } = require('@wix/urls');
 const { encode } = require('ngeohash');
 
 const { COLLECTIONS, ADDRESS_STATUS_TYPES } = require('../public/consts');
-const { formatAddress, generateId } = require('../public/Utils/sharedUtils');
+const { formatAddress, generateId, findMainAddress } = require('../public/Utils/sharedUtils');
 
 const { CONFIG_KEYS, GEO_HASH_PRECISION, MEMBERSHIPS_TYPES } = require('./consts');
 const { wixData } = require('./elevated-modules');
@@ -91,18 +91,17 @@ function getAddressDisplayOptions(member) {
   }
   return displayOptions;
 }
-function getAddressesByStatus(addresses = [], addressDisplayOption = []) {
+function getMoreAddressesToDisplay(addresses = [], addressDisplayOption = []) {
   const visible = addresses.filter(addr => addr.addressStatus !== ADDRESS_STATUS_TYPES.DONT_SHOW);
   if (visible.length < 2) {
     return [];
   }
-  const opts = Array.isArray(addressDisplayOption) ? addressDisplayOption : [];
-  const mainOpt = opts.find(o => o.isMain);
+  const mainAddress = findMainAddress(addressDisplayOption, addresses);
+  const remainingAddressesToFormat = mainAddress
+    ? visible.filter(addr => addr?.key !== mainAddress.key)
+    : visible;
 
-  // Only filter out main address if explicitly set in addressDisplayOption
-  const addressesToFormat = mainOpt ? visible.filter(addr => addr?.key !== mainOpt.key) : visible;
-
-  return addressesToFormat
+  return remainingAddressesToFormat
     .map(addr => {
       const addressString = formatAddress(addr);
       return addressString ? { _id: generateId(), address: addressString } : null;
@@ -216,7 +215,7 @@ module.exports = {
   getSiteBaseUrl,
   encodeXml,
   formatDateOnly,
-  getAddressesByStatus,
+  getMoreAddressesToDisplay,
   isPAC_STAFF,
   searchAllItems,
 };

package/dev-only-scripts/find-duplicate-urls.js

@@ -1,7 +1,6 @@
 const fs = require('fs');
 const path = require('path');
 
-// eslint-disable-next-line import/no-unresolved
 const csv = require('csv-parser');
 
 /**

package/package.json

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

package/pages/Profile.js

@@ -59,9 +59,9 @@ async function profileOnReady({ $w: _$w }) {
   }
 
   function setupAdditionalAddresses() {
-    _$w('#moreAdressesRepeater').data = profileData.processedAddresses;
+    _$w('#moreAdressesRepeater').data = profileData.moreAddressesToDisplay;
 
-    if (profileData.processedAddresses.length > 0) {
+    if (profileData.moreAddressesToDisplay.length > 0) {
       _$w('#moreLocationButton').expand();
       _$w('#addressTitle').collapse();
     }