@salesforcedevs/dx-components 1.2.6-avatar-button-4 → 1.2.6-avatar-button-6

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforcedevs/dx-components",
-    "version": "1.2.6-avatar-button-4",
+    "version": "1.2.6-avatar-button-6",
     "description": "DX Lightning web components",
     "license": "MIT",
     "engines": {

package/src/modules/dx/avatarButton/avatarButton.ts

@@ -311,38 +311,43 @@ export default class AvatarButton extends LightningElement {
             return;
         }
 
-        const nextUserInfo: UserInfo = {};
-        nextUserInfo.avatarImgSrc = userInfo.photos?.thumbnail;
-        nextUserInfo.firstName = userInfo.given_name;
-        nextUserInfo.lastName = userInfo.family_name;
-        nextUserInfo.username = userInfo.preferred_username;
-        nextUserInfo.id = userInfo.user_id;
-        nextUserInfo.orgId = userInfo.organization_id;
-        nextUserInfo.fullName =
-            nextUserInfo.firstName && nextUserInfo.lastName
-                ? `${nextUserInfo.firstName} ${nextUserInfo.lastName}`
-                : nextUserInfo.firstName || nextUserInfo.lastName || "";
+        const uncleanNextUserInfo: UserInfo = {};
+        uncleanNextUserInfo.avatarImgSrc = userInfo.photos?.thumbnail;
+        uncleanNextUserInfo.firstName = userInfo.given_name;
+        uncleanNextUserInfo.lastName = userInfo.family_name;
+        uncleanNextUserInfo.username = userInfo.preferred_username;
+        uncleanNextUserInfo.id = userInfo.user_id;
+        uncleanNextUserInfo.orgId = userInfo.organization_id;
+        uncleanNextUserInfo.fullName =
+            uncleanNextUserInfo.firstName && uncleanNextUserInfo.lastName
+                ? `${uncleanNextUserInfo.firstName} ${uncleanNextUserInfo.lastName}`
+                : uncleanNextUserInfo.firstName ||
+                  uncleanNextUserInfo.lastName ||
+                  "";
 
         if (userInfo.custom_attributes) {
             // ProfilePictureUrl updates earlier than photos.thumbnail, so we prefer it if it's available here.
-            nextUserInfo.avatarImgSrc =
+            uncleanNextUserInfo.avatarImgSrc =
                 userInfo.custom_attributes.ProfilePictureUrl ||
-                nextUserInfo.avatarImgSrc;
-            nextUserInfo.company = userInfo.custom_attributes.CompanyName;
-            nextUserInfo.relationship =
+                uncleanNextUserInfo.avatarImgSrc;
+            uncleanNextUserInfo.company =
+                userInfo.custom_attributes.CompanyName;
+            uncleanNextUserInfo.relationship =
                 userInfo.custom_attributes.RelationshipToSalesforce;
-            nextUserInfo.role = userInfo.custom_attributes.Role;
+            uncleanNextUserInfo.role = userInfo.custom_attributes.Role;
         }
 
-        // A little extra safety here just to make sure we aren't receiving/displaying any unsafe values...
-        const htmlSafeNextUserInfo: UserInfo = Object.fromEntries(
-            Object.entries(nextUserInfo).map(([key, val]) => [
-                key,
-                escapeHtml(val)
-            ])
-        );
+        const cleanUserInfo: UserInfo = {};
+        Object.entries(uncleanNextUserInfo).forEach(([key, val]) => {
+            // uncleanUserInfo may have undefined values which we don't want in the final product
+            if (val && val !== "undefined") {
+                // Also a little extra safety to make sure we aren't injecting anything unsafe...
+                cleanUserInfo[key as keyof UserInfo] = escapeHtml(val);
+            }
+        });
 
-        this.userInfo = defaults(htmlSafeNextUserInfo, this.userInfo);
+        // Keep the old info for any values _not_ defined on `cleanUserInfo`
+        this.userInfo = defaults(cleanUserInfo, this.userInfo);
     };
 
     private requestUserInfo = async () => {