diff --git a/src/webparts/requestPortal/features/myRequests/myRequests.service.ts b/src/webparts/requestPortal/features/myRequests/myRequests.service.ts index 5f2e48e..edbb0bf 100644 --- a/src/webparts/requestPortal/features/myRequests/myRequests.service.ts +++ b/src/webparts/requestPortal/features/myRequests/myRequests.service.ts @@ -25,6 +25,7 @@ import { import { effectiveUserEmail } from '../../shared/impersonation'; import { DATA_SITE_URL } from '../../shared/config'; import { IAssignee, IMyRequest } from './models/IMyRequest'; +import { parseAuthorOwsUser } from './parseAuthorOwsUser'; import { IMyRequestsTypeDebug } from './models/IMyRequestsDebug'; import { traceLinkedTypes, traceResultRows, traceSearchHits } from './searchTrace'; import { perfScope, traced } from '../../shared/perfTrace'; @@ -106,22 +107,6 @@ const MP_CODE = 'RequestUniqueCodeOWSTEXT'; // создатель (см. toMyRequestFromSearch); основной — MP_AUTHOR/parseAuthorOwsUser. const OWN_SELECT_PROPERTIES = ['ListItemID', 'Path', 'Title', 'Created', 'Author', MP_AUTHOR, MP_REQUEST_STATUS, MP_CODE]; -/** - * Managed property Person-поля хранит несколько значений одной строкой через - * " | ": «<почта> | <ФИО> | <служебный id> » (первые два — - * проверено вручную, tools/sp props, 2026-08-20; хвост вылез в UI 2026-09-29, - * когда ФИО стали брать отсюда). Поэтому ФИО — ровно ВТОРОЙ кусок, а не «всё - * после первого разделителя». - * - * Разделитель — именно " | " С ПРОБЕЛАМИ: claims-логин - * `i:0#.f|membership|user@domain` содержит "|" без пробелов вокруг, и деление по - * голому "|" порезало бы его на части. - */ -export function parseAuthorOwsUser(value: unknown): { email: string; name: string } { - const parts = (typeof value === 'string' ? value : '').split(' | ').map((p) => p.trim()); - return { email: parts[0] ?? '', name: parts[1] ?? '' }; -} - // effectiveUserEmail, а не pageContext напрямую: при отладочной подмене личности // (см. shared/impersonation.ts) «мои заявки» и очереди согласования должны // отбираться по подменённому сотруднику — иначе подмена показывала бы чужой diff --git a/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.test.ts b/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.test.ts index 743d305..079aac9 100644 --- a/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.test.ts +++ b/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.test.ts @@ -1,4 +1,4 @@ -import { parseAuthorOwsUser } from './myRequests.service'; +import { parseAuthorOwsUser } from './parseAuthorOwsUser'; describe('parseAuthorOwsUser', () => { it('ФИО — ровно второй кусок, служебный хвост отбрасывается', () => { diff --git a/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.ts b/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.ts new file mode 100644 index 0000000..8f3b008 --- /dev/null +++ b/src/webparts/requestPortal/features/myRequests/parseAuthorOwsUser.ts @@ -0,0 +1,18 @@ +/** + * Managed property Person-поля хранит несколько значений одной строкой через + * " | ": «<почта> | <ФИО> | <служебный id> » (первые два — + * проверено вручную, tools/sp props, 2026-08-20; хвост вылез в UI 2026-09-29, + * когда ФИО стали брать отсюда). Поэтому ФИО — ровно ВТОРОЙ кусок, а не «всё + * после первого разделителя». + * + * Разделитель — именно " | " С ПРОБЕЛАМИ: claims-логин + * `i:0#.f|membership|user@domain` содержит "|" без пробелов вокруг, и деление по + * голому "|" порезало бы его на части. + * + * Отдельным модулем, без SharePoint-импортов: myRequests.service тянет PnPjs, + * который Jest проекта не разбирает (ESM), — так функцию можно тестировать. + */ +export function parseAuthorOwsUser(value: unknown): { email: string; name: string } { + const parts = (typeof value === 'string' ? value : '').split(' | ').map((p) => p.trim()); + return { email: parts[0] ?? '', name: parts[1] ?? '' }; +}