11 Commits

Author SHA1 Message Date
gorbenko.pavel 58e67680d0 Merge branch 'main' into сlearance-form
Tests / frontend-test (pull_request) Successful in 1m16s
2026-08-17 10:24:13 +05:00
gorbenko.pavel 6aa5d786d7 теперь все работает правильно 2026-08-17 09:33:44 +05:00
gorbenko.pavel 3da12fc622 вроде работает правильно 2026-08-17 09:13:49 +05:00
gorbenko.pavel 4627675ccf вроде все заработало 2026-08-14 17:56:17 +05:00
gorbenko.pavel c9db7261ed вроде есть заглушка 2026-08-14 17:24:45 +05:00
gorbenko.pavel 53e1d58ffe батчинг 2026-08-14 15:31:27 +05:00
gorbenko.pavel 595ab02b16 имперсонейшн 2026-08-14 12:47:52 +05:00
gorbenko.pavel 42fe721b13 права на редактирование определенных полей 2026-08-14 12:26:46 +05:00
gorbenko.pavel fa551caf5e новая форма 2026-08-14 12:09:28 +05:00
Тимур Городецкий a937e33f2e refactor: StatusBadge поверх ui-kit Badge
Tests / frontend-test (push) Successful in 4m20s
2026-08-14 12:07:09 +05:00
Тимур Городецкий 64527ebd5b feat: навигация на ui-kit SIdebar + мобильный drawer
Tests / frontend-test (push) Successful in 1m9s
2026-08-14 11:35:49 +05:00
28 changed files with 989 additions and 529 deletions
+4 -4
View File
@@ -17,7 +17,7 @@
"@microsoft/sp-webpart-base": "1.22.1",
"@pnp/sp": "^4.20.0",
"@pnp/spfx-controls-react": "^3.25.0",
"@tnsu/ui-kit": "^0.2.0",
"@tnsu/ui-kit": "^0.3.0",
"react": "17.0.1",
"react-dom": "17.0.1",
"tslib": "2.3.1"
@@ -9870,9 +9870,9 @@
"license": "0BSD"
},
"node_modules/@tnsu/ui-kit": {
"version": "0.2.0",
"resolved": "http://git.tnsu.kz/api/packages/timur.gorodetskiy/npm/%40tnsu%2Fui-kit/-/0.2.0/ui-kit-0.2.0.tgz",
"integrity": "sha512-8vQsA91lSbmtlqROlIYkgSH3cNEH5DCVI5vM4mks2RWwvSpYet0zfe2B3wfBcnb+cAWrbLGfCwe81KRDcJ/k+Q==",
"version": "0.3.0",
"resolved": "http://git.tnsu.kz/api/packages/timur.gorodetskiy/npm/%40tnsu%2Fui-kit/-/0.3.0/ui-kit-0.3.0.tgz",
"integrity": "sha512-V9pTS/rBmU9KInczI8jJ/KHT96hfX0OlnjIJ2NZ/iEQTOE93I9xMfmgpj83PSRrI6/xzyybu+8k/qVO6LP926w==",
"license": "ISC",
"peerDependencies": {
"@fluentui/react": "^8.106.4",
+1 -1
View File
@@ -21,7 +21,7 @@
"@microsoft/sp-webpart-base": "1.22.1",
"@pnp/sp": "^4.20.0",
"@pnp/spfx-controls-react": "^3.25.0",
"@tnsu/ui-kit": "^0.2.0",
"@tnsu/ui-kit": "^0.3.0",
"react": "17.0.1",
"react-dom": "17.0.1",
"tslib": "2.3.1"
@@ -13,10 +13,13 @@ import RequestPortal from './components/RequestPortal';
import { IRequestPortalProps } from './components/IRequestPortalProps';
import { initSP } from './shared/pnp/pnpjs';
import { installErrorLogging } from './shared/diagnostics';
import { setImpersonatedEmail } from './shared/impersonation';
import { ErrorOverlay } from './shared/ui';
export interface IRequestPortalWebPartProps {
description: string;
/** Отладочная подмена личности — см. shared/impersonation.ts. На бою пусто. */
debugUserEmail: string;
}
export default class RequestPortalWebPart extends BaseClientSideWebPart<IRequestPortalWebPartProps> {
@@ -25,6 +28,10 @@ export default class RequestPortalWebPart extends BaseClientSideWebPart<IRequest
private _environmentMessage: string = '';
public render(): void {
// Именно здесь, а не в onInit: свойство меняется в панели без перезапуска
// веб-части, и render вызывается на каждое такое изменение.
setImpersonatedEmail(this.properties.debugUserEmail);
const element: React.ReactElement = React.createElement(
ErrorOverlay,
undefined,
@@ -117,6 +124,11 @@ export default class RequestPortalWebPart extends BaseClientSideWebPart<IRequest
groupFields: [
PropertyPaneTextField('description', {
label: strings.DescriptionFieldLabel
}),
PropertyPaneTextField('debugUserEmail', {
label: strings.DebugUserEmailFieldLabel,
description: strings.DebugUserEmailFieldDescription,
placeholder: 'employee@tnsu.kz'
})
]
}
@@ -81,17 +81,38 @@
display: none;
}
// На узких экранах рейл уходит наверх.
@media (max-width: 900px) {
// Мобильная кнопка-бургер (открывает выезжающий Sidebar). На десктопе скрыта.
.mobileNav {
display: none;
margin-bottom: $sp-4;
}
// < 768px: рейл превращается в выезжающий drawer (см. @tnsu/ui-kit Sidebar). Он
// position:fixed и выходит из потока — колонка .side схлопывается сама, main
// занимает всю ширину. Показываем бургер в шапке контента.
@media (max-width: 768px) {
.layout {
flex-direction: column;
min-height: 0;
}
.side {
width: 100%;
.mobileNav {
display: block;
}
}
// Полоса отладочной подмены личности (см. shared/impersonation.ts). Красная и
// во всю ширину намеренно: это не подсказка, а предупреждение о том, что портал
// показывает не ваши данные — незаметная плашка тут хуже, чем никакой.
.impersonationBar {
padding: $sp-3 $sp-4;
margin-bottom: $sp-4;
border: 1px solid $red-500;
border-radius: $r-md;
background: rgba($red-500, .08);
color: $navy-900;
font-size: $fs-sm;
line-height: 1.45;
}
// Стили конкретных фич живут в features/<name>/<Name>.module.scss.
// Здесь — только оболочка (root + раскладка колонок), общая для всех фич.
@@ -2,7 +2,8 @@ import * as React from 'react';
import styles from './RequestPortal.module.scss';
import '@tnsu/ui-kit/styles.css';
import type { IRequestPortalProps } from './IRequestPortalProps';
import { IconSprite } from '@tnsu/ui-kit';
import { IconSprite, Sidebar, SidebarToggle } from '@tnsu/ui-kit';
import type { ISidebarItem } from '@tnsu/ui-kit';
import { ConfirmHost } from '../shared/ui';
import {
navigate,
@@ -19,8 +20,8 @@ import {
OwnTab,
} from '../shared/routing';
import { onRequestsChanged, emitRequestsChanged } from '../shared/requestEvents';
import { getImpersonatedEmail } from '../shared/impersonation';
import { confirmDiscardFormChanges, markFormClean, markFormDirty } from '../shared/unsavedChanges';
import { SideNav } from './SideNav/SideNav';
import { Drawer } from './Drawer/Drawer';
import { CategoryTabs } from './CategoryTabs/CategoryTabs';
import { CATEGORIES, DEFAULT_CATEGORY_ID } from '../data/categories';
@@ -57,8 +58,14 @@ const RequestPortal: React.FC<IRequestPortalProps> = ({ hasTeamsContext, context
.filter(Boolean)
.join(' ');
// Читаем при рендере, а не через состояние: значение выставляет веб-часть
// (setImpersonatedEmail в render) до создания этого элемента, и меняется оно
// только вместе с новым render всей веб-части.
const impersonatedEmail = getImpersonatedEmail();
const route = useRoute();
const [navCollapsed, setNavCollapsed] = React.useState<boolean>(false);
const [mobileNavOpen, setMobileNavOpen] = React.useState<boolean>(false);
const [navCounts, setNavCounts] = React.useState<INavCounts | undefined>(undefined);
const [activeCategoryId, setActiveCategoryId] = React.useState<string>(() => {
@@ -250,26 +257,47 @@ const RequestPortal: React.FC<IRequestPortalProps> = ({ hasTeamsContext, context
}).catch(() => undefined);
};
// Разделы бокового меню для пакетного Sidebar (счётчики — из navCounts).
const navItems: ISidebarItem[] = [
{ id: 'new', title: 'Новая заявка', icon: 'plus' },
{ id: 'approvals', title: 'На согласовании', icon: 'user-check', count: navCounts?.approvals },
{ id: 'execution', title: 'На исполнении', icon: 'helmet', count: navCounts?.execution },
{ id: 'rework', title: 'Доработка', icon: 'edit', count: navCounts?.rework },
{ id: 'own', title: 'Мои заявки', icon: 'clipboard' },
{ id: 'search', title: 'Поиск', icon: 'search' },
];
return (
<section className={rootClasses}>
<IconSprite />
<ConfirmHost />
{/* Подмену личности легко забыть включённой — тогда портал молча показывал
бы чужие заявки и чужую очередь. Полосу поэтому не прячем и не даём
закрыть: она исчезает только когда свойство веб-части очищено. */}
{impersonatedEmail && (
<div className={styles.impersonationBar}>
Включена отладочная подмена: портал работает от имени <strong>{impersonatedEmail}</strong>.
Очистите поле в свойствах веб-части, чтобы вернуться к своей учётной записи.
</div>
)}
<div className={styles.layout}>
<div className={styles.side}>
<SideNav
<Sidebar
items={navItems}
activeId={route.nav}
onChange={goto}
counts={{
approvals: navCounts?.approvals,
execution: navCounts?.execution,
rework: navCounts?.rework,
}}
onChange={(id) => goto(id as NavSection)}
brand="Портал заявок"
collapsed={navCollapsed}
onToggleCollapsed={() => setNavCollapsed((v) => !v)}
mobileOpen={mobileNavOpen}
onMobileClose={() => setMobileNavOpen(false)}
/>
</div>
<main className={styles.main}>
<div className={styles.mobileNav}>
<SidebarToggle onClick={() => setMobileNavOpen(true)} />
</div>
{/* Раздел «Новая заявка» держим СМОНТИРОВАННЫМ и просто прячем стилем,
когда открыта другая вкладка. Иначе React размонтировал бы форму, и
всё введённое пропадало бы при переключении на «Мои заявки» и
@@ -1,154 +0,0 @@
@import '../../../../styles/design-tokens';
// Тёмный боковой рейл портала на всю высоту (фирменный navy-градиент, золотой
// маркер активного пункта). Не «плавает»: примыкает к контенту, тянется по высоте.
.nav {
display: flex;
flex-direction: column;
width: 248px;
padding: $sp-4 0 $sp-3;
background: $grad-header;
box-shadow: inset -1px 0 0 rgba(255, 255, 255, .06);
transition: width .2s;
}
.navCollapsed {
width: 64px;
}
// ---- Шапка рейла: бренд + сворачивание ----
.head {
display: flex;
align-items: center;
justify-content: space-between;
gap: $sp-2;
padding: 0 $sp-4 $sp-4;
border-bottom: 1px solid rgba(255, 255, 255, .08);
}
.navCollapsed .head {
flex-direction: column;
gap: $sp-3;
padding: 0 0 $sp-3;
}
.brand {
display: flex;
align-items: center;
gap: $sp-2;
min-width: 0;
}
.brandText {
font-family: $font-serif;
font-size: 15px;
font-weight: 600;
color: $white;
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
}
.toggle {
flex-shrink: 0;
display: inline-flex;
align-items: center;
justify-content: center;
width: 26px;
height: 26px;
padding: 0;
border: 1px solid rgba(255, 255, 255, .18);
border-radius: $r-sm;
background: transparent;
color: rgba(255, 255, 255, .6);
cursor: pointer;
transition: .16s;
&:hover {
border-color: $gold-400;
color: $gold-300;
}
}
// ---- Пункты разделов ----
.items {
display: flex;
flex-direction: column;
gap: 2px;
margin-top: $sp-4;
}
.navCollapsed .items {
margin-top: $sp-3;
}
.item {
position: relative;
display: flex;
align-items: center;
gap: $sp-3;
width: 100%;
padding: 12px 18px;
border: 0;
border-left: 3px solid transparent;
background: transparent;
color: rgba(255, 255, 255, .62);
font-family: $font-sans;
font-size: 13.5px;
font-weight: 500;
text-align: left;
white-space: nowrap;
cursor: pointer;
transition: color .18s, background .18s, border-color .18s;
&:hover {
color: rgba(255, 255, 255, .92);
background: rgba(255, 255, 255, .04);
}
}
.navCollapsed .item {
justify-content: center;
padding: 12px 0;
}
.itemActive {
color: $white;
background: rgba(255, 255, 255, .07);
border-left-color: $gold-500;
.itemIcon {
color: $gold-400;
}
}
.itemIcon {
flex-shrink: 0;
display: inline-flex;
color: inherit;
}
.itemTitle {
flex: 1;
min-width: 0;
overflow: hidden;
text-overflow: ellipsis;
}
.badge {
flex-shrink: 0;
min-width: 18px;
padding: 1px 6px;
border-radius: $r-full;
background: $gold-500;
color: $navy-950;
font-size: $fs-overline;
font-weight: 700;
text-align: center;
}
.navCollapsed .badge {
position: absolute;
top: 6px;
right: 8px;
}
@@ -1,87 +0,0 @@
import * as React from 'react';
import styles from './SideNav.module.scss';
import { Icon, IconName } from '@tnsu/ui-kit';
import { NavSection } from '../../shared/routing';
interface INavItem {
id: NavSection;
title: string;
icon: IconName;
}
// Порядок неслучаен: сверху действие (создать), затем три очереди — то, что
// ждёт пользователя прямо сейчас, — и только потом архив и поиск.
const ITEMS: INavItem[] = [
{ id: 'new', title: 'Новая заявка', icon: 'plus' },
{ id: 'approvals', title: 'На согласовании', icon: 'user-check' },
{ id: 'execution', title: 'На исполнении', icon: 'helmet' },
{ id: 'rework', title: 'Доработка', icon: 'edit' },
{ id: 'own', title: 'Мои заявки', icon: 'clipboard' },
{ id: 'search', title: 'Поиск', icon: 'search' },
];
interface ISideNavProps {
activeId: NavSection;
onChange: (id: NavSection) => void;
/** Счётчики очередей: сколько заявок ждёт пользователя в каждом разделе. */
counts?: Partial<Record<NavSection, number>>;
collapsed: boolean;
onToggleCollapsed: () => void;
}
// Тёмный боковой рейл портала на всю высоту: бренд сверху, разделы снизу.
// Сворачивается до узкой полосы с одними иконками.
export const SideNav: React.FC<ISideNavProps> = ({
activeId,
onChange,
counts,
collapsed,
onToggleCollapsed,
}) => {
const navClasses = [styles.nav, collapsed ? styles.navCollapsed : ''].filter(Boolean).join(' ');
return (
<nav className={navClasses} aria-label="Разделы портала">
<div className={styles.head}>
<div className={styles.brand}>
{!collapsed && <span className={styles.brandText}>Портал заявок</span>}
</div>
<button
type="button"
className={styles.toggle}
onClick={onToggleCollapsed}
aria-label={collapsed ? 'Развернуть меню' : 'Свернуть меню'}
title={collapsed ? 'Развернуть меню' : 'Свернуть меню'}
>
<Icon name={collapsed ? 'chevron-right' : 'chevron-left'} size="sm" />
</button>
</div>
<div className={styles.items}>
{ITEMS.map((item) => {
const isActive = item.id === activeId;
const itemClasses = [styles.item, isActive ? styles.itemActive : ''].filter(Boolean).join(' ');
const count = counts?.[item.id];
const showCount = !!count;
return (
<button
key={item.id}
type="button"
className={itemClasses}
aria-current={isActive ? 'page' : undefined}
onClick={() => onChange(item.id)}
title={collapsed ? item.title : undefined}
>
<span className={styles.itemIcon}>
<Icon name={item.icon} size="sm" />
</span>
{!collapsed && <span className={styles.itemTitle}>{item.title}</span>}
{showCount && <span className={styles.badge}>{count}</span>}
</button>
);
})}
</div>
</nav>
);
};
@@ -1,3 +1,4 @@
import { SPFI } from '@pnp/sp';
import { spForSite } from '../../shared/pnp/pnpjs';
import '@pnp/sp/webs';
import '@pnp/sp/lists';
@@ -7,8 +8,10 @@ import '@pnp/sp/site-users/web';
import '@pnp/sp/search';
import { WebPartContext } from '@microsoft/sp-webpart-base';
import { getRequestTypes, IRequestType, REQUEST_STATUS } from '../../shared/approval';
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).
@@ -22,11 +25,24 @@ const DATA_LIST_SUFFIX = /\s*—\s*данные\s*$/;
// свежесть); при поиске по тексту эта подстраховка не применяется (см. RequestsTableView).
const RECENT_MS = 10 * 60 * 1000;
// Сколько строк согласования максимум забираем из Search за один запрос
// (см. getInvolvedRowsViaSearch). Постраничного перебора здесь нет: набор
// целиком уходит в слияние и сортировку по дате заявки, поэтому при превышении
// лимита пишем предупреждение в лог, а не молча обрезаем.
const SEARCH_ROW_LIMIT = 500;
// Сколько строк максимум запрашиваем у Search за ОДИН HTTP-вызов — верхняя
// граница RowLimit самого SharePoint Search REST API. Полный набор всегда
// собирается несколькими такими вызовами через StartRow (см.
// fetchAllSearchRows) — это не потолок итогового результата.
const SEARCH_PAGE_SIZE = 500;
// Абсолютный потолок на суммарное число строк, которые fetchAllSearchRows
// заберёт за все страницы, — защита от зависания на аномально большом
// индексе, а не ожидаемый рабочий предел. При достижении пишем предупреждение
// и возвращаем то, что успели собрать, вместо бесконечного перебора.
const SEARCH_HARD_CAP = 5000;
// Сколько Id максимум перечисляем в одном REST $filter (`Id eq 1 or Id eq 2…`)
// при дочитывании полей по найденным через Search элементам (см. loadByIds).
// SharePoint REST не резиновый по длине URL — несколько сотен ИЛИ уже рискуют
// упереться в лимит, поэтому набор Id одного типа режется на такие пачки,
// а не уходит одним запросом целиком.
const REST_ID_FILTER_CHUNK = 100;
// Сколько последних заявок автора дочитываем через REST в довесок к Search
// в разделе «Мои заявки» (см. getFreshOwnRowsForType). Это записи ОДНОГО
@@ -41,26 +57,35 @@ const MP_APPROVER = 'TansuSiteApproverOWSUSER';
// Статус СТРОКИ МАРШРУТА (колонка ApprovalStatus списка «— согласование») —
// Choice-поле, отсюда суффикс OWSCHCS (подтверждено отладочным перебором
// 2026-08-06; у Person-полей он OWSUSER, см. MP_APPROVER). Колонку не путать с
// RequestStatus списка ДАННЫХ — это статус ЗАЯВКИ целиком.
//
// Для статуса ЗАЯВКИ (список ДАННЫХ) отдельной managed property здесь
// НАМЕРЕННО нет: имя для произвольной Choice-колонки в схеме поиска либо надо
// знать точно, либо угадывать по конвенции — угадали неверно (или колонка
// после переименования Status → RequestStatus ещё не переиндексирована), и
// фильтр по статусу молча возвращал 0 заявок. Поэтому статус заявки
// фильтруется НЕ в KQL, а в памяти после дочитывания реальных полей через REST
// (см. getOwnRequestsByStatus) — Search там ищет только по надёжному
// AuthorOWSUSER, статус ни при чём.
// RequestStatus списка ДАННЫХ — это статус ЗАЯВКИ целиком (см. MP_REQUEST_STATUS
// ниже).
const MP_STATUS = 'ApprovalStatusOWSCHCS';
// Статус ЗАЯВКИ (колонка RequestStatus списка ДАННЫХ) — Choice-поле, тот же
// суффикс OWSCHCS, без префикса группы колонок сайта (поле локальное для
// списка, как и ApprovalStatus выше). Раньше здесь стояло «угадали неверно,
// фильтр молча возвращает 0» — перепроверено вручную через
// /_api/search/query 2026-08-17: и голый запрос по этому свойству, и в
// связке с Path, отдают счётчики, совпадающие с REST-эталоном (см.
// getOwnRequestsByStatus). Старый вывод о поломке был артефактом кривого
// тестового Path (задвоенный сегмент /sites/…), а не реальным ограничением
// SharePoint — сама схема поиска исправна.
const MP_REQUEST_STATUS = 'RequestStatusOWSCHCS';
// effectiveUserEmail, а не pageContext напрямую: при отладочной подмене личности
// (см. shared/impersonation.ts) «мои заявки» и очереди согласования должны
// отбираться по подменённому сотруднику — иначе подмена показывала бы чужой
// маршрут в форме, но собственные списки заявок.
export function currentUserEmail(context: WebPartContext): string {
return (context.pageContext.user.email || '').toLowerCase();
return effectiveUserEmail(context).toLowerCase();
}
// Типы заявок, у которых 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;
}
// Что показывать в разделе «На согласовании»:
@@ -269,12 +294,12 @@ async function indexApprovalsForType(
// SharePoint REST — резолвим email в числовой Id пользователя (как и везде в
// проекте, см. ensureUser) и фильтруем по TansuSiteApproverId.
//
// ApprovalStatus в $filter НЕ используем: подтверждено на практике — фильтр
// вида `ApprovalStatus eq 'На согласовании'` (кириллица в $filter) молча
// возвращает 0 строк, даже когда без него те же строки находятся с этим же
// значением статуса. Поэтому фильтруем только по TansuSiteApproverId
// (числовое значение, проблем нет), а статус сравниваем в JS после
// получения строк.
// ApprovalStatus в $filter теперь ИСПОЛЬЗУЕМ: раньше здесь считалось, что
// кириллица в значении делает такой фильтр ненадёжным, — перепроверено
// вручную через REST 2026-08-17 (все три статуса списка согласования дали
// точное совпадение с эталоном), теория не подтвердилась. Старый вывод о
// поломке был артефактом кривого тестового запроса, а не реальным
// ограничением SharePoint.
const claimsLogin = tokenEmail.indexOf('|') >= 0 ? tokenEmail : `i:0#.f|membership|${tokenEmail}`;
const approver = await sp.web.ensureUser(claimsLogin);
@@ -289,20 +314,20 @@ async function indexApprovalsForType(
// getInvolvedRowsViaSearch, поэтому решение «моя очередь сейчас» всегда
// точное, даже когда список кандидатов пришёл из Search.
const cutoffIso = new Date(Date.now() - RECENT_MS).toISOString();
// Статус — прямо в $filter (см. комментарий про ApprovalStatus выше):
// REST-довесок сразу отдаёт только нужные строки, без постфильтрации в JS.
const statusFilter =
statuses.length > 0 ? ` and (${statuses.map((s) => `ApprovalStatus eq '${s}'`).join(' or ')})` : '';
const recentRows = (await approvalList.items
.filter(`TansuSiteApproverId eq ${approver.Id} and Modified ge '${cutoffIso}'`)
.filter(`TansuSiteApproverId eq ${approver.Id} and Modified ge '${cutoffIso}'${statusFilter}`)
.select('Id', 'TicketId', 'ApprovalStatus')
.top(2000)()) as Array<Record<string, unknown>>;
// Статусы уходят в KQL: сервер сразу отдаёт только нужные строки,
// Статусы уходят и в KQL: сервер сразу отдаёт только нужные строки,
// а не всё, где пользователь есть в маршруте.
const searchRows = await getInvolvedRowsViaSearch(context, type, approvalList, statuses);
const seenIds = new Set(recentRows.map((r) => r.Id));
const allRows = recentRows.concat(searchRows.filter((r) => !seenIds.has(r.Id)));
// REST-довесок берётся по Modified без учёта статуса (кириллица в $filter
// ненадёжна, см. выше) — отбор по статусу делаем здесь, на клиенте.
stepRows = allRows.filter((r) => statuses.indexOf(r.ApprovalStatus as string) !== -1);
stepRows = recentRows.concat(searchRows.filter((r) => !seenIds.has(r.Id)));
} else {
// 'involved' — используется ТОЛЬКО для текстового поиска (см.
// getPendingApprovalIndex/searchMyRequests) — здесь важна не мгновенная
@@ -401,7 +426,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}*"`;
@@ -411,29 +438,24 @@ async function getInvolvedRowsViaSearch(
}
const kql = clauses.join(' AND ');
const results = await sp.search({
Querytext: kql,
RowLimit: SEARCH_ROW_LIMIT,
SelectProperties: ['ListItemID'],
TrimDuplicates: false,
});
const total = results.TotalRows || 0;
const ids = (results.PrimarySearchResults as Array<Record<string, unknown>>)
const hits = await fetchAllSearchRows(sp, kql, ['ListItemID'], `«${type.name}»`);
const ids = hits
.map((r) => Number(r.ListItemID))
.filter((id) => !isNaN(id) && id > 0);
if (total > SEARCH_ROW_LIMIT) {
console.warn(
`[Текущие заявки] «${type.name}»: Search вернул ${total} строк, взяли первые ${SEARCH_ROW_LIMIT} — остальные не попадут в выборку`
);
}
if (ids.length === 0) return [];
return (await approvalList.items
.filter(ids.map((id) => `Id eq ${id}`).join(' or '))
// Id режем на пачки — тот же предел длины URL REST $filter, что и в
// loadByIds (см. REST_ID_FILTER_CHUNK).
const chunks = await Promise.all(
chunk(ids, REST_ID_FILTER_CHUNK).map((idsChunk) =>
approvalList.items
.filter(idsChunk.map((id) => `Id eq ${id}`).join(' or '))
.select('Id', 'TicketId', 'ApprovalStatus')
.top(ids.length)()) as Array<Record<string, unknown>>;
.top(idsChunk.length)()
)
);
return (chunks as Array<Array<Record<string, unknown>>>).reduce((all, r) => all.concat(r), []);
} catch (e) {
console.error(
`[Текущие заявки] «${type.name}»: Search по списку согласования не сработал:`,
@@ -508,10 +530,64 @@ export interface IOwnRequestsPage {
}
const PAGE_SIZE = 20;
// Сколько заявок максимум перечисляем в KQL при поиске среди уже отобранных
// (см. filterPendingByQuery) — иначе строка запроса станет неподъёмной.
// Сколько заявок-кандидатов максимум перечисляем в ОДНОМ KQL-запросе при
// поиске среди уже отобранных (см. filterPendingByQuery) — не потолок
// итогового результата, а размер чанка: кандидатов режем на такие пачки и
// прогоняем каждую отдельным запросом, чтобы строка запроса не разрослась
// до неподъёмной, но при этом ни один кандидат не терялся.
const KQL_ID_LIMIT = 300;
function chunk<T>(items: T[], size: number): T[][] {
const result: T[][] = [];
for (let i = 0; i < items.length; i += size) {
result.push(items.slice(i, i + size));
}
return result;
}
// Собрать ВЕСЬ результат Search по запросу, а не одну страницу: сам
// SharePoint Search REST API ограничивает RowLimit одного вызова (см.
// SEARCH_PAGE_SIZE), поэтому при большом совпадающем наборе перебираем
// StartRow, пока индекс не перестанет отдавать новые строки или пока не
// упрёмся в SEARCH_HARD_CAP (аварийный потолок, а не ожидаемый лимит — при
// его достижении пишем предупреждение и отдаём то, что успели собрать).
async function fetchAllSearchRows(
sp: SPFI,
kql: string,
selectProperties: string[],
label: string,
sortList?: Array<{ Property: string; Direction: number }>
): Promise<Array<Record<string, unknown>>> {
const rows: Array<Record<string, unknown>> = [];
let startRow = 0;
for (;;) {
const results = await sp.search({
Querytext: kql,
RowLimit: SEARCH_PAGE_SIZE,
StartRow: startRow,
SelectProperties: selectProperties,
TrimDuplicates: false,
...(sortList ? { SortList: sortList } : {}),
});
const page = results.PrimarySearchResults as Array<Record<string, unknown>>;
rows.push(...page);
const total = results.TotalRows || 0;
startRow += page.length;
if (page.length === 0 || startRow >= total) break;
if (rows.length >= SEARCH_HARD_CAP) {
console.warn(
`[Текущие заявки] ${label}: индекс отдал ${total} строк — забрали только первые ${rows.length} (аварийный потолок)`
);
break;
}
}
return rows;
}
function siteOrigin(siteUrl: string): string {
const match = /^https?:\/\/[^/]+/.exec(siteUrl);
return match ? match[0] : siteUrl;
@@ -521,6 +597,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 {
@@ -542,23 +636,26 @@ function freeTextClause(query: string): string | undefined {
//
// Дедуп по (тип, Id) — ЗДЕСЬ, а не у вызывающей стороны: во всех запросах
// стоит TrimDuplicates: false (нужно, чтобы найти черновики и «свои» копии —
// стандартная дедупликация Search иногда схлопывает и их), а без неё индекс
// может вернуть один и тот же элемент несколькими хитами (переиндексация,
// несколько «версий» в crawl). Раньше хиты превращались в строки один-в-один,
// и дубликат из индекса становился дубликатом строки в таблице — с одинаковым
// React-ключом, то есть видимым «залипанием» одной и той же заявки дважды.
// стандартная дедупликация Search схлопывает по содержимому, а не по Id, и
// вместе с ними может случайно схлопнуть две по-настоящему разные, но похожие
// заявки), а без неё индекс может вернуть один и тот же элемент несколькими
// хитами (переиндексация, несколько «версий» в crawl). Раньше хиты
// превращались в строки один-в-один, и дубликат из индекса становился
// дубликатом строки в таблице — с одинаковым React-ключом, то есть видимым
// «залипанием» одной и той же заявки дважды.
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) => {
@@ -571,6 +668,11 @@ function hitsToEntries(
// Дочитать поля заявок по найденным Id (Search отдаёт не все колонки, и для
// кастомных полей нет гарантии, что они проиндексированы как managed property).
//
// Id одного типа режем на пачки REST_ID_FILTER_CHUNK: теперь, когда Search
// собирает результат целиком (см. fetchAllSearchRows), их может набраться
// заметно больше сотни — один $filter из тысяч «Id eq N or …» упёрся бы в
// предел длины URL SharePoint REST.
async function loadByIds(
context: WebPartContext,
entries: Array<{ id: number; type: IRequestType }>,
@@ -587,26 +689,30 @@ async function loadByIds(
await Promise.all(
Array.from(idsByType.entries()).map(async ([type, ids]) => {
const sp = spForSite(type.siteUrl as string, context);
const rows = (await sp.web
.getList(type.listId as string)
.items.filter(ids.map((id) => `Id eq ${id}`).join(' or '))
const list = sp.web.getList(type.listId as string);
await Promise.all(
chunk(ids, REST_ID_FILTER_CHUNK).map(async (idsChunk) => {
const rows = (await list.items
.filter(idsChunk.map((id) => `Id eq ${id}`).join(' or '))
.select('Id', 'Title', 'RequestStatus', 'Created', 'Author/EMail', 'Author/Title')
.expand('Author')
.top(ids.length)()) as Array<Record<string, unknown>>;
.top(idsChunk.length)()) as Array<Record<string, unknown>>;
rows.forEach((r) => {
const author = r.Author as { EMail?: string; Title?: string } | undefined;
byKey.set(`${type.id}-${r.Id as number}`, toMyRequest(r, author, type, kind));
});
})
);
})
);
return byKey;
}
// Последние OWN_REST_TOP заявок автора одного типа обычным REST-ом, в обход
// Search — сырьё для довеска по свежести (см. getFreshOwnRowsForType).
// Фильтруем только по AuthorId: статус в $filter не используем, потому что
// кириллица в сравнении там уже подводила (см. indexApprovalsForType) — отбор
// по статусу делает вызывающая сторона.
// Фильтруем только по AuthorId, без статуса: эта функция возвращает ВСЕ
// заявки автора, отбор по статусу (если он нужен) делает вызывающая сторона.
async function getOwnRowsForType(context: WebPartContext, type: IRequestType): Promise<IMyRequest[]> {
try {
const sp = spForSite(type.siteUrl as string, context);
@@ -728,7 +834,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);
@@ -758,17 +864,10 @@ export async function searchOwnRequests(
// Свои заявки с конкретным статусом ЗАЯВКИ (не строки маршрута) — «Доработка»
// и фильтр по статусу на вкладке «Инициированные».
//
// Статус НЕ уходит в KQL: единственная надёжная managed property здесь —
// AuthorOWSUSER (стандартное поле автора), а для произвольной Choice-колонки
// имя managed property надо либо знать из схемы поиска, либо угадывать по
// конвенции — угадали неверно (или она ещё не переиндексирована после
// переименования колонки Status → RequestStatus), из-за чего фильтр по
// статусу молча возвращал 0. Поэтому здесь Search ищет ТОЛЬКО по автору
// (+ опционально текст) — это находит ВСЕ заявки автора вплоть до
// KQL_ID_LIMIT, — а отбор по статусу и постраничность делаются в памяти по
// уже дочитанным через REST реальным полям (см. searchOwnAll/loadByIds).
// Тот же принцип, что и в filterPendingByQuery: Search сужает набор по
// надёжному полю, остальное — на клиенте.
// Статус уходит прямо в KQL через MP_REQUEST_STATUS (проверено вручную
// 2026-08-17, см. комментарий у константы) — это настоящий серверный запрос
// с RowLimit/StartRow, как и searchOwnRequests, а не «стянуть всё и
// отфильтровать в памяти».
export async function getOwnRequestsByStatus(
context: WebPartContext,
types: IRequestType[],
@@ -776,28 +875,52 @@ export async function getOwnRequestsByStatus(
page: number,
query: string = ''
): Promise<IOwnRequestsPage> {
const all = await searchOwnAll(context, types, query);
const searchFiltered = all.filter((r) => r.status === status);
if (types.length === 0) return { rows: [], totalRows: 0 };
const sp = spForSite(types[0].siteUrl as string, context);
const claimsLogin = `i:0#.f|membership|${currentUserEmail(context)}`;
const clauses = [
`(${types.map((t) => `Path:"${listPathPrefix(t)}*"`).join(' OR ')})`,
`AuthorOWSUSER:"${claimsLogin}"`,
`${MP_REQUEST_STATUS}:"${status}"`,
];
const text = freeTextClause(query);
if (text) clauses.push(text);
const kql = clauses.join(' AND ');
const results = await sp.search({
Querytext: kql,
RowLimit: PAGE_SIZE,
StartRow: (page - 1) * PAGE_SIZE,
SelectProperties: ['ListItemID', 'Title', 'Path'],
TrimDuplicates: false,
// Свежие сверху — как в searchOwnRequests.
SortList: [{ Property: 'Created', Direction: 1 }],
});
const totalRows = results.TotalRows || 0;
const entries = hitsToEntries(results, types);
const byKey = entries.length > 0 ? await loadByIds(context, entries, 'own') : new Map<string, IMyRequest>();
const searchRows = entries.map((e) => byKey.get(`${e.type.id}-${e.id}`)).filter((r): r is IMyRequest => !!r);
if (page !== 1 || query.trim()) return { rows: searchRows, totalRows };
// Довесок по недавним изменениям — только на первой странице и только без
// текстового поиска (см. getOwnRequests: тот же принцип — довесок относится
// к «показать всё, свежее по дате», а не к результату полнотекстового поиска).
let merged = searchFiltered;
if (page === 1 && !query.trim()) {
// текстового поиска (тот же принцип, что в getOwnRequests): заявку могли
// вернуть на доработку только что, индекс мог не успеть её подхватить.
const freshRows = (await Promise.all(types.map((t) => getRecentlyChangedOwnRowsForType(context, t))))
.reduce((acc, r) => acc.concat(r), [] as IMyRequest[])
.filter((r) => r.status === status);
const seen = new Set(searchFiltered.map((r) => `${r.requestTypeId}-${r.id}`));
const onlyFresh = freshRows.filter((r) => !seen.has(`${r.requestTypeId}-${r.id}`));
merged = onlyFresh.concat(searchFiltered);
}
const searchKeys = new Set(searchRows.map((r) => `${r.requestTypeId}-${r.id}`));
const onlyFresh = freshRows.filter((r) => !searchKeys.has(`${r.requestTypeId}-${r.id}`));
const sorted = merged.sort((a, b) => b.created.localeCompare(a.created));
return {
rows: sorted.slice((page - 1) * PAGE_SIZE, page * PAGE_SIZE),
totalRows: sorted.length,
};
const merged = onlyFresh
.concat(searchRows)
.sort((a, b) => b.created.localeCompare(a.created))
.slice(0, PAGE_SIZE);
return { rows: merged, totalRows: totalRows + onlyFresh.length };
}
// Список согласования по типу заявки: Title списка данных → Title списка
@@ -860,17 +983,17 @@ export async function hydrateDecisionComments(
const approvalList = await approvalListFor(context, type);
if (!approvalList) return;
// Фильтруем только по TicketId (числовое поле), статус сравниваем в JS:
// кириллица в $filter по Choice-колонке молча возвращает 0 строк —
// ровно та же причина, что и в indexApprovalsForType.
// TicketId (OR-цепочка) в скобках обязателен: у OData `and` связывает
// сильнее `or», без скобок фильтр разобрался бы как
// «TicketId eq 1 or (TicketId eq 2 and ApprovalStatus eq …)».
const ticketFilter = typeRows.map((r) => `TicketId eq ${r.id}`).join(' or ');
const stepRows = (await approvalList.items
.filter(typeRows.map((r) => `TicketId eq ${r.id}`).join(' or '))
.select('TicketId', 'ApprovalStatus', 'Comment')
.filter(`(${ticketFilter}) and ApprovalStatus eq '${decisionStatus}'`)
.select('TicketId', 'Comment')
.top(DECISION_COMMENT_ROW_LIMIT)()) as Array<Record<string, unknown>>;
const commentByTicket = new Map<number, string>();
stepRows.forEach((r) => {
if (r.ApprovalStatus !== decisionStatus) return;
const ticketId = r.TicketId as number | undefined;
const comment = ((r.Comment as string) || '').trim();
// Такая строка по заявке одна (и возврат, и отклонение останавливают
@@ -900,6 +1023,11 @@ export async function hydrateDecisionComments(
// обычным REST). Здесь среди НИХ фильтруем по тексту — только через Search
// (KQL ограничивается ListItemID этих заявок, текст идёт свободным термом по
// всем полям), без клиентского REST-матчинга по свежести.
//
// Кандидатов режем на чанки по KQL_ID_LIMIT и гоняем каждый отдельным
// запросом (см. chunk/fetchAllSearchRows) — раньше «лишние» сверх лимита
// кандидаты молча выпадали из поиска, теперь ни один не теряется, просто
// уходит несколькими запросами вместо одного.
export async function filterPendingByQuery(
context: WebPartContext,
types: IRequestType[],
@@ -909,34 +1037,35 @@ export async function filterPendingByQuery(
const text = freeTextClause(query);
if (!text || rows.length === 0) return rows;
if (rows.length > KQL_ID_LIMIT) {
console.warn(
`[Текущие заявки] Поиск среди ждущих решения ограничен ${KQL_ID_LIMIT} заявками из ${rows.length}`
);
}
const candidates = rows.slice(0, KQL_ID_LIMIT);
const scopeTypes = types.filter((t) => candidates.some((r) => r.requestTypeId === t.id));
const scopeTypes = types.filter((t) => rows.some((r) => r.requestTypeId === t.id));
if (scopeTypes.length === 0) return [];
const sp = spForSite(scopeTypes[0].siteUrl as string, context);
const pathClause = `(${scopeTypes.map((t) => `Path:"${listPathPrefix(t)}*"`).join(' OR ')})`;
const found = new Set<string>();
await Promise.all(
chunk(rows, KQL_ID_LIMIT).map(async (candidates) => {
const kql = [
`(${scopeTypes.map((t) => `Path:"${listAbsUrl(t)}*"`).join(' OR ')})`,
pathClause,
`(${candidates.map((r) => `ListItemID:${r.id}`).join(' OR ')})`,
text,
].join(' AND ');
const results = await sp.search({
Querytext: kql,
RowLimit: KQL_ID_LIMIT,
SelectProperties: ['ListItemID', 'Title', 'Path'],
TrimDuplicates: false,
});
const hits = await fetchAllSearchRows(
sp,
kql,
['ListItemID', 'Title', 'Path'],
'Поиск среди ждущих решения'
);
// ListItemID уникален только внутри своего списка — сверяем пару тип+Id.
const found = new Set(hitsToEntries(results, scopeTypes).map((e) => `${e.type.id}-${e.id}`));
return candidates.filter((r) => found.has(`${r.requestTypeId}-${r.id}`));
hitsToEntries({ PrimarySearchResults: hits }, scopeTypes).forEach((e) =>
found.add(`${e.type.id}-${e.id}`)
);
})
);
return rows.filter((r) => found.has(`${r.requestTypeId}-${r.id}`));
}
// ---------------------------------------------------------------------------
@@ -956,8 +1085,10 @@ export interface ISearchPage {
totalRows: number;
}
// Как searchOwnRequests, но без $skip/$top — сразу весь найденный срез (до
// KQL_ID_LIMIT). Только Search, без REST-подстраховки на свежесть.
// Как searchOwnRequests, но без $skip/$top — сразу весь найденный срез
// (см. fetchAllSearchRows — весь, а не первые KQL_ID_LIMIT: тот лимит здесь
// больше не действует, это чанк-размер для другой функции, filterPendingByQuery).
// Только Search, без REST-подстраховки на свежесть.
export async function searchOwnAll(
context: WebPartContext,
types: IRequestType[],
@@ -968,24 +1099,25 @@ 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);
if (text) clauses.push(text);
const kql = clauses.join(' AND ');
const results = await sp.search({
Querytext: kql,
RowLimit: KQL_ID_LIMIT,
SelectProperties: ['ListItemID', 'Title', 'Path'],
TrimDuplicates: false,
});
const hits = await fetchAllSearchRows(sp, kql, ['ListItemID', 'Title', 'Path'], 'Свои заявки');
const results = { PrimarySearchResults: hits, TotalRows: hits.length };
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;
}
/**
@@ -0,0 +1,167 @@
import { IRequestType } from '../../shared/approval';
/**
* Точечная отладка поиска по ОДНОМУ типу заявки.
*
* Зачем отдельный модуль: тип может выпасть из выдачи на любом из пяти шагов
* (не пришёл из API маршрутов → не попал в KQL → индекс его не отдал → Path
* хита не совпал с адресом списка → REST не вернул строку), и снаружи все пять
* выглядят одинаково — «заявок этого типа нет». Трассировка печатает каждый
* шаг отдельно, чтобы было видно, на каком именно набор становится пустым.
*
* Включение/выключение БЕЗ пересборки — из консоли браузера:
* window.rpSearchTrace = 'обходной' // подстрока имени типа или ключа формы
* window.rpSearchTrace = true // следить за всеми типами
* window.rpSearchTrace = false // выключить
* Значение сверяется с именем типа и с pageUrl (ключом формы), регистр не важен.
*/
const DEFAULT_TRACE: string | boolean = 'обходной';
const PREFIX = '[Поиск]';
function traceSetting(): string | boolean {
const w = window as unknown as { rpSearchTrace?: string | boolean };
return w.rpSearchTrace === undefined ? DEFAULT_TRACE : w.rpSearchTrace;
}
function matches(type: IRequestType, setting: string | boolean): boolean {
if (setting === true) return true;
if (!setting) return false;
const needle = String(setting).toLowerCase();
return (
(type.name || '').toLowerCase().indexOf(needle) >= 0 ||
(type.pageUrl || '').toLowerCase().indexOf(needle) >= 0
);
}
/** Типы, за которыми сейчас следим. Пусто — трассировка выключена. */
export function tracedTypes(types: IRequestType[]): IRequestType[] {
const setting = traceSetting();
if (!setting) return [];
return types.filter((t) => matches(t, setting));
}
export function traceEnabled(): boolean {
return !!traceSetting();
}
/**
* Шаг 1: что вообще пришло из API маршрутов и что из этого пригодно для чтения.
*
* Тип без listId/siteUrl молча отбрасывается (см. getLinkedTypes) — до поиска
* он не доходит вообще, и это первая версия, которую надо исключить.
*/
export function traceLinkedTypes(all: IRequestType[], linked: IRequestType[]): void {
const setting = traceSetting();
if (!setting) return;
const watched = all.filter((t) => matches(t, setting));
if (watched.length === 0) {
console.warn(
`${PREFIX} Тип по фильтру «${String(setting)}» не найден среди ${all.length} типов из API маршрутов. ` +
`Есть: ${all.map((t) => `${t.name} (${t.pageUrl})`).join(', ')}`
);
return;
}
watched.forEach((t) => {
const isLinked = linked.some((l) => l.id === t.id);
console.log(`${PREFIX} Тип «${t.name}» (id=${t.id}, ключ=${t.pageUrl}):`, {
'дошёл до поиска': isLinked,
listId: t.listId ?? '(нет)',
siteUrl: t.siteUrl ?? '(нет)',
});
if (!isLinked) {
console.warn(
`${PREFIX} Тип «${t.name}» в поиске НЕ участвует: у него нет listId и/или siteUrl ` +
`в approval-route API — заполните их в админке, иначе портал не знает, где лежит список.`
);
}
});
}
/**
* Шаг 2–4: KQL, сырые хиты индекса и сопоставление их с адресом списка.
*
* Path каждого хита сверяется с абсолютным адресом списка (см. hitsToEntries).
* Если индекс что-то отдал, а совпадений с этим типом нет — печатаем образцы
* путей: расхождение обычно видно глазами (другой сайт, /Lists/ против
* человекочитаемого адреса, иной регистр).
*/
export function traceSearchHits(
label: string,
kql: string,
types: IRequestType[],
results: { PrimarySearchResults: unknown[]; TotalRows?: number },
listAbsUrl: (type: IRequestType) => string
): void {
const watched = tracedTypes(types);
if (watched.length === 0) return;
const hits = (results.PrimarySearchResults as Array<Record<string, unknown>>).map((r) =>
String(r.Path || '').toLowerCase()
);
// Со слешем на конце — ровно как в боевом сопоставлении (см. listPathPrefix):
// иначе трассировка показывала бы «хиты есть» там, где код уже отдал их
// другому типу с более коротким адресом (…/Lists/List1 против …/Lists/List10).
const prefix = (t: IRequestType): string => `${listAbsUrl(t).toLowerCase().replace(/\/+$/, '')}/`;
console.log(`${PREFIX} ${label}: запрос отправлен`, {
KQL: kql,
'хитов получено': hits.length,
'TotalRows индекса': results.TotalRows ?? 0,
});
watched.forEach((t) => {
const url = prefix(t);
const mine = hits.filter((p) => p.indexOf(url) === 0);
console.log(`${PREFIX} ${label}: «${t.name}» — хитов индекса ${mine.length}`, {
'адрес списка (ожидаемый префикс Path)': url,
});
if (mine.length === 0) {
// Индекс мог не отдать НИЧЕГО (тогда дело не в путях), либо отдать
// чужие хиты (тогда почти наверняка не совпал префикс).
console.warn(
`${PREFIX} ${label}: по типу «${t.name}» индекс не вернул ни одного хита. ` +
(hits.length === 0
? 'Хитов нет вообще — проверьте сам запрос и права.'
: `Пути первых чужих хитов (сравните с ожидаемым префиксом выше): ${hits
.slice(0, 5)
.join(' | ')}`)
);
}
});
}
/**
* Шаг 5: сколько строк этого типа дожило до выдачи после сопоставления с
* адресом списка и дочитывания полей через REST.
*
* Расхождение «хиты были, строк нет» означает, что элементы из индекса в
* списке уже не существуют (удалены, индекс отстал) либо REST их не отдал.
*/
export function traceResultRows(
label: string,
types: IRequestType[],
entries: Array<{ type: IRequestType }>,
rows: Array<{ requestTypeId: number }>
): void {
const watched = tracedTypes(types);
if (watched.length === 0) return;
watched.forEach((t) => {
const mineEntries = entries.filter((e) => e.type.id === t.id).length;
const mineRows = rows.filter((r) => r.requestTypeId === t.id).length;
console.log(
`${PREFIX} ${label}: «${t.name}» — сопоставлено со списком ${mineEntries}, в выдаче ${mineRows}`
);
if (mineEntries > 0 && mineRows === 0) {
console.warn(
`${PREFIX} ${label}: по типу «${t.name}» элементы в индексе есть, но ни одной строки ` +
`не дочиталось через REST — вероятно, элементы удалены (индекс отстал) или список недоступен.`
);
}
});
}
@@ -70,17 +70,30 @@ const ClearanceForm: React.FC<IRequestFormProps> = ({ requestType, context, onCa
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [editId]);
// Выбор увольняемого сотрудника подтягивает его текущую должность — это только
// отправная точка для поля «Должность» (её можно поправить вручную, например
// если человек занимал руководящую должность до перевода), а не жёсткая связь.
// Новая заявка — дата расторжения по умолчанию сегодняшняя (типичный случай:
// обходной лист оформляют в день увольнения). При открытии на редактирование
// уже сохранённую дату не перезатираем.
useEffect(() => {
if (editId) {
return;
}
const today = new Date().toISOString().slice(0, 10);
setData((prev) => ({ ...prev, TerminationDate: prev.TerminationDate || today }));
}, [editId]);
// Должность/подразделение больше не вводятся вручную — подтягиваются из
// SharePoint User Profile (SPS-JobTitle/SPS-Department) при выборе сотрудника.
// Если профиль не синхронизирован и поле там пусто — пока просто остаётся
// пустым (TODO: фолбэк на справочник api.tnsu.kz, если это окажется частой
// проблемой на практике).
const handleEmployeeChange = (login: string): void => {
setData((prev) => ({ ...prev, Employee: login }));
setData((prev) => ({ ...prev, Employee: login, Position: '', Department: '' }));
if (!login) {
return;
}
approval
.getJobTitle(login)
.then((title) => setData((prev) => ({ ...prev, Position: title || prev.Position })))
.getProfileFields(login)
.then((profile) => setData((prev) => ({ ...prev, Position: profile.jobTitle, Department: profile.department })))
.catch(() => undefined);
};
@@ -12,3 +12,35 @@
justify-content: flex-end;
margin-bottom: $sp-4;
}
// Полоса «вы можете заполнить свои отметки» для профильного согласующего
// (КДП / отдел подбора) — см. approverRights.ts.
.reviewBar {
display: flex;
align-items: center;
gap: $sp-3;
margin-bottom: $sp-4;
padding: $sp-3 $sp-4;
border: 1px solid $gold-300;
border-radius: $r-md;
background: $gold-50;
}
.reviewHint {
flex: 1;
min-width: 0;
font-size: $fs-sm;
color: $navy-800;
}
.savedNote {
font-size: $fs-sm;
color: $green-600;
font-weight: 600;
}
.saveError {
font-size: $fs-sm;
color: $red-500;
font-weight: 600;
}
@@ -2,7 +2,7 @@ import * as React from 'react';
import { useState, useEffect, useMemo, useCallback } from 'react';
import type { WebPartContext } from '@microsoft/sp-webpart-base';
import styles from './ClearanceView.module.scss';
import { Alert, FormCard, InitiatorLine, FormModeProvider, PrintButton } from '../../shared/ui';
import { Alert, Button, FormCard, InitiatorLine, FormModeProvider, PrintButton } from '../../shared/ui';
import {
ApproverService,
DecisionPanel,
@@ -10,11 +10,19 @@ import {
ApprovalMatrix,
useMatrixEditor,
notifyActiveStep,
useRequestTypeId
useRequestTypeId,
APPROVAL_PHASE
} from '../../shared/approval';
import { IFormData, EMPTY_DATA, SetField } from './model';
import { loadClearanceRequest } from './load';
import { clearanceApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit';
import {
clearanceApprovalService,
updateApproverFields,
IApproverEditableFields,
DATA_LIST_TITLE,
APPROVAL_LIST_TITLE
} from './submit';
import { canEditHrRecordsFields, canEditParkingField } from './approverRights';
import { RequestSection } from './sections/RequestSection';
interface IClearanceViewProps {
@@ -24,18 +32,22 @@ interface IClearanceViewProps {
formKey: string;
}
// В режиме чтения поля не редактируются, поэтому обновление — пустышка.
const NOOP_SET: SetField = () => undefined;
type SaveState = 'idle' | 'saving' | 'saved' | 'error';
// Обходной лист только для просмотра (режим согласующего). Переиспользует ту
// же секцию, что и форма подачи; редактирование выключено через
// <FormModeProvider readOnly> — поля сами становятся неизменяемыми.
// Обходной лист в режиме согласующего. Форма read-only, КРОМЕ отметок, которые
// заполняет профильный отдел на своём шаге: КДП — карта СКУД и сумма обучения,
// отдел подбора — парковочная карта (см. approverRights.ts).
const ClearanceView: React.FC<IClearanceViewProps> = ({ context, itemId, formKey }) => {
const [data, setData] = useState<IFormData>(EMPTY_DATA);
const [status, setStatus] = useState<string>('');
const [code, setCode] = useState<string>('');
const [loading, setLoading] = useState<boolean>(true);
const [error, setError] = useState<string | null>(null);
const [hrRecordsEditable, setHrRecordsEditable] = useState<boolean>(false);
const [parkingEditable, setParkingEditable] = useState<boolean>(false);
const [saveState, setSaveState] = useState<SaveState>('idle');
const set: SetField = (key, value) => setData((prev) => ({ ...prev, [key]: value }));
const approval = useMemo(() => clearanceApprovalService(), []);
const { id: requestTypeId } = useRequestTypeId(context, formKey);
@@ -73,6 +85,55 @@ const ClearanceView: React.FC<IClearanceViewProps> = ({ context, itemId, formKey
};
}, [itemId]);
// Права на отметки — по должности ТЕКУЩЕГО активного шага пользователя.
// isDelegator намеренно не даёт прав: делегировавший не решает за делегата
// (см. DecisionPanel), значит и отметки за него ставить не должен.
useEffect(() => {
let alive = true;
approverService
.getPendingRowForCurrentUser(itemId)
.then((pending) => {
if (!alive) return;
const active = !!pending && !pending.isDelegator && pending.phase !== APPROVAL_PHASE.execution;
setHrRecordsEditable(active && canEditHrRecordsFields(pending?.position));
setParkingEditable(active && canEditParkingField(pending?.position));
})
.catch(() => {
if (alive) {
setHrRecordsEditable(false);
setParkingEditable(false);
}
});
return () => {
alive = false;
};
// decisionSeq: после решения/делегирования активный шаг меняется — права пересчитываем.
}, [itemId, approverService, decisionSeq]);
// Пишем ТОЛЬКО те отметки, на которые у согласующего есть права: иначе он
// затёр бы чужие поля значениями, которые видел у себя на экране.
const saveApproverFields = useCallback((): Promise<void> => {
const fields: Partial<IApproverEditableFields> = {};
if (hrRecordsEditable) {
fields.AccessCard = data.AccessCard;
fields.TrainingAmount = data.TrainingAmount;
}
if (parkingEditable) {
fields.ParkingCard = data.ParkingCard;
}
return updateApproverFields(itemId, fields);
}, [itemId, hrRecordsEditable, parkingEditable, data.AccessCard, data.TrainingAmount, data.ParkingCard]);
const handleSaveFields = async (): Promise<void> => {
setSaveState('saving');
try {
await saveApproverFields();
setSaveState('saved');
} catch {
setSaveState('error');
}
};
if (loading) {
return <div className={styles.state}>Загрузка заявки…</div>;
}
@@ -84,7 +145,8 @@ const ClearanceView: React.FC<IClearanceViewProps> = ({ context, itemId, formKey
);
}
const sectionProps = { data, set: NOOP_SET, context };
const canEditAnything = hrRecordsEditable || parkingEditable;
const sectionProps = { data, set, context, hrRecordsEditable, parkingEditable };
const notify = (): void => {
// requestTypeId ещё не резолвился — решение уже применилось (строка списка
@@ -104,6 +166,21 @@ const ClearanceView: React.FC<IClearanceViewProps> = ({ context, itemId, formKey
<PrintButton />
</div>
{canEditAnything && (
<div className={`${styles.reviewBar} rp-no-print`}>
<span className={styles.reviewHint}>
{hrRecordsEditable
? 'Как согласующий вы можете отметить карту СКУД и указать сумму за обучение.'
: 'Как согласующий вы можете отметить парковочную карту.'}
</span>
<Button variant="secondary" onClick={handleSaveFields} disabled={saveState === 'saving'}>
Сохранить
</Button>
{saveState === 'saved' && <span className={styles.savedNote}>Сохранено</span>}
{saveState === 'error' && <span className={styles.saveError}>Ошибка сохранения</span>}
</div>
)}
<div className="rp-print-root">
<FormCard
title="Обходной лист"
@@ -125,6 +202,9 @@ const ClearanceView: React.FC<IClearanceViewProps> = ({ context, itemId, formKey
service={approverService}
requestId={itemId}
search={(q) => approval.searchUsers(q)}
// Отметки сохраняются вместе с решением — иначе согласующий, забывший
// нажать «Сохранить», согласовал бы заявку со старыми значениями.
onBeforeDecide={canEditAnything ? saveApproverFields : undefined}
onDecided={(newStatus) => {
setStatus(newStatus);
setDecisionSeq((n) => n + 1);
@@ -0,0 +1,53 @@
/**
* Кто из согласующих какие поля обходного листа может править на своём шаге.
*
* Инициатор (HR) отправляет заявку с пустыми отметками — фактические данные знают
* профильные подразделения, каждое своё:
* - отдел кадрового делопроизводства — карта СКУД и сумма за обучение;
* - отдел подбора и адаптации персонала — парковочная карта.
*
* Должность берётся из строки согласования (колонка Title, см. IApprovalRow.position),
* то есть та, что подставил маршрут при планировании. Сравнение — по ключевым словам,
* а не по точной строке: формулировки в справочнике различаются («Начальник отдела…»,
* «Руководитель отдела…»), а тратить на это отдельную настройку в админке смысла нет.
*
* ВАЖНО про требование управленческого маркера (начальник/руководитель): без него
* права получили бы и рядовые сотрудники этих отделов («Специалист по направлению
* кадрового делопроизводства» в справочнике есть), оказавшиеся согласующими на любом
* другом шаге. Если понадобится отдать права и специалистам — убрать MANAGER_MARKERS
* из проверки.
*/
const MANAGER_MARKERS = ['начальник', 'руководитель'];
/** Отдел кадрового делопроизводства (КДП) — правит карту СКУД и сумму обучения. */
const HR_RECORDS_MARKERS = ['кадров', 'делопроизводств'];
/** Отдел подбора и адаптации персонала — правит парковочную карту. */
const RECRUITING_MARKERS = ['подбор', 'адаптац'];
function normalize(position: string | undefined): string {
return (position || '').toLowerCase();
}
function matchesAll(position: string, markers: string[]): boolean {
return markers.every((m) => position.indexOf(m) !== -1);
}
function isManagerOf(position: string | undefined, departmentMarkers: string[]): boolean {
const normalized = normalize(position);
if (!normalized) return false;
return (
MANAGER_MARKERS.some((m) => normalized.indexOf(m) !== -1) && matchesAll(normalized, departmentMarkers)
);
}
/** Карта СКУД и сумма обучения — начальник отдела кадрового делопроизводства. */
export function canEditHrRecordsFields(position: string | undefined): boolean {
return isManagerOf(position, HR_RECORDS_MARKERS);
}
/** Парковочная карта — начальник отдела подбора и адаптации персонала. */
export function canEditParkingField(position: string | undefined): boolean {
return isManagerOf(position, RECRUITING_MARKERS);
}
@@ -2,8 +2,27 @@ import * as React from 'react';
import { CollapseSection, FormField, TextInput, MoneyInput, Toggle, PeopleField } from '../../../shared/ui';
import { ISectionProps, EMPLOYEE_PLACEHOLDER } from '../model';
// Должность/подразделение подтягиваются автоматически при выборе сотрудника
// (см. ClearanceForm.handleEmployeeChange — SharePoint User Profile) —
// свободный ввод здесь намеренно убран, чтобы не расходиться с профилем.
const NOOP = (): void => undefined;
interface IRequestSectionProps extends ISectionProps {
/** Согласующий из КДП правит карту СКУД и сумму обучения даже в режиме просмотра. */
hrRecordsEditable?: boolean;
/** Согласующий из отдела подбора правит парковочную карту. */
parkingEditable?: boolean;
}
/** Единственная секция формы «Обходной лист»: увольняемый, должность, подразделение, даты, карты, обучение. */
export const RequestSection: React.FC<ISectionProps> = ({ data, set, context, onEmployeeChange }) => (
export const RequestSection: React.FC<IRequestSectionProps> = ({
data,
set,
context,
onEmployeeChange,
hrRecordsEditable,
parkingEditable
}) => (
<CollapseSection title="Обходной лист" icon="people">
<FormField label="Сотрудник" htmlFor="Employee" required full>
<PeopleField
@@ -14,10 +33,22 @@ export const RequestSection: React.FC<ISectionProps> = ({ data, set, context, on
/>
</FormField>
<FormField label="Должность" htmlFor="Position" required>
<TextInput id="Position" value={data.Position} onChange={(v) => set('Position', v)} />
<TextInput
id="Position"
value={data.Position}
onChange={NOOP}
readOnly
placeholder="Подставится автоматически по выбранному сотруднику"
/>
</FormField>
<FormField label="Подразделение" htmlFor="Department">
<TextInput id="Department" value={data.Department} onChange={(v) => set('Department', v)} />
<TextInput
id="Department"
value={data.Department}
onChange={NOOP}
readOnly
placeholder="Подставится автоматически по выбранному сотруднику"
/>
</FormField>
<FormField label="Дата расторжения договора" htmlFor="TerminationDate" required>
<TextInput
@@ -27,14 +58,31 @@ export const RequestSection: React.FC<ISectionProps> = ({ data, set, context, on
onChange={(v) => set('TerminationDate', v)}
/>
</FormField>
{/* Явный readOnly={false} перебивает read-only контекст просмотра (профильный
согласующий правит своё поле); undefined — обычное поведение из FormMode. */}
<FormField label="Обучение: сумма" htmlFor="TrainingAmount">
<MoneyInput id="TrainingAmount" value={data.TrainingAmount} onChange={(v) => set('TrainingAmount', v)} />
<MoneyInput
id="TrainingAmount"
value={data.TrainingAmount}
onChange={(v) => set('TrainingAmount', v)}
readOnly={hrRecordsEditable ? false : undefined}
/>
</FormField>
<FormField label="Парковочная карта">
<Toggle checked={data.ParkingCard} onChange={(v) => set('ParkingCard', v)} onLabel="Да" />
<Toggle
checked={data.ParkingCard}
onChange={(v) => set('ParkingCard', v)}
onLabel="Да"
readOnly={parkingEditable ? false : undefined}
/>
</FormField>
<FormField label="Карта СКУД">
<Toggle checked={data.AccessCard} onChange={(v) => set('AccessCard', v)} onLabel="Да" />
<Toggle
checked={data.AccessCard}
onChange={(v) => set('AccessCard', v)}
onLabel="Да"
readOnly={hrRecordsEditable ? false : undefined}
/>
</FormField>
</CollapseSection>
);
@@ -11,6 +11,34 @@ export function clearanceApprovalService(): ApprovalListService {
return new ApprovalListService(APPROVAL_LIST_TITLE);
}
/** Отметки, которые проставляет не инициатор, а профильный согласующий (см. approverRights.ts). */
export interface IApproverEditableFields {
AccessCard: boolean;
TrainingAmount: string;
ParkingCard: boolean;
}
/**
* Обновить только те отметки, что заполняет согласующий на своём шаге.
* Пишутся ровно переданные поля: у каждого отдела свой набор прав, и лишние
* ключи затёрли бы чужие отметки значениями из формы этого согласующего.
*/
export async function updateApproverFields(
id: number,
fields: Partial<IApproverEditableFields>
): Promise<void> {
const item: Record<string, unknown> = {};
if (fields.AccessCard !== undefined) item.AccessCard = fields.AccessCard;
if (fields.ParkingCard !== undefined) item.ParkingCard = fields.ParkingCard;
if (fields.TrainingAmount !== undefined) {
const amount = Number(fields.TrainingAmount);
item.TrainingAmount =
Number.isFinite(amount) && fields.TrainingAmount.trim() !== '' ? amount : null;
}
if (Object.keys(item).length === 0) return;
await getSP().web.lists.getByTitle(DATA_LIST_TITLE).items.getById(id).update(item);
}
/** email / claims-логин → пользователь сайта (Id для Person-поля + Title для заголовка). */
async function ensureEmployee(login: string): Promise<{ Id: number; Title: string }> {
const full = login.indexOf('|') >= 0 ? login : `i:0#.f|membership|${login}`;
@@ -34,6 +34,7 @@ import {
deleteAttachment,
IAttachmentInfo
} from '../../shared/attachments/attachments.service';
import { effectiveUserEmail } from '../../shared/impersonation';
import { IRequestFormProps } from '../../models/types';
import { IFormData, EMPTY_DATA, SetField } from './model';
import { saveHireRequest, hireApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit';
@@ -89,7 +90,7 @@ const RecruitingHireForm: React.FC<IRequestFormProps> = ({ requestType, context,
// редактирование, маршрут читается из уже записанных строк — планирование
// затёрло бы правки, сделанные при первом сохранении.
const approval = useMemo(() => hireApprovalService(), []);
const userEmail = context.pageContext.user.email;
const userEmail = effectiveUserEmail(context);
// pageUrl формы (requestType.id совпадает с ним по конвенции портала, см.
// data/requests.ts) → числовой RequestTypeId, нужный approval-route API.
const requestTypeIdState = useRequestTypeId(context, requestType.id);
@@ -33,6 +33,7 @@ import {
deleteAttachment,
IAttachmentInfo
} from '../../shared/attachments/attachments.service';
import { effectiveUserEmail } from '../../shared/impersonation';
import { IRequestFormProps } from '../../models/types';
import { IFormData, EMPTY_DATA, SetField } from './model';
import { saveSelectionRequest, selectionApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit';
@@ -86,7 +87,7 @@ const RecruitingSelectionForm: React.FC<IRequestFormProps> = ({ requestType, con
// У заявки на редактировании маршрут читается из уже записанных строк, чтобы не
// затереть правки инициатора при первом сохранении.
const approval = useMemo(() => selectionApprovalService(), []);
const userEmail = context.pageContext.user.email;
const userEmail = effectiveUserEmail(context);
// pageUrl формы (requestType.id совпадает с ним по конвенции портала, см.
// data/requests.ts) → числовой RequestTypeId, нужный approval-route API.
const requestTypeIdState = useRequestTypeId(context, requestType.id);
@@ -33,6 +33,7 @@ import {
deleteAttachment,
IAttachmentInfo
} from '../../shared/attachments/attachments.service';
import { effectiveUserEmail } from '../../shared/impersonation';
import { IRequestFormProps } from '../../models/types';
import { IFormData, EMPTY_DATA, SetField } from './model';
import {
@@ -132,7 +133,7 @@ const Ticket1CForm: React.FC<IRequestFormProps> = ({ requestType, context, onCan
}, [editId]);
const approval = useMemo(() => ticket1cApprovalService(), []);
const userEmail = context.pageContext.user.email;
const userEmail = effectiveUserEmail(context);
// pageUrl формы (requestType.id совпадает с ним по конвенции портала, см.
// data/requests.ts) → числовой RequestTypeId, нужный approval-route API.
const requestTypeIdState = useRequestTypeId(context, requestType.id);
@@ -34,6 +34,7 @@ import {
deleteAttachment,
IAttachmentInfo
} from '../../shared/attachments/attachments.service';
import { effectiveUserEmail } from '../../shared/impersonation';
import { IRequestFormProps } from '../../models/types';
import { IFormData, EMPTY_DATA, SetField } from './model';
import { saveTmcTransferRequest, tmcTransferApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit';
@@ -63,7 +64,10 @@ const TmcTransferForm: React.FC<IRequestFormProps> = ({ requestType, context, on
const today = new Date().toISOString().slice(0, 10);
setData((prev) => ({
...prev,
RequesterEmail: user.email || '',
// email — через effectiveUserEmail (учитывает отладочную подмену личности);
// displayName подменить нечем, справочник ФИО тут не читается — при
// подмене имя останется настоящим, для отладки это неважно.
RequesterEmail: effectiveUserEmail(context),
AuthorName: user.displayName || '',
RequestDate: prev.RequestDate || today
}));
@@ -95,7 +99,7 @@ const TmcTransferForm: React.FC<IRequestFormProps> = ({ requestType, context, on
}, [editId]);
const approval = useMemo(() => tmcTransferApprovalService(), []);
const userEmail = context.pageContext.user.email;
const userEmail = effectiveUserEmail(context);
// pageUrl формы (requestType.id совпадает с ним по конвенции портала, см.
// data/requests.ts) → числовой RequestTypeId, нужный approval-route API.
const requestTypeIdState = useRequestTypeId(context, requestType.id);
@@ -34,6 +34,7 @@ import {
deleteAttachment,
IAttachmentInfo
} from '../../shared/attachments/attachments.service';
import { effectiveUserEmail } from '../../shared/impersonation';
import { IRequestFormProps } from '../../models/types';
import { IFormData, EMPTY_DATA, SetField } from './model';
import { saveWorkplaceRequest, workplaceApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit';
@@ -108,7 +109,7 @@ const WorkplaceSetupForm: React.FC<IRequestFormProps> = ({ requestType, context,
}, [sourceId]);
const approval = useMemo(() => workplaceApprovalService(), []);
const userEmail = context.pageContext.user.email;
const userEmail = effectiveUserEmail(context);
// pageUrl формы (requestType.id совпадает с ним по конвенции портала, см.
// data/requests.ts) → числовой RequestTypeId, нужный approval-route API.
const requestTypeIdState = useRequestTypeId(context, requestType.id);
+2
View File
@@ -3,6 +3,8 @@ define([], function() {
"PropertyPaneDescription": "Description",
"BasicGroupName": "Group Name",
"DescriptionFieldLabel": "Description Field",
"DebugUserEmailFieldLabel": "Отладка: работать от имени сотрудника",
"DebugUserEmailFieldDescription": "Только для отладки маршрутов. Портал будет считать текущим пользователем указанный e-mail: по нему планируется маршрут, отбираются «свои» заявки и очередь согласования. Прав это не даёт — запись в списки всё равно идёт под вашей учёткой. На боевой странице оставьте пустым.",
"AppLocalEnvironmentSharePoint": "The app is running on your local environment as SharePoint web part",
"AppLocalEnvironmentTeams": "The app is running on your local environment as Microsoft Teams app",
"AppLocalEnvironmentOffice": "The app is running on your local environment in office.com",
+2
View File
@@ -2,6 +2,8 @@ declare interface IRequestPortalWebPartStrings {
PropertyPaneDescription: string;
BasicGroupName: string;
DescriptionFieldLabel: string;
DebugUserEmailFieldLabel: string;
DebugUserEmailFieldDescription: string;
AppLocalEnvironmentSharePoint: string;
AppLocalEnvironmentTeams: string;
AppLocalEnvironmentOffice: string;
@@ -5,6 +5,8 @@ import { IList } from '@pnp/sp/lists';
import '@pnp/sp/items';
import '@pnp/sp/site-users/web';
import '@pnp/sp/profiles';
import '@pnp/sp/batching';
import { getImpersonatedEmail } from '../impersonation';
import { IApprovalRow } from './models/IApprovalRow';
import { IMatrixRow } from './models/IMatrixRow';
import { ISiteUser } from './models/ISiteUser';
@@ -105,11 +107,31 @@ export class ApprovalListService {
);
}
for (const { a, order } of withOrder.filter((x) => !!x.a.email)) {
const approverId = await this.resolveUserId(a.email);
await this.list().items.add({
const toWrite = withOrder.filter((x) => !!x.a.email);
if (toWrite.length === 0) return;
// Резолв Id согласующих — независимые запросы (ensureUser по email),
// ждать их по очереди незачем. На длинном маршруте (десяток+ шагов) это
// было основным источником задержки наравне с самой записью строк ниже.
const approverIds = await Promise.all(toWrite.map(({ a }) => this.resolveUserId(a.email)));
// Сами строки — ОДНИМ HTTP-запросом через batch API SharePoint, а не по
// POST на строку: add() на batched-инстансе не уходит в сеть сразу, а
// копится и отправляется одним вызовом по execute().
//
// ВАЖНО: batchedList (сам список) переиспользуем, а .items достаём ЗАНОВО
// на каждой итерации — PnPjs один и тот же Items-инстанс во второй add()
// внутри одного батча не даёт («This instance is already part of a batch»,
// см. docs/concepts/batching#reuse). .getById(id) ниже в deleteForRequest
// этой проблемы не имеет — он и так создаёт новый инстанс на каждый вызов.
const [batchedSP, execute] = getSP().batched();
const batchedList = batchedSP.web.lists.getByTitle(this.listTitle);
toWrite.forEach(({ a, order }, i) => {
// void: промис от add() на batched-инстансе нарочно не ждём поштучно —
// он разрешится вместе со всеми остальными после execute() ниже.
void batchedList.items.add({
Title: a.position || a.email,
[`${FIELDS.approver}Id`]: approverId,
[`${FIELDS.approver}Id`]: approverIds[i],
[FIELDS.status]: status,
[`${FIELDS.ticket}Id`]: requestId,
[FIELDS.order]: order,
@@ -119,7 +141,8 @@ export class ApprovalListService {
[FIELDS.slaHours]: a.slaHours ?? null,
[FIELDS.askOverdueReason]: a.askOverdueReason ?? null,
});
}
});
await execute();
}
// Продвигает очередь: среди ещё не активных строк со статусом «Ждёт очереди»
@@ -205,6 +228,7 @@ export class ApprovalListService {
.items.filter(`${FIELDS.ticket}Id eq ${requestId}`)
.select(
'Id',
'Title',
FIELDS.status,
FIELDS.order,
FIELDS.allowReassign,
@@ -224,6 +248,7 @@ export class ApprovalListService {
approverId: approver?.Id ?? 0,
order: (r[FIELDS.order] as number) ?? 0,
allowReassign: (r[FIELDS.allowReassign] as boolean) ?? false,
position: (r.Title as string) || '',
// Пустая фаза (старые строки без колонки) трактуется как «Согласование».
phase: (r[FIELDS.phase] as string) || APPROVAL_PHASE.approval,
delegatedById: delegatedBy?.Id ?? 0,
@@ -301,7 +326,15 @@ export class ApprovalListService {
// сотрудника отдаёт именно e-mail (см. pickedLogin), а getPropertiesFor ждёт
// account name — приводим e-mail к клеймовому виду так же, как ensure().
public async getJobTitle(loginName: string): Promise<string> {
if (!loginName) return '';
return (await this.getProfileFields(loginName)).jobTitle;
}
// Должность + подразделение из профиля SharePoint (SPS-JobTitle/SPS-Department)
// ОДНИМ запросом — используется там, где нужны оба поля (напр. предзаполнение
// «Обходного листа»), чтобы не дёргать getPropertiesFor дважды подряд.
// '' по каждому полю, если недоступно/профиль не синхронизирован.
public async getProfileFields(loginName: string): Promise<{ jobTitle: string; department: string }> {
if (!loginName) return { jobTitle: '', department: '' };
const account = loginName.indexOf('|') >= 0 ? loginName : `i:0#.f|membership|${loginName}`;
try {
const props = (await getSP().profiles.getPropertiesFor(account)) as {
@@ -313,9 +346,9 @@ export class ApprovalListService {
: ((raw as { results?: Array<{ Key: string; Value: string }> })?.results ?? []);
const map: Record<string, string> = {};
for (const p of list) map[p.Key] = p.Value;
return map['SPS-JobTitle'] || map.Title || '';
return { jobTitle: map['SPS-JobTitle'] || map.Title || '', department: map['SPS-Department'] || '' };
} catch {
return '';
return { jobTitle: '', department: '' };
}
}
@@ -333,13 +366,28 @@ export class ApprovalListService {
// Удаляет все строки согласования по заявке (при возврате/повторной отправке).
public async deleteForRequest(requestId: number): Promise<void> {
const rows = await this.getForRequest(requestId);
for (const row of rows) {
await this.list().items.getById(row.id).delete();
}
if (rows.length === 0) return;
// writeRoute вызывает это перед перезаписью (пересохранение черновика,
// повторная отправка) — тот же per-row POST/DELETE в цикле, что и
// раньше в createSteps, и то же решение: один batch вместо N запросов.
const [batchedSP, execute] = getSP().batched();
const batchedItems = batchedSP.web.lists.getByTitle(this.listTitle).items;
rows.forEach((row) => {
// void — по той же причине, что и в createSteps выше.
void batchedItems.getById(row.id).delete();
});
await execute();
}
// Id текущего пользователя на сайте (для сравнения с полем «Согласующий»).
// При отладочной подмене личности (см. shared/impersonation.ts) отдаём Id
// подменённого сотрудника: иначе «моя очередь согласования» и делегирование
// считались бы по реальной учётке, и подмена не позволяла бы проверить шаг
// чужого согласующего — ради чего она и заводилась.
public async getCurrentUserId(): Promise<number> {
const impersonated = getImpersonatedEmail();
if (impersonated) return this.resolveUserId(impersonated);
const user = await getSP().web.currentUser.select('Id')();
return user.Id;
}
@@ -360,6 +408,24 @@ export class ApprovalListService {
}
}
// Id пользователя сайта → email. Обратное к resolveUserId: строки маршрута
// хранят согласующего числовым Id (Person-поле), а адрес нужен там, где
// решение принимается именно по почте — сейчас это заглушка рассылки
// (см. notify.service.ts).
//
// '' при любой неудаче (пользователя нет, поле пустое, запрос упал) —
// вызывающая сторона трактует это как «адрес неизвестен» и ведёт себя
// консервативно, а не считает, что проверка пройдена.
public async resolveUserEmail(userId: number): Promise<string> {
if (!userId) return '';
try {
const user = (await getSP().web.siteUsers.getById(userId).select('Email')()) as { Email?: string };
return user.Email || '';
} catch {
return '';
}
}
private async ensure(email: string): Promise<{ Id: number; Title: string }> {
const login = email.indexOf('|') >= 0 ? email : `i:0#.f|membership|${email}`;
return getSP().web.ensureUser(login);
@@ -5,6 +5,10 @@ export interface IApprovalRow {
approverId: number; // Id пользователя сайта в поле «Согласующий»
order: number; // колонка Order (очередь согласования)
allowReassign: boolean; // AllowReassignToOtherApprovers
/** Должность/роль шага — колонка Title строки (см. createSteps: `a.position || a.email`).
* Нужна формам, где ПРАВА согласующего зависят от его должности (напр. «Обходной
* лист»: КДП правит СКУД и сумму обучения, подбор — парковочную карту). */
position: string;
phase: string; // Phase: «Согласование» | «Исполнение» (см. APPROVAL_PHASE)
/** Id пользователя сайта, который делегировал ЭТУ строку (DelegatedBy). 0 — строка не делегирована
* (пришла из маршрута напрямую). Позволяет делегировавшему переназначить шаг ещё раз,
@@ -4,6 +4,21 @@ import { FLOW_RESOURCE, FLOW_NOTIFY_URL } from '../config';
import { ApprovalListService } from './approvalList.service';
import { REQUEST_STATUS } from './models/statuses';
/**
* ВРЕМЕННАЯ ЗАГЛУШКА на период обкатки рассылки: уведомление отправляется,
* только если активный шаг назначен на этот адрес. Для всех остальных
* согласующих поток не вызывается вообще — письма не будет.
*
* Ограничение стоит ЗДЕСЬ, на стороне портала, а не в потоке Power Automate,
* потому что адресата поток определяет САМ по RowID — тело запроса на выбор
* получателя не влияет. Значит, единственный способ не отправить письмо, не
* трогая поток, — не дёргать его. Переадресовать письмо на тестовый ящик
* отсюда нельзя по той же причине: поток такого параметра не принимает.
*
* Пустая строка — заглушка снята, уведомления идут всем как обычно.
*/
const NOTIFY_ONLY_TO = 'gorbenko.pavel@tnsu.kz';
// Данные формы для тела запроса к потоку (постоянны для формы).
export interface INotifyTarget {
requestTypeId: number; // RequestTypeId (id типа заявки во внешнем API маршрутов)
@@ -27,6 +42,24 @@ async function postNotify(context: WebPartContext, body: Record<string, unknown>
}
}
// Разрешена ли отправка на согласующего активной строки (см. NOTIFY_ONLY_TO).
//
// Неизвестный адрес (пользователь не найден, поле пустое, запрос упал) —
// НЕ отправляем: пока заглушка включена, сомнение трактуется в пользу
// «письмо живому человеку лучше не слать», а не наоборот.
async function isNotifyAllowed(approval: ApprovalListService, approverId: number): Promise<boolean> {
if (!NOTIFY_ONLY_TO) return true;
const email = await approval.resolveUserEmail(approverId);
if (email.toLowerCase() === NOTIFY_ONLY_TO.toLowerCase()) return true;
console.warn(
`[Оповещение] Включена заглушка рассылки: письмо для «${email || 'адрес неизвестен'}» не отправлено ` +
`(разрешён только ${NOTIFY_ONLY_TO})`
);
return false;
}
// Находит АКТИВНЫЙ шаг маршрута (На согласовании / На исполнении) и уведомляет его.
// Вызывать после отправки заявки и после каждого положительного решения. Поток по
// переданным полям сам находит согласующего/исполнителя и шлёт уведомление.
@@ -45,6 +78,9 @@ export async function notifyActiveStep(
if (!active) {
return;
}
if (!(await isNotifyAllowed(approval, active.approverId))) {
return;
}
await postNotify(context, {
RequestTypeId: target.requestTypeId,
RowID: active.id,
@@ -1,63 +0,0 @@
@import '../../../../../styles/design-tokens';
.badge {
display: inline-flex;
align-items: center;
padding: 5px 12px;
border-radius: $r-sm;
font-family: $font-sans;
font-size: 12.5px;
font-weight: 600;
letter-spacing: .2px;
white-space: nowrap;
}
// Компактный вариант — для ячейки таблицы (RequestsTableView/SearchView),
// повторяет размеры, что были у их собственной .status до объединения.
.sm {
padding: 3px 10px;
border-radius: $r-md;
font-size: $fs-caption;
font-weight: 600;
letter-spacing: normal;
}
.draft {
background: $gray-100;
color: $gray-700;
}
.queued {
background: $gray-100;
color: $gray-500;
}
.onApproval {
background: $blue-50;
color: $blue-600;
}
.approved {
background: $green-50;
color: $green-600;
}
.rejected {
background: $red-50;
color: $red-500;
}
.returned {
background: $gold-50;
color: $amber-700;
}
.execution {
background: $navy-50;
color: $navy-800;
}
.closed {
background: $gray-100;
color: $gray-500;
}
@@ -1,40 +1,33 @@
import * as React from 'react';
import styles from './StatusBadge.module.scss';
import { Badge, BadgeVariant } from '@tnsu/ui-kit';
import { REQUEST_STATUS } from '../models/statuses';
// Цвет бейджа по статусу заявки. Неизвестный статус → нейтральный (как черновик).
const VARIANT: Record<string, string> = {
[REQUEST_STATUS.draft]: styles.draft,
[REQUEST_STATUS.queued]: styles.queued,
[REQUEST_STATUS.onApproval]: styles.onApproval,
[REQUEST_STATUS.onExecution]: styles.onApproval, // активный шаг — как «на согласовании»
[REQUEST_STATUS.approved]: styles.approved,
[REQUEST_STATUS.executed]: styles.approved, // выполнено — как «согласовано» (зелёный)
[REQUEST_STATUS.rejected]: styles.rejected,
[REQUEST_STATUS.returned]: styles.returned,
[REQUEST_STATUS.execution]: styles.execution,
[REQUEST_STATUS.closed]: styles.closed,
// Единственное место, где СТАТУС ЗАЯВКИ превращается в цвет. Сам цвет живёт в
// generic Badge (@tnsu/ui-kit) — здесь только смысловой маппинг «статус → variant».
// Раньше у таблиц была своя копия этой раскраски (statusClass) — она разошлась с
// этой, поэтому обе копии убраны в пользу общего компонента.
const STATUS_VARIANT: Record<string, BadgeVariant> = {
[REQUEST_STATUS.draft]: 'neutral',
[REQUEST_STATUS.queued]: 'muted',
[REQUEST_STATUS.onApproval]: 'info',
[REQUEST_STATUS.onExecution]: 'info', // активный шаг — как «на согласовании»
[REQUEST_STATUS.approved]: 'success',
[REQUEST_STATUS.executed]: 'success', // выполнено — как «согласовано» (зелёный)
[REQUEST_STATUS.rejected]: 'danger',
[REQUEST_STATUS.returned]: 'warning',
[REQUEST_STATUS.execution]: 'primary',
[REQUEST_STATUS.closed]: 'muted'
};
export interface IStatusBadgeProps {
status: string;
/**
* 'md' (по умолчанию) — шапка формы/просмотра заявки; 'sm' — компактный
* вариант для ячейки таблицы (RequestsTableView/SearchView). Цвет статуса
* не зависит от размера — единственный источник цвета: VARIANT выше.
*/
/** 'md' (по умолчанию) — шапка формы; 'sm' — компактный для ячейки таблицы. */
size?: 'sm' | 'md';
}
/**
* Цветная плашка статуса заявки — ЕДИНСТВЕННОЕ место, где статус превращается
* в цвет. У таблиц раньше была своя копия этой раскраски (statusClass в
* RequestsTableView.tsx/SearchView.tsx) — она разошлась с этой (на согласовании
* и возвращена на доработку оказались перепутаны местами), поэтому обе копии
* убраны в пользу общего компонента.
*/
/** Цветная плашка статуса заявки поверх generic Badge из @tnsu/ui-kit. */
export const StatusBadge: React.FC<IStatusBadgeProps> = ({ status, size = 'md' }) => (
<span className={`${styles.badge} ${size === 'sm' ? styles.sm : ''} ${VARIANT[status] || styles.draft}`}>
<Badge variant={STATUS_VARIANT[status] || 'neutral'} size={size}>
{status}
</span>
</Badge>
);
@@ -0,0 +1,39 @@
import { WebPartContext } from '@microsoft/sp-webpart-base';
/**
* ОТЛАДОЧНАЯ подмена «текущего пользователя»: портал ведёт себя так, будто им
* пользуется указанный сотрудник. Нужна, чтобы проверять маршруты и права
* согласующего (например, «карту СКУД правит начальник КДП» в обходном листе),
* не выпрашивая учётку у каждого участника маршрута.
*
* Значение задаётся в свойствах веб-части (property pane, см.
* RequestPortalWebPart.getPropertyPaneConfiguration) и живёт на уровне модуля —
* иначе его пришлось бы протаскивать пропсами через всё дерево до каждой формы.
*
* ЧТО ЭТО МЕНЯЕТ: только то, ЧЕЙ email портал отправляет как инициатора при
* планировании маршрута и по какому пользователю фильтрует «свои» заявки и
* очередь согласования.
*
* ЧЕГО ЭТО НЕ МЕНЯЕТ: прав доступа. Запись в SharePoint по-прежнему идёт под
* реальной учёткой (Author/Editor в списках — настоящий пользователь), токен к
* approval-route API тоже настоящий. То есть подмена не даёт увидеть то, к чему
* нет доступа, но ПОЗВОЛЯЕТ принять решение за чужой шаг согласования — поэтому
* на бою поле должно быть пустым (см. предупреждающую полосу в RequestPortal).
*/
let impersonatedEmail = '';
/** Вызывается веб-частью на каждый render — свойство могли поменять в панели. */
export function setImpersonatedEmail(email: string | undefined): void {
impersonatedEmail = (email || '').trim();
}
/** Заданная подмена ('' — подмены нет). Для предупреждающей полосы в интерфейсе. */
export function getImpersonatedEmail(): string {
return impersonatedEmail;
}
/** Email, под которым портал работает: подменённый, если задан, иначе — из токена. */
export function effectiveUserEmail(context: WebPartContext): string {
return impersonatedEmail || context.pageContext.user.email || '';
}