improve regex, fixes replacements of usernames with same trunk

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

1 files changed, 12 insertions, 1 deletions

diff --git a/apps/comments/js/commentstabview.js b/apps/comments/js/commentstabview.js
index 23f2bc489e4..9c0e6a6b68c 100644
--- a/apps/comments/js/commentstabview.js
+++ b/apps/comments/js/commentstabview.js
@@ -256,9 +256,20 @@
 
 			for(var i in mentions) {
 				var mention = '@' + mentions[i].mentionId;
+
+
+				// escape possible regex characters in the name
 				mention = mention.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
 				var displayName = '<b>'+ _.escape(mentions[i].mentionDisplayName)+'</b>';
-				message = message.replace(new RegExp(mention, 'g'), displayName);
+
+				// replace every mention either at the start of the input or after a whitespace
+				// followed by a non-word character.
+				message = message.replace(new RegExp("(^|\\s)(" + mention + ")(\\b|?![_-@.])", 'g'),
+					function(match, p1) {
+						// to  get number of whitespaces (0 vs 1) right
+						return p1+displayName;
+					}
+				);
 			}
 
 			return message;