Merge pull request #5270 from matrix-org/t3chguy/rs/3620

null-guard defaultAvatarUrlForString
2020-10-05 12:31:01 +01:00 · 2020-10-05 12:31:01 +01:00 · 4f52afdb3c
parent 01031e33cb cc43de6ebf
commit 4f52afdb3c
1 changed files with 1 additions and 0 deletions
--- a/src/Avatar.js
+++ b/src/Avatar.js
@ -82,6 +82,7 @@ function urlForColor(color) {
 const colorToDataURLCache = new Map();

 export function defaultAvatarUrlForString(s) {
+    if (!s) return ""; // XXX: should never happen but empirically does by evidence of a rageshake
    const defaultColors = ['#0DBD8B', '#368bd6', '#ac3ba8'];
    let total = 0;
    for (let i = 0; i < s.length; ++i) {