devol
/
bitwarden-estensione-browser
mirror of https://github.com/bitwarden/browser
1
0
Fork 0
Code Issues 1 Releases Wiki Activity

[PS-136] stripped off accented characters from search field for browser client (#812) 

* stripped off accented characters from search field for browser client

* moved normalization from component to search service

* fixed conflicts

* removed normalization from cipher component

* added comments to normalize method

* added comments to normalize method
This commit is contained in:
Gbubemi Smith 2022-05-24 23:40:17 +01:00 committed by GitHub
parent 0a072b6452
commit cc751e0287
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
common/src/services/search.service.ts
View File

@ -34,6 +34,7 @@ export class SearchService implements SearchServiceAbstraction {
}
isSearchable(query: string): boolean {
query = this.normalizeSearchQuery(query);
const notSearchable =
query == null ||
(this.index == null && query.length < this.searchableMinLength) ||
@ -97,7 +98,7 @@ export class SearchService implements SearchServiceAbstraction {
): Promise<CipherView[]> {
const results: CipherView[] = [];
if (query != null) {
query = query.trim().toLowerCase();
query = this.normalizeSearchQuery(query.trim().toLowerCase());
}
if (query === "") {
query = null;
@ -165,7 +166,7 @@ export class SearchService implements SearchServiceAbstraction {
}
searchCiphersBasic(ciphers: CipherView[], query: string, deleted = false) {
query = query.trim().toLowerCase();
query = this.normalizeSearchQuery(query.trim().toLowerCase());
return ciphers.filter((c) => {
if (deleted !== c.isDeleted) {
return false;
@ -187,7 +188,7 @@ export class SearchService implements SearchServiceAbstraction {
}
searchSends(sends: SendView[], query: string) {
query = query.trim().toLocaleLowerCase();
query = this.normalizeSearchQuery(query.trim().toLocaleLowerCase());
if (query === null) {
return sends;
}
@ -294,12 +295,14 @@ export class SearchService implements SearchServiceAbstraction {
const checkFields = fields.every((i: any) => searchableFields.includes(i));
if (checkFields) {
return token
.toString()
.normalize("NFD")
.replace(/[\u0300-\u036f]/g, "");
return this.normalizeSearchQuery(token.toString());
}
return token;
}
// Remove accents/diacritics characters from text. This regex is equivalent to the Diacritic unicode property escape, i.e. it will match all diacritic characters.
private normalizeSearchQuery(query: string): string {
return query?.normalize("NFD").replace(/[\u0300-\u036f]/g, "");
}
}