Escape special characters in GFM auto complete highlighting

Fixes #60552
This commit is contained in:
Jan Beckmann 2019-04-25 08:11:20 +00:00 committed by Phil Hughes
parent 3fe07ce7d1
commit 5848f2833a
3 changed files with 47 additions and 0 deletions

View file

@ -477,6 +477,16 @@ class GfmAutoComplete {
} }
return null; return null;
}, },
highlighter(li, query) {
// override default behaviour to escape dot character
// see https://github.com/ichord/At.js/pull/576
if (!query) {
return li;
}
const escapedQuery = query.replace(/[.+]/, '\\$&');
const regexp = new RegExp(`>\\s*([^<]*?)(${escapedQuery})([^<]*)\\s*<`, 'ig');
return li.replace(regexp, (str, $1, $2, $3) => `> ${$1}<strong>${$2}</strong>${$3} <`);
},
}; };
} }

View file

@ -0,0 +1,5 @@
---
title: Fix autocomplete dropdown for usernames starting with period
merge_request: 27533
author: Jan Beckmann
type: fixed

View file

@ -209,6 +209,38 @@ describe('GfmAutoComplete', () => {
}); });
}); });
describe('DefaultOptions.highlighter', () => {
beforeEach(() => {
atwhoInstance = { setting: {} };
});
it('should return li if no query is given', () => {
const liTag = '<li></li>';
const highlightedTag = gfmAutoCompleteCallbacks.highlighter.call(atwhoInstance, liTag);
expect(highlightedTag).toEqual(liTag);
});
it('should highlight search query in li element', () => {
const liTag = '<li><img src="" />string</li>';
const query = 's';
const highlightedTag = gfmAutoCompleteCallbacks.highlighter.call(atwhoInstance, liTag, query);
expect(highlightedTag).toEqual('<li><img src="" /> <strong>s</strong>tring </li>');
});
it('should highlight search query with special char in li element', () => {
const liTag = '<li><img src="" />te.st</li>';
const query = '.';
const highlightedTag = gfmAutoCompleteCallbacks.highlighter.call(atwhoInstance, liTag, query);
expect(highlightedTag).toEqual('<li><img src="" /> te<strong>.</strong>st </li>');
});
});
describe('isLoading', () => { describe('isLoading', () => {
it('should be true with loading data object item', () => { it('should be true with loading data object item', () => {
expect(GfmAutoComplete.isLoading({ name: 'loading' })).toBe(true); expect(GfmAutoComplete.isLoading({ name: 'loading' })).toBe(true);