Remove undefined logins to prevent crash

Expensify · Mar 31, 2023 · 413dd0d · 413dd0d
1 parent 27dca97
commit 413dd0d
Showing 1 changed file with 18 additions and 14 deletions.
diff --git a/src/libs/OptionsListUtils.js b/src/libs/OptionsListUtils.js
@@ -103,22 +103,26 @@ function getPersonalDetailsForLogins(logins, personalDetails) {
     if (!personalDetails) {
         return personalDetailsForLogins;
     }
-    _.each(logins, (login) => {
-        let personalDetail = personalDetails[login];
-        if (!personalDetail) {
-            personalDetail = {
-                login,
-                displayName: Str.removeSMSDomain(login),
-                avatar: ReportUtils.getDefaultAvatar(login),
-            };
-        }
+    _.chain(logins)
+
+        // Somehow it's possible for the logins coming from report.participants to contain undefined values so we use compact to remove them.
+        .compact()
+        .each(logins, (login) => {
+            let personalDetail = personalDetails[login];
+            if (!personalDetail) {
+                personalDetail = {
+                    login,
+                    displayName: Str.removeSMSDomain(login),
+                    avatar: ReportUtils.getDefaultAvatar(login),
+                };
+            }
 
-        if (login === CONST.EMAIL.CONCIERGE) {
-            personalDetail.avatar = CONST.CONCIERGE_ICON_URL;
-        }
+            if (login === CONST.EMAIL.CONCIERGE) {
+                personalDetail.avatar = CONST.CONCIERGE_ICON_URL;
+            }
 
-        personalDetailsForLogins[login] = personalDetail;
-    });
+            personalDetailsForLogins[login] = personalDetail;
+        });
     return personalDetailsForLogins;
 }