Merge pull request #25816 from margelo/perf/fix-avatar-calculation

Performance: optimize getDefaultAvatar calculation
Expensify · Aug 24, 2023 · 329cac3 · 329cac3
2 parents 85032a8 + bd1dafc
commit 329cac3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/libs/UserUtils.js b/src/libs/UserUtils.js
@@ -88,8 +88,8 @@ function getDefaultAvatar(accountID = -1) {
     }
 
     // There are 24 possible default avatars, so we choose which one this user has based
-    // on a simple hash of their login. Note that Avatar count starts at 1.
-    const accountIDHashBucket = hashText(accountID.toString(), CONST.DEFAULT_AVATAR_COUNT) + 1;
+    // on a simple modulo operation of their login number. Note that Avatar count starts at 1.
+    const accountIDHashBucket = (accountID % CONST.DEFAULT_AVATAR_COUNT) + 1;
 
     return defaultAvatars[`Avatar${accountIDHashBucket}`];
 }