krikkert/habitica
1
0
Fork 0
mirror of https://github.com/HabitRPG/habitica.git synced 2025-12-17 14:47:53 +01:00
Code Issues Packages Projects Releases Wiki Activity

refactored the highlightUsers-method, only matches mentions and not emails (#10982)

Browse Source
This commit is contained in:
negue
2019-02-10 19:27:22 +01:00
committed by Matteo Pagliazzi
parent 0155491a68
commit b9aaccdf13
2 changed files with 33 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
test/client/unit/specs/libs/highlightUsers.js
View File

@@ -3,9 +3,10 @@ import habiticaMarkdown from 'habitica-markdown';
describe('highlightUserAndEmail', () => {
it('highlights displayname', () => {
const text = 'hello @displayedUser';
const text = 'hello @displayedUser with text after';
const result = highlightUsers(text, 'user', 'displayedUser');
expect(result).to.contain('<span class="at-highlight at-text">@displayedUser</span>');
});
@@ -16,17 +17,23 @@ describe('highlightUserAndEmail', () => {
expect(result).to.contain('<span class="at-highlight at-text">@user</span>');
});
it('highlights email with username', () => {
const text = habiticaMarkdown.render('hello hello@user.com');
it('not highlights any email', () => {
const text = habiticaMarkdown.render('hello@example.com');
const result = highlightUsers(text, 'user', 'displayedUser');
expect(result).to.contain('>hello<span class="at-highlight at-text">@user</span>.com</a>');
const result = highlightUsers(text, 'example', 'displayedUser');
expect(result).to.not.contain('<span class="at-highlight">@example</span>');
});
it('highlights any mention', () => {
const text = habiticaMarkdown.render('hello y.@user.com other words');
const result = highlightUsers(text, 'test', 'displayedUser');
expect(result).to.contain('<span class="at-text">@user</span>');
it('complex highlight', () => {
const plainText = 'a bit more @mentions to @use my@mentions.com broken.@mail.com';
const text = habiticaMarkdown.render(plainText);
const result = highlightUsers(text, 'use', 'mentions');
expect(result).to.contain('<span class="at-highlight at-text">@mentions</span>');
expect(result).to.contain('<span class="at-highlight at-text">@use</span>');
expect(result).to.not.contain('<span class="at-highlight at-text">@mentions</span>.com');
});
});

24
website/client/libs/highlightUsers.js
View File

@@ -1,18 +1,24 @@
import escapeRegExp from 'lodash/escapeRegExp';
const optionalAnchorTagRegExStr = '(<\\w[^>]*)?'; // everything including the anchor tag is recognized
const mentionRegExStr = '(@[\\w-]+)';
const optionalPostMentionRegExStr = '(\\.\\w+)?'; // like dot-TLD
const finalMentionRegEx = new RegExp(`${optionalAnchorTagRegExStr}${mentionRegExStr}${optionalPostMentionRegExStr}`, 'gi');
export function highlightUsers (text, userName, displayName) {
const findAnyMentionRegex = '@[\\w-]+(?:\\b)';
const currentUser = [`@${userName}`, `@${displayName}`].map(escapeRegExp);
const atRegex = new RegExp(`${findAnyMentionRegex}`, 'gi');
const currentUser = [`@${userName}`, `@${displayName}`];
if (atRegex.test(text)) {
text = text.replace(atRegex, match => {
if (currentUser.includes(match)) {
return `<span class="at-highlight at-text">${match}</span>`;
text = text.replace(finalMentionRegEx, (fullMatched, preMention, mentionStr, postMention) => {
if (preMention && preMention.includes('<a') || Boolean(postMention)) {
return fullMatched;
}
return `<span class="at-text">${match}</span>`;
const isUserMention = currentUser.includes(mentionStr) ? 'at-text' : '';
return fullMatched.replace(mentionStr, `<span class="at-highlight ${isUserMention}">${mentionStr}</span>`);
});
}
return text;
}