вроде все заработало
This commit is contained in:
@@ -10,6 +10,7 @@ import { getRequestTypes, IRequestType, REQUEST_STATUS } from '../../shared/appr
|
||||
import { effectiveUserEmail } from '../../shared/impersonation';
|
||||
import { IMyRequest } from './models/IMyRequest';
|
||||
import { IMyRequestsTypeDebug } from './models/IMyRequestsDebug';
|
||||
import { traceLinkedTypes, traceResultRows, traceSearchHits } from './searchTrace';
|
||||
|
||||
// Список данных «... — данные» → список согласования «... — согласование»
|
||||
// (конвенция именования; см. forms/*/provisioning.json и forms/*/approval/provisioning.json).
|
||||
@@ -65,7 +66,9 @@ export function currentUserEmail(context: WebPartContext): string {
|
||||
// Типы заявок, у которых approval-route API уже отдал listId/siteUrl (готовы к чтению).
|
||||
export async function getLinkedTypes(context: WebPartContext): Promise<IRequestType[]> {
|
||||
const types = await getRequestTypes(context);
|
||||
return types.filter((t) => !!t.listId && !!t.siteUrl);
|
||||
const linked = types.filter((t) => !!t.listId && !!t.siteUrl);
|
||||
traceLinkedTypes(types, linked);
|
||||
return linked;
|
||||
}
|
||||
|
||||
// Что показывать в разделе «На согласовании»:
|
||||
@@ -406,7 +409,9 @@ async function getInvolvedRowsViaSearch(
|
||||
.expand('RootFolder')()) as { RootFolder?: { ServerRelativeUrl?: string } };
|
||||
const relUrl = rootFolder.RootFolder?.ServerRelativeUrl;
|
||||
if (!relUrl) return [];
|
||||
const approvalListAbsUrl = `${siteOrigin(type.siteUrl as string)}${relUrl}`;
|
||||
// Слеш на конце обязателен: без него префикс …/Lists/List1 совпадает и с
|
||||
// путями элементов …/Lists/List10 (см. listPathPrefix).
|
||||
const approvalListAbsUrl = `${siteOrigin(type.siteUrl as string)}${relUrl.replace(/\/+$/, '')}/`;
|
||||
|
||||
const sp = spForSite(type.siteUrl as string, context);
|
||||
const pathClause = `Path:"${approvalListAbsUrl}*"`;
|
||||
@@ -526,6 +531,24 @@ function listAbsUrl(type: IRequestType): string {
|
||||
return `${siteOrigin(type.siteUrl as string)}${type.listId}`;
|
||||
}
|
||||
|
||||
// Префикс пути для KQL-условия Path и для сопоставления хитов с типом.
|
||||
//
|
||||
// Обязательно СО слешем на конце: адреса списков в этом сайте отличаются только
|
||||
// номером (…/Lists/List1, …/Lists/List10), и «голый» префикс List1 совпадает с
|
||||
// путями элементов List10. В KQL это притягивало чужие элементы, а при разборе
|
||||
// ответа хит уезжал в первый подошедший тип — заявки «Обходного листа»
|
||||
// приписывались списку List1, дальше их Id искались не в том списке и строки
|
||||
// молча исчезали из выдачи.
|
||||
function listPathPrefix(type: IRequestType): string {
|
||||
return `${listAbsUrl(type).replace(/\/+$/, '')}/`;
|
||||
}
|
||||
|
||||
// Хит принадлежит типу, только если путь элемента лежит ВНУТРИ списка —
|
||||
// сравнение по границе сегмента, не по «голому» началу строки.
|
||||
export function pathBelongsToList(path: string, type: IRequestType): boolean {
|
||||
return path.toLowerCase().indexOf(listPathPrefix(type).toLowerCase()) === 0;
|
||||
}
|
||||
|
||||
// KQL не терпит спецсимволов в свободном терме — оставляем буквы (лат./кир.),
|
||||
// цифры и пробелы. (без \p{L} — целевой ES-таргет проекта его не поддерживает)
|
||||
function sanitizeForKql(text: string): string {
|
||||
@@ -556,14 +579,15 @@ function hitsToEntries(
|
||||
results: { PrimarySearchResults: unknown[] },
|
||||
types: IRequestType[]
|
||||
): Array<{ id: number; type: IRequestType }> {
|
||||
const urls = types.map((t) => ({ type: t, url: listAbsUrl(t).toLowerCase() }));
|
||||
const seen = new Set<string>();
|
||||
return (results.PrimarySearchResults as Array<Record<string, unknown>>)
|
||||
.map((r) => ({ id: Number(r.ListItemID), path: String(r.Path || '').toLowerCase() }))
|
||||
.map((r) => ({ id: Number(r.ListItemID), path: String(r.Path || '') }))
|
||||
.filter((h) => !isNaN(h.id) && h.id > 0)
|
||||
.map((h) => {
|
||||
const matched = urls.filter((u) => h.path.indexOf(u.url) === 0)[0];
|
||||
return matched ? { id: h.id, type: matched.type } : undefined;
|
||||
// Сравнение по границе сегмента (см. listPathPrefix): при «голом»
|
||||
// indexOf хиты List10 доставались типу List1 — тому, что раньше в массиве.
|
||||
const matched = types.filter((t) => pathBelongsToList(h.path, t))[0];
|
||||
return matched ? { id: h.id, type: matched } : undefined;
|
||||
})
|
||||
.filter((e): e is { id: number; type: IRequestType } => !!e)
|
||||
.filter((e) => {
|
||||
@@ -733,7 +757,7 @@ export async function searchOwnRequests(
|
||||
const claimsLogin = `i:0#.f|membership|${currentUserEmail(context)}`;
|
||||
|
||||
const clauses = [
|
||||
`(${types.map((t) => `Path:"${listAbsUrl(t)}*"`).join(' OR ')})`,
|
||||
`(${types.map((t) => `Path:"${listPathPrefix(t)}*"`).join(' OR ')})`,
|
||||
`AuthorOWSUSER:"${claimsLogin}"`,
|
||||
];
|
||||
const text = freeTextClause(query);
|
||||
@@ -926,7 +950,7 @@ export async function filterPendingByQuery(
|
||||
|
||||
const sp = spForSite(scopeTypes[0].siteUrl as string, context);
|
||||
const kql = [
|
||||
`(${scopeTypes.map((t) => `Path:"${listAbsUrl(t)}*"`).join(' OR ')})`,
|
||||
`(${scopeTypes.map((t) => `Path:"${listPathPrefix(t)}*"`).join(' OR ')})`,
|
||||
`(${candidates.map((r) => `ListItemID:${r.id}`).join(' OR ')})`,
|
||||
text,
|
||||
].join(' AND ');
|
||||
@@ -973,7 +997,7 @@ export async function searchOwnAll(
|
||||
const sp = spForSite(types[0].siteUrl as string, context);
|
||||
const claimsLogin = `i:0#.f|membership|${currentUserEmail(context)}`;
|
||||
const clauses = [
|
||||
`(${types.map((t) => `Path:"${listAbsUrl(t)}*"`).join(' OR ')})`,
|
||||
`(${types.map((t) => `Path:"${listPathPrefix(t)}*"`).join(' OR ')})`,
|
||||
`AuthorOWSUSER:"${claimsLogin}"`,
|
||||
];
|
||||
const text = freeTextClause(query);
|
||||
@@ -988,9 +1012,14 @@ export async function searchOwnAll(
|
||||
});
|
||||
|
||||
|
||||
traceSearchHits('Свои заявки', kql, types, results, listAbsUrl);
|
||||
|
||||
const entries = hitsToEntries(results, types);
|
||||
const byKey = entries.length > 0 ? await loadByIds(context, entries, 'own') : new Map<string, IMyRequest>();
|
||||
return entries.map((e) => byKey.get(`${e.type.id}-${e.id}`)).filter((r): r is IMyRequest => !!r);
|
||||
const rows = entries.map((e) => byKey.get(`${e.type.id}-${e.id}`)).filter((r): r is IMyRequest => !!r);
|
||||
|
||||
traceResultRows('Свои заявки', types, entries, rows);
|
||||
return rows;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user