fix: missing the new arabic diacritics settings in a few places (#373)
This commit is contained in:
@@ -6,7 +6,10 @@ describe('The Query class', () => {
|
|||||||
|
|
||||||
it('should correctly parse string queries', () => {
|
it('should correctly parse string queries', () => {
|
||||||
// Act
|
// Act
|
||||||
const query = new Query(stringQuery, { ignoreDiacritics: true })
|
const query = new Query(stringQuery, {
|
||||||
|
ignoreDiacritics: true,
|
||||||
|
ignoreArabicDiacritics: true,
|
||||||
|
})
|
||||||
|
|
||||||
// Assert
|
// Assert
|
||||||
const segments = query.query.text
|
const segments = query.query.text
|
||||||
@@ -25,7 +28,10 @@ describe('The Query class', () => {
|
|||||||
|
|
||||||
it('should not exclude words when there is no space before', () => {
|
it('should not exclude words when there is no space before', () => {
|
||||||
// Act
|
// Act
|
||||||
const query = new Query('foo bar-baz', { ignoreDiacritics: true })
|
const query = new Query('foo bar-baz', {
|
||||||
|
ignoreDiacritics: true,
|
||||||
|
ignoreArabicDiacritics: true,
|
||||||
|
})
|
||||||
|
|
||||||
// Assert
|
// Assert
|
||||||
expect(query.query.exclude.text).toHaveLength(0)
|
expect(query.query.exclude.text).toHaveLength(0)
|
||||||
@@ -34,7 +40,10 @@ describe('The Query class', () => {
|
|||||||
describe('.getExactTerms()', () => {
|
describe('.getExactTerms()', () => {
|
||||||
it('should an array of strings containg "exact" values', () => {
|
it('should an array of strings containg "exact" values', () => {
|
||||||
// Act
|
// Act
|
||||||
const query = new Query(stringQuery, { ignoreDiacritics: true })
|
const query = new Query(stringQuery, {
|
||||||
|
ignoreDiacritics: true,
|
||||||
|
ignoreArabicDiacritics: true,
|
||||||
|
})
|
||||||
|
|
||||||
// Assert
|
// Assert
|
||||||
expect(query.getExactTerms()).toEqual(['lorem ipsum', 'sit amet'])
|
expect(query.getExactTerms()).toEqual(['lorem ipsum', 'sit amet'])
|
||||||
|
|||||||
@@ -141,6 +141,7 @@
|
|||||||
}
|
}
|
||||||
query = new Query(searchQuery, {
|
query = new Query(searchQuery, {
|
||||||
ignoreDiacritics: plugin.settings.ignoreDiacritics,
|
ignoreDiacritics: plugin.settings.ignoreDiacritics,
|
||||||
|
ignoreArabicDiacritics: plugin.settings.ignoreArabicDiacritics,
|
||||||
})
|
})
|
||||||
cancelableQuery = cancelable(
|
cancelableQuery = cancelable(
|
||||||
new Promise(resolve => {
|
new Promise(resolve => {
|
||||||
|
|||||||
@@ -88,6 +88,7 @@ export function getApi(plugin: OmnisearchPlugin) {
|
|||||||
async search(q: string): Promise<ResultNoteApi[]> {
|
async search(q: string): Promise<ResultNoteApi[]> {
|
||||||
const query = new Query(q, {
|
const query = new Query(q, {
|
||||||
ignoreDiacritics: plugin.settings.ignoreDiacritics,
|
ignoreDiacritics: plugin.settings.ignoreDiacritics,
|
||||||
|
ignoreArabicDiacritics: plugin.settings.ignoreArabicDiacritics,
|
||||||
})
|
})
|
||||||
const raw = await plugin.searchEngine.getSuggestions(query)
|
const raw = await plugin.searchEngine.getSuggestions(query)
|
||||||
return mapResults(plugin, raw)
|
return mapResults(plugin, raw)
|
||||||
|
|||||||
Reference in New Issue
Block a user