yuuza/outline
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Missing context in search results (#6549)

Browse Source 
closes #6548
This commit is contained in:
Tom Moor
2024-02-16 09:34:05 -08:00
committed by GitHub
parent 561606f6f4
commit 69eb7b9db4
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
server/models/helpers/SearchHelper.test.ts
View File

@@ -186,6 +186,7 @@ describe("SearchHelper", () => {
}); });
const { results } = await SearchHelper.searchForUser(user, "test"); const { results } = await SearchHelper.searchForUser(user, "test");
expect(results.length).toBe(1); expect(results.length).toBe(1);
expect(results[0].ranking).toBeTruthy();
expect(results[0].document?.id).toBe(document.id); expect(results[0].document?.id).toBe(document.id);
}); });

8
server/models/helpers/SearchHelper.ts
View File

@@ -49,9 +49,11 @@ type SearchOptions = {
}; };
type RankedDocument = Document & { type RankedDocument = Document & {
id: string;
dataValues: Partial<Document> & {
searchRanking: number; searchRanking: number;
searchContext: string; searchContext: string;
id: string; };
}; };
export default class SearchHelper { export default class SearchHelper {
@@ -394,8 +396,8 @@ export default class SearchHelper {
): SearchResponse { ): SearchResponse {
return { return {
results: map(results, (result) => ({ results: map(results, (result) => ({
ranking: result.searchRanking, ranking: result.dataValues.searchRanking,
context: removeMarkdown(result.searchContext, { context: removeMarkdown(result.dataValues.searchContext, {
stripHTML: false, stripHTML: false,
}), }),
document: find(documents, { document: find(documents, {