Merge pull request #2424 from hmhealey/escape

Changed mention parser to ignore empty string mentions

1 files changed, 5 insertions, 0 deletions

diff --git a/webapp/utils/text_formatting.jsx b/webapp/utils/text_formatting.jsx
index 9833b995c..4c8b5e24c 100644
--- a/webapp/utils/text_formatting.jsx
+++ b/webapp/utils/text_formatting.jsx
@@ -213,6 +213,11 @@ function highlightCurrentMentions(text, tokens) {
     }
 
     for (const mention of UserStore.getCurrentMentionKeys()) {
+        // occasionally we get an empty mention which matches a bunch of empty strings
+        if (!mention) {
+            continue;
+        }
+
         output = output.replace(new RegExp(`(^|\\W)(${escapeRegex(mention)})\\b`, 'gi'), replaceCurrentMentionWithToken);
     }