From e49d0f0171e06abdfd73735885408c0f5d0ef7dc Mon Sep 17 00:00:00 2001 From: "gorbenko.pavel" Date: Fri, 14 Aug 2026 11:12:12 +0500 Subject: [PATCH] =?UTF-8?q?=D0=BD=D0=BE=D0=B2=D0=B0=D1=8F=20=D1=84=D0=BE?= =?UTF-8?q?=D1=80=D0=BC=D0=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/forms.md | 15 ++ src/webparts/requestPortal/data/requests.ts | 1 + .../features/approve/ApproveFeature.tsx | 2 +- .../forms/Clearance/ClearanceForm.module.scss | 20 ++ .../forms/Clearance/ClearanceForm.tsx | 223 ++++++++++++++++++ .../forms/Clearance/ClearanceView.module.scss | 14 ++ .../forms/Clearance/ClearanceView.tsx | 143 +++++++++++ .../requestPortal/forms/Clearance/README.md | 48 ++++ .../requestPortal/forms/Clearance/load.ts | 56 +++++ .../requestPortal/forms/Clearance/model.ts | 49 ++++ .../Clearance/sections/RequestSection.tsx | 40 ++++ .../requestPortal/forms/Clearance/submit.ts | 84 +++++++ .../RecruitingHire/RecruitingHireForm.tsx | 43 ++-- .../RecruitingHire/RecruitingHireView.tsx | 30 ++- .../forms/RecruitingHire/submit.ts | 6 +- .../RecruitingSelectionForm.tsx | 40 ++-- .../RecruitingSelectionView.tsx | 18 +- .../forms/RecruitingSelection/submit.ts | 5 +- .../forms/Ticket1C/Ticket1CForm.tsx | 33 ++- .../forms/Ticket1C/Ticket1CView.tsx | 30 ++- .../requestPortal/forms/Ticket1C/submit.ts | 9 +- .../forms/TmcTransfer/TmcTransferForm.tsx | 47 ++-- .../forms/TmcTransfer/TmcTransferView.tsx | 36 +-- .../requestPortal/forms/TmcTransfer/submit.ts | 6 +- .../forms/Vacation/VacationForm.tsx | 38 +-- .../forms/Vacation/VacationView.tsx | 21 +- .../requestPortal/forms/Vacation/submit.ts | 8 +- .../WorkplaceSetup/WorkplaceSetupForm.tsx | 43 ++-- .../WorkplaceSetup/WorkplaceSetupView.tsx | 39 ++- .../forms/WorkplaceSetup/submit.ts | 5 +- src/webparts/requestPortal/forms/registry.ts | 3 + .../requestPortal/forms/viewRegistry.ts | 10 + .../shared/approval/approvalRoute.service.ts | 37 +++ .../requestPortal/shared/approval/index.ts | 4 +- .../shared/approval/useRequestTypeId.ts | 46 ++++ 35 files changed, 1052 insertions(+), 200 deletions(-) create mode 100644 src/webparts/requestPortal/forms/Clearance/ClearanceForm.module.scss create mode 100644 src/webparts/requestPortal/forms/Clearance/ClearanceForm.tsx create mode 100644 src/webparts/requestPortal/forms/Clearance/ClearanceView.module.scss create mode 100644 src/webparts/requestPortal/forms/Clearance/ClearanceView.tsx create mode 100644 src/webparts/requestPortal/forms/Clearance/README.md create mode 100644 src/webparts/requestPortal/forms/Clearance/load.ts create mode 100644 src/webparts/requestPortal/forms/Clearance/model.ts create mode 100644 src/webparts/requestPortal/forms/Clearance/sections/RequestSection.tsx create mode 100644 src/webparts/requestPortal/forms/Clearance/submit.ts create mode 100644 src/webparts/requestPortal/shared/approval/useRequestTypeId.ts diff --git a/docs/forms.md b/docs/forms.md index 638e9f0..9a11499 100644 --- a/docs/forms.md +++ b/docs/forms.md @@ -11,6 +11,20 @@ в `forms/viewRegistry.ts`. 4. Плитка в `data/requests.ts`. +## Формы больше не хардкодят RequestTypeId +Числовой id типа заявки во внешнем approval-route API (`GET /api/request-types`) +выдаётся сервером при регистрации типа в админке и непредсказуем заранее (в +старой pto-tracker у «отпуска» было 1, при заведении типа в новой админке он +получил id 11). Раньше каждая форма хардкодила это число константой вида +`VACATION_REQUEST_TYPE_ID = 11`; теперь — резолвит его в рантайме по своему +`pageUrl` (тот же rptype, что и `id` в `data/requests.ts`) через +`useRequestTypeId(context, pageUrl)` (`shared/approval`). Результат кешируется +на уровне модуля на время сессии (см. `getRequestTypeIdByPageUrl` в +`approvalRoute.service.ts`) — повторный вызов из другой формы не бьёт по сети заново. +- Форма подачи знает свой `pageUrl` из пропа `requestType.id`. +- Режим согласующего (`<Форма>View`) получает его пропом `formKey` (см. + `IApprovalViewProps` в `forms/viewRegistry.ts` и `ApproveFeature.tsx`). + ## Что НЕ дублируем в карточке (лежит в других местах) - Колонки списков → `forms/<Форма>/provisioning.json`. - Параметры ссылок (rptype/rpmode/rpid/rpsource) → [`routing.md`](./routing.md). @@ -30,6 +44,7 @@ | [`recruiting-hire`](../src/webparts/requestPortal/forms/RecruitingHire/README.md) | Заявка на прием | Рекрутинг | да | да | да | маска телефона, деньги, условное «взамен кого», источник для орг. места | ✅ | | [`it-workplace-setup`](../src/webparts/requestPortal/forms/WorkplaceSetup/README.md) | Организация рабочего места | IT | да | да | да | префилл из «прием», поля ПО/доступы правит руководитель | ✅ | | [`it-1c-ticket`](../src/webparts/requestPortal/forms/Ticket1C/README.md) | Заявка в отдел 1С | IT | да | нет | да | контакты предзаполняются из Graph (/me); вместо фазы исполнения — карточка в Microsoft Planner (согласовано/сразу без согласования — рабочий бакет, отклонено — отдельный бакет) | ✅ | +| [`clearance-form`](../src/webparts/requestPortal/forms/Clearance/README.md) | Обходной лист | Отдел кадров | да | нет | да | подаётся HR за увольняемого (People-picker); условие «IsManager» резолвится по должности из самой заявки; да/нет-поля (парковка, СКУД), сумма обучения | ✅ | | `hr-transfer` | Перемещение сотрудников | Отдел кадров | — | — | — | — | ⏳ заглушка (плитка есть, форма не сделана) | Легенда: **Согл.** — есть фаза согласования; **Исп.** — есть фаза исполнения; diff --git a/src/webparts/requestPortal/data/requests.ts b/src/webparts/requestPortal/data/requests.ts index d63a712..ff90d39 100644 --- a/src/webparts/requestPortal/data/requests.ts +++ b/src/webparts/requestPortal/data/requests.ts @@ -15,6 +15,7 @@ export const REQUEST_TYPES: IRequestType[] = [ // Отдел кадров { id: 'hr-vacation', categoryId: 'hr', title: 'Заявка на отпуск', icon: 'calendar' }, + { id: 'clearance-form', categoryId: 'hr', title: 'Обходной лист', icon: 'clipboard' }, { id: 'hr-transfer', categoryId: 'hr', title: 'на перемещение сотрудников', icon: 'send' }, // Склад / Снабжение diff --git a/src/webparts/requestPortal/features/approve/ApproveFeature.tsx b/src/webparts/requestPortal/features/approve/ApproveFeature.tsx index 6ca8103..2f674f8 100644 --- a/src/webparts/requestPortal/features/approve/ApproveFeature.tsx +++ b/src/webparts/requestPortal/features/approve/ApproveFeature.tsx @@ -20,7 +20,7 @@ const ApproveFeature: React.FC = ({ context, formKey, item
{ViewComponent ? ( Загрузка…
}> - + ) : (
Для формы «{formKey}» режим согласования пока не настроен.
diff --git a/src/webparts/requestPortal/forms/Clearance/ClearanceForm.module.scss b/src/webparts/requestPortal/forms/Clearance/ClearanceForm.module.scss new file mode 100644 index 0000000..e1ef36d --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/ClearanceForm.module.scss @@ -0,0 +1,20 @@ +@import '../../../../styles/design-tokens'; + +.back { + display: inline-flex; + align-items: center; + gap: $sp-2; + margin-bottom: $sp-4; + padding: 0; + border: 0; + background: transparent; + color: $navy-700; + font-family: $font-sans; + font-size: $fs-sm; + cursor: pointer; + transition: color .18s; + + &:hover { + color: $gold-600; + } +} diff --git a/src/webparts/requestPortal/forms/Clearance/ClearanceForm.tsx b/src/webparts/requestPortal/forms/Clearance/ClearanceForm.tsx new file mode 100644 index 0000000..f21246c --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/ClearanceForm.tsx @@ -0,0 +1,223 @@ +import * as React from 'react'; +import { useState, useMemo, useCallback, useEffect } from 'react'; +import styles from './ClearanceForm.module.scss'; +import { + Alert, + FormCard, + InitiatorLine, + FormModeProvider, + Icon, + RequestNumber, + SubmitActions, + useRequestSubmit +} from '../../shared/ui'; +import { + ApprovalMatrix, + StatusBadge, + useMatrixEditor, + getPlannedMatrix, + notifyActiveStep, + useRequestTypeId, + REQUEST_STATUS, + isEditableStatus, + IApproverToWrite, + findUnassignedStep +} from '../../shared/approval'; +import { IRequestFormProps } from '../../models/types'; +import { IFormData, EMPTY_DATA, SetField } from './model'; +import { saveClearanceRequest, clearanceApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; +import { loadClearanceRequest } from './load'; +import { RequestSection } from './sections/RequestSection'; + +const ClearanceForm: React.FC = ({ requestType, context, onCancel, onSaved, editId }) => { + const [data, setData] = useState(EMPTY_DATA); + const [requestId, setRequestId] = useState(editId); + const [requestStatus, setRequestStatus] = useState(undefined); + const [requestCode, setRequestCode] = useState(undefined); + const submit = useRequestSubmit(); + + const readOnly = !isEditableStatus(requestStatus); + const set: SetField = (key, value) => setData((prev) => ({ ...prev, [key]: value })); + + const approval = useMemo(() => clearanceApprovalService(), []); + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; + + // Открыта СВОЯ заявка на редактирование (черновик/возврат) — подгружаем содержимое и статус. + useEffect(() => { + if (!editId) { + return; + } + let alive = true; + loadClearanceRequest(editId) + .then((loaded) => { + if (alive) { + setData(loaded.data); + setRequestStatus(loaded.status); + setRequestCode(loaded.code || undefined); + } + }) + .catch((e) => { + if (alive) { + submit.setError(`Не удалось загрузить заявку №${editId}: ${(e as Error).message}`); + } + }); + return () => { + alive = false; + }; + // eslint-disable-next-line react-hooks/exhaustive-deps + }, [editId]); + + // Выбор увольняемого сотрудника подтягивает его текущую должность — это только + // отправная точка для поля «Должность» (её можно поправить вручную, например + // если человек занимал руководящую должность до перевода), а не жёсткая связь. + const handleEmployeeChange = (login: string): void => { + setData((prev) => ({ ...prev, Employee: login })); + if (!login) { + return; + } + approval + .getJobTitle(login) + .then((title) => setData((prev) => ({ ...prev, Position: title || prev.Position }))) + .catch(() => undefined); + }; + + // Планируемый маршрут резолвится по e-mail УВОЛЬНЯЕМОГО (data.Employee, не + // текущего пользователя — форму подаёт HR, а не сам увольняемый), а условие + // «IsManager» на бэкенде вычисляется из поля Position — передаём его в + // fieldValues, иначе шаг с этим условием получит предупреждение «должность не + // передана» (см. TemplateRouteService.ResolveAsync в approval-route). + const loadMatrix = useCallback( + () => + editId + ? approval.getMatrixForRequest(editId) + : getPlannedMatrix(context, requestTypeId as number, data.Employee, approval, 'Approval', null, [ + { fieldName: 'Position', value: data.Position } + ]), + [context, approval, editId, data.Employee, data.Position, requestTypeId] + ); + const matrix = useMatrixEditor( + loadMatrix, + `${data.Employee}-${editId ?? 'new'}-${data.Position}`, + !readOnly, + !!editId || (!!data.Employee && !!requestTypeId), + (login) => approval.getJobTitle(login) + ); + + const isValid = !!(data.Employee && data.Position.trim() && data.TerminationDate); + const validate = (): string | undefined => { + if (!isValid) return 'Заполните обязательные поля: сотрудник, должность, дата расторжения договора.'; + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; + + const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => + rows.map((r) => ({ + email: r.email, + position: r.position, + allowReassign: r.allowReassign, + order: r.order, + // Транзитом из маршрута в строку списка: наследование бэкенд применил + // при resolve, форма только переносит значения (см. IMatrixRow.slaHours). + slaHours: r.slaHours, + askOverdueReason: r.askOverdueReason + })); + + const save = (status: string): (() => Promise) => async (): Promise => { + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } + const { id, code } = await saveClearanceRequest(requestId, rtId, data, toApprovers(matrix.rows), status); + setRequestId(id); + setRequestStatus(status); + if (code) { + setRequestCode(code); + } + if (onSaved) { + onSaved(); + } + // Отправка на согласование — уведомляем первый активный шаг (поток оповещения). + if (status === REQUEST_STATUS.onApproval) { + await notifyActiveStep(context, approval, id, { + requestTypeId: rtId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE + }); + } + }; + + // На отправку (в отличие от черновика) маршрут должен быть полностью + // укомплектован — иначе шаг без назначенного человека молча пропадёт при записи. + const validateSubmit = (): string | undefined => { + const formError = validate(); + if (formError) return formError; + if (matrix.error) { + return `Не удалось определить маршрут согласования: ${matrix.error}. Попробуйте ещё раз или обратитесь к администратору.`; + } + const unassigned = findUnassignedStep(matrix.rows); + if (unassigned) { + return `Шаг ${unassigned.order} маршрута согласования («${unassigned.position || 'без должности'}») без назначенного человека — выберите согласующего или удалите шаг.`; + } + return undefined; + }; + + const handleSaveDraft = (): Promise => submit.run('draft', save(REQUEST_STATUS.draft), validate); + const handleSubmit = (): Promise => submit.run('submit', save(REQUEST_STATUS.onApproval), validateSubmit); + + const sectionProps = { data, set, context, onEmployeeChange: handleEmployeeChange }; + + return ( +
+ + + } + headerRight={ + requestId || requestStatus ? ( + <> + {requestStatus && } + {requestCode && } + + ) : undefined + } + > + {submit.savedKind && !submit.error && ( + + {submit.savedKind === 'draft' ? 'Черновик сохранён.' : 'Заявка отправлена на согласование.'} + + )} + {submit.error && {submit.error}} + + + + + + approval.searchUsers(q)} /> + + + +
+ ); +}; + +export default ClearanceForm; diff --git a/src/webparts/requestPortal/forms/Clearance/ClearanceView.module.scss b/src/webparts/requestPortal/forms/Clearance/ClearanceView.module.scss new file mode 100644 index 0000000..6a27763 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/ClearanceView.module.scss @@ -0,0 +1,14 @@ +@import '../../../../styles/design-tokens'; + +.state { + padding: $sp-8; + color: $gray-500; + font-size: $fs-sm; + text-align: center; +} + +.toolbar { + display: flex; + justify-content: flex-end; + margin-bottom: $sp-4; +} diff --git a/src/webparts/requestPortal/forms/Clearance/ClearanceView.tsx b/src/webparts/requestPortal/forms/Clearance/ClearanceView.tsx new file mode 100644 index 0000000..22657de --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/ClearanceView.tsx @@ -0,0 +1,143 @@ +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 { + ApproverService, + DecisionPanel, + StatusBadge, + ApprovalMatrix, + useMatrixEditor, + notifyActiveStep, + useRequestTypeId +} 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 { RequestSection } from './sections/RequestSection'; + +interface IClearanceViewProps { + context: WebPartContext; + itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; +} + +// В режиме чтения поля не редактируются, поэтому обновление — пустышка. +const NOOP_SET: SetField = () => undefined; + +// Обходной лист только для просмотра (режим согласующего). Переиспользует ту +// же секцию, что и форма подачи; редактирование выключено через +// — поля сами становятся неизменяемыми. +const ClearanceView: React.FC = ({ context, itemId, formKey }) => { + const [data, setData] = useState(EMPTY_DATA); + const [status, setStatus] = useState(''); + const [code, setCode] = useState(''); + const [loading, setLoading] = useState(true); + const [error, setError] = useState(null); + + const approval = useMemo(() => clearanceApprovalService(), []); + const { id: requestTypeId } = useRequestTypeId(context, formKey); + const approverService = useMemo(() => new ApproverService(approval, DATA_LIST_TITLE), [approval]); + + // Матрица маршрута из фактических строк заявки (read-only). decisionSeq в + // reloadKey: после решения строки на сервере меняются — перечитываем матрицу. + const [decisionSeq, setDecisionSeq] = useState(0); + const loadApprovers = useCallback(() => approval.getMatrixForRequest(itemId), [approval, itemId]); + const approvers = useMatrixEditor(loadApprovers, `a-${itemId}-${decisionSeq}`, false, true, (l) => + approval.getJobTitle(l) + ); + + useEffect(() => { + let alive = true; + setLoading(true); + loadClearanceRequest(itemId) + .then((loaded) => { + if (alive) { + setData(loaded.data); + setStatus(loaded.status); + setCode(loaded.code); + setError(null); + setLoading(false); + } + }) + .catch((e) => { + if (alive) { + setError(`Не удалось загрузить заявку №${itemId}: ${(e as Error).message}`); + setLoading(false); + } + }); + return () => { + alive = false; + }; + }, [itemId]); + + if (loading) { + return
Загрузка заявки…
; + } + if (error) { + return ( +
+ {error} +
+ ); + } + + const sectionProps = { data, set: NOOP_SET, context }; + + const notify = (): void => { + // requestTypeId ещё не резолвился — решение уже применилось (строка списка + // обновлена), просто нечего уведомлять на следующем шаге; не проваливаем + // весь UI ради необязательного оповещения. + if (!requestTypeId) return; + notifyActiveStep(context, approval, itemId, { + requestTypeId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE + }).catch(() => undefined); + }; + + return ( +
+
+ +
+ +
+ } + headerRight={status ? : undefined} + > + + + + + + approval.searchUsers(q)} /> +
+ + {/* Панель решения — ВНЕ печати и FormModeProvider (комментарий редактируемый). */} +
+ approval.searchUsers(q)} + onDecided={(newStatus) => { + setStatus(newStatus); + setDecisionSeq((n) => n + 1); + notify(); + }} + onDelegated={() => { + setDecisionSeq((n) => n + 1); + notify(); + }} + /> +
+
+ ); +}; + +export default ClearanceView; diff --git a/src/webparts/requestPortal/forms/Clearance/README.md b/src/webparts/requestPortal/forms/Clearance/README.md new file mode 100644 index 0000000..56a1f96 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/README.md @@ -0,0 +1,48 @@ +# Обходной лист (clearance-form) + +**Назначение:** оформление увольнения — обход подразделений (склад, СКУД, обучение) с отметками и согласованием. +**Категория:** Отдел кадров +**Списки SP:** «Обходной лист — данные» / «Обходной лист — согласование» +**RequestTypeId (внешний API маршрутов):** не хардкодится — резолвится в рантайме по +`pageUrl` («clearance-form») через `useRequestTypeId` (см. «Формы больше не хардкодят +RequestTypeId» в `docs/forms.md`). Тип уже зарегистрирован в админке approval-route +(список данных «Обходной лист — данные», список SharePoint `/sites/requests/Lists/List10`). + +## Маршрут +- Фазы: только согласование. Фазы исполнения нет — после согласования заявка + просто получает статус «Согласовано» (как у «Заявки на отпуск»). +- **Матрица редактируема заявителем** при новой заявке / черновике / возврате: + `useMatrixEditor(editable = !readOnly)`. Заявитель (HR) может добавлять/удалять/ + переставлять согласующих поверх планируемого маршрута. +- Условие `IsManager` в маршруте (если настроено в админке) резолвится по полю + **Position ЭТОЙ ЗАЯВКИ** — «должность» увольняемого сотрудника, а не должность + того, кто заполняет форму. Поэтому `Position` передаётся в `fieldValues` при + планировании маршрута (`getPlannedMatrix(..., [{ fieldName: 'Position', ... }])`). + Если поле пустое, шаг с этим условием попадёт в маршрут с предупреждением + «должность не передана» (см. approval-route `TemplateRouteService.ResolveAsync`). + +## Особое поведение +- **Заявку подаёт не сам увольняемый, а HR/руководитель — за другого сотрудника.** + Поле «Сотрудник» — People-picker (колонка User `Employee`), содержит e-mail + **увольняемого**. Планируемый маршрут резолвится по этому e-mail + (`getPlannedMatrix(..., data.Employee, ...)`), а не по e-mail текущего + пользователя — как в «Заявке на отпуск». +- **Автоподстановка должности при выборе сотрудника** — только отправная точка: + подтягивается текущая должность из профиля (`SPS-JobTitle` через + `approval.getJobTitle`), но её можно поправить вручную (например, если + человек занимал руководящую позицию до перевода на другую). +- **Да/нет-поля** («Парковочная карта», «Карта СКУД») — `Toggle`, колонки + `ParkingCard`/`AccessCard` (Boolean). +- **Сумма обучения** — `MoneyInput` (разделитель тысяч в UI, в списке — «сырое» + число), колонка `TrainingAmount` (Number, 2 знака), поле необязательное. +- **Дата расторжения договора** — обычный `input type=date`, без вычислений + (в отличие от `EndDate` в «Заявке на отпуск», которая считается). +- **Заголовок элемента** (`Title`): `Обходной лист — {ФИО} — дд.мм.гггг`. + +## Секции формы +RequestSection (сотрудник · должность · подразделение · дата расторжения · сумма обучения · парковочная карта · карта СКУД) + +## Точки входа +- rptype: `clearance-form` +- Режим согласующего (просмотр): есть (`ClearanceView`) +- «Создать на основе»: нет diff --git a/src/webparts/requestPortal/forms/Clearance/load.ts b/src/webparts/requestPortal/forms/Clearance/load.ts new file mode 100644 index 0000000..c7b8c14 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/load.ts @@ -0,0 +1,56 @@ +import { getSP } from '../../shared/pnp/pnpjs'; +import { IFormData, EMPTY_DATA } from './model'; + +const LIST_TITLE = 'Обходной лист — данные'; + +export interface ILoadedRequest { + data: IFormData; + /** Ф.И.О. увольняемого сотрудника из поля Employee (для заголовка режима чтения). */ + employeeName: string; + status: string; + /** Уникальный код заявки; '' — если запись создана до появления поля. */ + code: string; +} + +/** Прочитать обходной лист по id: модель формы + статус (правка / просмотр). */ +export async function loadClearanceRequest(itemId: number): Promise { + const item = (await getSP() + .web.lists.getByTitle(LIST_TITLE) + .items.getById(itemId) + .select( + 'Id', + 'Position', + 'Department', + 'TerminationDate', + 'ParkingCard', + 'AccessCard', + 'TrainingAmount', + 'RequestStatus', + 'RequestUniqueCode', + 'Employee/Title', + 'Employee/EMail' + ) + .expand('Employee')()) as Record; + + const employee = item.Employee as { Title?: string; EMail?: string } | undefined; + const termination = (item.TerminationDate as string) || ''; + const amount = item.TrainingAmount; + + const data: IFormData = { + ...EMPTY_DATA, + Employee: employee?.EMail || '', + Position: (item.Position as string) || '', + Department: (item.Department as string) || '', + TerminationDate: termination.slice(0, 10), // ISO → YYYY-MM-DD для input type=date + ParkingCard: !!item.ParkingCard, + AccessCard: !!item.AccessCard, + TrainingAmount: amount === null || amount === undefined ? '' : String(amount) + }; + + return { + data, + employeeName: employee?.Title || '', + status: (item.RequestStatus as string) || '', + code: (item.RequestUniqueCode as string) || '' + }; +} diff --git a/src/webparts/requestPortal/forms/Clearance/model.ts b/src/webparts/requestPortal/forms/Clearance/model.ts new file mode 100644 index 0000000..68bbff0 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/model.ts @@ -0,0 +1,49 @@ +import { WebPartContext } from '@microsoft/sp-webpart-base'; + +/** + * Данные формы «Обходной лист». Ключи по возможности = internalName колонок + * SP-списка «Обходной лист — данные». Employee хранит e-mail УВОЛЬНЯЕМОГО + * сотрудника (то, что отдаёт PeopleField) — заявку подаёт HR/руководитель, + * а не сам увольняемый, поэтому и маршрут согласования резолвится по email + * из этого поля, а не по email инициатора (как в «Заявке на отпуск»). + */ +export interface IFormData { + /** E-mail увольняемого сотрудника (People). Пусто = не выбран. */ + Employee: string; + Position: string; + Department: string; + /** ISO YYYY-MM-DD (input type=date). */ + TerminationDate: string; + ParkingCard: boolean; + AccessCard: boolean; + /** Сумма обучения — «сырые» цифры без пробелов (см. MoneyInput). Пусто = не указана. */ + TrainingAmount: string; +} + +/** Обновление одного поля формы с сохранением типа значения. */ +export type SetField = (key: K, value: IFormData[K]) => void; + +/** Пропсы секции формы: срез данных + колбэк обновления + контекст (people picker). */ +export interface ISectionProps { + data: IFormData; + set: SetField; + context: WebPartContext; + /** + * Выбор сотрудника подтягивает его должность — поэтому у поля «Сотрудник» + * отдельный обработчик. Если не передан (режим чтения) — правка через set. + */ + onEmployeeChange?: (login: string) => void; +} + +export const EMPTY_DATA: IFormData = { + Employee: '', + Position: '', + Department: '', + TerminationDate: '', + ParkingCard: false, + AccessCard: false, + TrainingAmount: '' +}; + +/** Плейсхолдер для поля выбора сотрудника. */ +export const EMPLOYEE_PLACEHOLDER = 'Начните вводить Ф.И.О. увольняемого сотрудника…'; diff --git a/src/webparts/requestPortal/forms/Clearance/sections/RequestSection.tsx b/src/webparts/requestPortal/forms/Clearance/sections/RequestSection.tsx new file mode 100644 index 0000000..3ce6e36 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/sections/RequestSection.tsx @@ -0,0 +1,40 @@ +import * as React from 'react'; +import { CollapseSection, FormField, TextInput, MoneyInput, Toggle, PeopleField } from '../../../shared/ui'; +import { ISectionProps, EMPLOYEE_PLACEHOLDER } from '../model'; + +/** Единственная секция формы «Обходной лист»: увольняемый, должность, подразделение, даты, карты, обучение. */ +export const RequestSection: React.FC = ({ data, set, context, onEmployeeChange }) => ( + + + set('Employee', v))} + placeholder={EMPLOYEE_PLACEHOLDER} + /> + + + set('Position', v)} /> + + + set('Department', v)} /> + + + set('TerminationDate', v)} + /> + + + set('TrainingAmount', v)} /> + + + set('ParkingCard', v)} onLabel="Да" /> + + + set('AccessCard', v)} onLabel="Да" /> + + +); diff --git a/src/webparts/requestPortal/forms/Clearance/submit.ts b/src/webparts/requestPortal/forms/Clearance/submit.ts new file mode 100644 index 0000000..8b0bca6 --- /dev/null +++ b/src/webparts/requestPortal/forms/Clearance/submit.ts @@ -0,0 +1,84 @@ +import { getSP } from '../../shared/pnp/pnpjs'; +import { ApprovalListService, IApproverToWrite, writeRoute } from '../../shared/approval'; +import { buildRequestUniqueCode } from '../../shared/requestCode'; +import { IFormData } from './model'; + +export const DATA_LIST_TITLE = 'Обходной лист — данные'; +export const APPROVAL_LIST_TITLE = 'Обходной лист — согласование'; + +/** Экземпляр сервиса списка согласования этой формы. */ +export function clearanceApprovalService(): ApprovalListService { + return new ApprovalListService(APPROVAL_LIST_TITLE); +} + +/** 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}`; + return getSP().web.ensureUser(full); +} + +function pad2(n: number): string { + return n < 10 ? `0${n}` : `${n}`; +} + +/** Осмысленный заголовок элемента (системная колонка Title). */ +function buildTitle(employeeName: string): string { + const now = new Date(); + return `Обходной лист — ${employeeName} — ${pad2(now.getDate())}.${pad2(now.getMonth() + 1)}.${now.getFullYear()}`; +} + +/** Поля элемента списка данных из модели формы (+ Employee по Id). */ +function buildItem(data: IFormData, status: string, employeeId: number, employeeName: string): Record { + const amount = Number(data.TrainingAmount); + return { + Title: buildTitle(employeeName), + RequestStatus: status, + EmployeeId: employeeId, + Position: data.Position || null, + Department: data.Department || null, + TerminationDate: data.TerminationDate || null, + ParkingCard: data.ParkingCard, + AccessCard: data.AccessCard, + TrainingAmount: Number.isFinite(amount) && data.TrainingAmount.trim() !== '' ? amount : null + }; +} + +export interface ISaveResult { + id: number; + /** Задан только при создании новой заявки (существующий код не меняется). */ + code?: string; +} + +/** + * Сохранить обходной лист: содержимое + маршрут согласования. + * existingId задан → обновляем ту же запись (черновик/доработка), иначе создаём новую. + * Фазы исполнения у обходного листа нет — writeRoute вызывается с пустым списком исполнителей. + * Возвращает Id заявки (+ код при создании). + */ +export async function saveClearanceRequest( + existingId: number | undefined, + requestTypeId: number, + data: IFormData, + approvers: IApproverToWrite[], + status: string +): Promise { + const list = getSP().web.lists.getByTitle(DATA_LIST_TITLE); + const employee = await ensureEmployee(data.Employee); + const item = buildItem(data, status, employee.Id, employee.Title || data.Employee); + + let id: number; + let code: string | undefined; + if (existingId) { + await list.items.getById(existingId).update(item); + id = existingId; + } else { + const added = await list.items.add(item); + id = (added as { Id: number }).Id; + // Код известен только после создания элемента (нужен его Id). + code = buildRequestUniqueCode(requestTypeId, id, new Date()); + await list.items.getById(id).update({ RequestUniqueCode: code }); + } + + await writeRoute(clearanceApprovalService(), id, approvers, [], status); + return { id, code }; +} diff --git a/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireForm.tsx b/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireForm.tsx index b82f3aa..6eae656 100644 --- a/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireForm.tsx +++ b/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireForm.tsx @@ -21,6 +21,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -35,13 +36,7 @@ import { } from '../../shared/attachments/attachments.service'; import { IRequestFormProps } from '../../models/types'; import { IFormData, EMPTY_DATA, SetField } from './model'; -import { - saveHireRequest, - hireApprovalService, - HIRE_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { saveHireRequest, hireApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { loadHireRequest } from './load'; import { ApplicationSection } from './sections/ApplicationSection'; import { CandidateSection } from './sections/CandidateSection'; @@ -95,20 +90,24 @@ const RecruitingHireForm: React.FC = ({ requestType, context, // затёрло бы правки, сделанные при первом сохранении. const approval = useMemo(() => hireApprovalService(), []); const userEmail = context.pageContext.user.email; + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; const loadMatrix = useCallback( () => editId ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase !== APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, HIRE_REQUEST_TYPE_ID, userEmail, approval, 'Approval'), - [context, userEmail, approval, editId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Approval'), + [context, userEmail, approval, editId, requestTypeId] ); const matrix = useMatrixEditor( loadMatrix, `${userEmail}-${editId ?? 'new'}`, !readOnly, - true, + !!editId || !!requestTypeId, (login) => approval.getJobTitle(login) ); @@ -119,20 +118,24 @@ const RecruitingHireForm: React.FC = ({ requestType, context, ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase === APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, HIRE_REQUEST_TYPE_ID, userEmail, approval, 'Execution'), - [context, userEmail, approval, editId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Execution'), + [context, userEmail, approval, editId, requestTypeId] ); const executors = useMatrixEditor( loadExecutors, `${userEmail}-${editId ?? 'new'}`, false, - true, + !!editId || !!requestTypeId, (login) => approval.getJobTitle(login) ); const isValid = data.CandidateName.trim().length > 0 && data.Position.trim().length > 0; - const validate = (): string | undefined => - isValid ? undefined : 'Заполните обязательные поля: Ф.И.О. кандидата, Должность.'; + const validate = (): string | undefined => { + if (!isValid) return 'Заполните обязательные поля: Ф.И.О. кандидата, Должность.'; + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => // order обязателен: параллельные шаги делят один номер, по позиции в @@ -159,8 +162,16 @@ const RecruitingHireForm: React.FC = ({ requestType, context, }; const save = (status: string): (() => Promise) => async (): Promise => { + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } const { id, code } = await saveHireRequest( requestId, + rtId, data, toApprovers(matrix.rows), toApprovers(executors.rows), @@ -182,7 +193,7 @@ const RecruitingHireForm: React.FC = ({ requestType, context, // Отправка на согласование — уведомляем первый активный шаг (поток оповещения). if (status === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: HIRE_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE, }); diff --git a/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireView.tsx b/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireView.tsx index 4bc3818..eabf02f 100644 --- a/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireView.tsx +++ b/src/webparts/requestPortal/forms/RecruitingHire/RecruitingHireView.tsx @@ -10,6 +10,7 @@ import { ApprovalMatrix, useMatrixEditor, notifyActiveStep, + useRequestTypeId, APPROVAL_PHASE } from '../../shared/approval'; import { navigate } from '../../shared/routing'; @@ -17,7 +18,7 @@ import { getAttachments, IAttachmentInfo } from '../../shared/attachments/attach import { IFormData, EMPTY_DATA, SetField } from './model'; import { loadHireRequest } from './load'; import { confirmDiscardFormChanges } from '../../shared/unsavedChanges'; -import { hireApprovalService, HIRE_REQUEST_TYPE_ID, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; +import { hireApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { ApplicationSection } from './sections/ApplicationSection'; import { CandidateSection } from './sections/CandidateSection'; import { PpeSection } from './sections/PpeSection'; @@ -26,6 +27,8 @@ import { ConditionsSection } from './sections/ConditionsSection'; interface IRecruitingHireViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } // Ключ формы «Организация рабочего места» в реестре (data/requests) — цель @@ -38,7 +41,7 @@ const NOOP_SET: SetField = () => undefined; // Заявка «на прием» только для просмотра (режим согласующего). // Переиспользует те же секции, что и форма подачи; редактирование выключено // через — поля сами становятся неизменяемыми. -const RecruitingHireView: React.FC = ({ context, itemId }) => { +const RecruitingHireView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -46,6 +49,8 @@ const RecruitingHireView: React.FC = ({ context, itemI const [loading, setLoading] = useState(true); const [error, setError] = useState(null); + const { id: requestTypeId } = useRequestTypeId(context, formKey); + // Сервис решений согласующего: список согласования + список данных этой формы. const approverService = useMemo( () => new ApproverService(hireApprovalService(), DATA_LIST_TITLE), @@ -120,6 +125,15 @@ const RecruitingHireView: React.FC = ({ context, itemI const sectionProps = { data, set: NOOP_SET, context }; + const notify = (): void => { + if (!requestTypeId) return; + notifyActiveStep(context, approval, itemId, { + requestTypeId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE, + }).catch(() => undefined); + }; + return (
@@ -187,21 +201,13 @@ const RecruitingHireView: React.FC = ({ context, itemI setStatus(newStatus); setDecisionSeq((n) => n + 1); // Очередь сдвинулась — уведомляем следующий активный шаг. - notifyActiveStep(context, approval, itemId, { - requestTypeId: HIRE_REQUEST_TYPE_ID, - dataListName: DATA_LIST_TITLE, - approvalListName: APPROVAL_LIST_TITLE, - }).catch(() => undefined); + notify(); }} onDelegated={() => { // Шаг переназначен: активной стала строка делегата — обновляем // матрицу и уведомляем нового активного участника. setDecisionSeq((n) => n + 1); - notifyActiveStep(context, approval, itemId, { - requestTypeId: HIRE_REQUEST_TYPE_ID, - dataListName: DATA_LIST_TITLE, - approvalListName: APPROVAL_LIST_TITLE, - }).catch(() => undefined); + notify(); }} />
diff --git a/src/webparts/requestPortal/forms/RecruitingHire/submit.ts b/src/webparts/requestPortal/forms/RecruitingHire/submit.ts index 99cffe0..79fdebc 100644 --- a/src/webparts/requestPortal/forms/RecruitingHire/submit.ts +++ b/src/webparts/requestPortal/forms/RecruitingHire/submit.ts @@ -5,9 +5,6 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'Прием на работу — данные'; export const APPROVAL_LIST_TITLE = 'Прием на работу — согласование'; -// Тип заявки «Заявка на прием» во внешнем approval-route API -// (GET /api/request-types → id 5). Захардкожен, как «отпуск = 1» в pto-tracker. -export const HIRE_REQUEST_TYPE_ID = 5; // Числовые поля: пустая строка → null, иначе Number(). const NUMBER_FIELDS: (keyof IFormData)[] = ['PpeHeight', 'PpeShoeSize', 'SalaryProbation', 'SalaryAfterProbation']; @@ -53,6 +50,7 @@ export interface ISaveResult { */ export async function saveHireRequest( existingId: number | undefined, + requestTypeId: number, data: IFormData, approvers: IApproverToWrite[], executors: IApproverToWrite[], @@ -70,7 +68,7 @@ export async function saveHireRequest( const added = await list.items.add(item); id = (added as { Id: number }).Id; // Код известен только после создания элемента (нужен его Id). - code = buildRequestUniqueCode(HIRE_REQUEST_TYPE_ID, id, new Date()); + code = buildRequestUniqueCode(requestTypeId, id, new Date()); await list.items.getById(id).update({ RequestUniqueCode: code }); } diff --git a/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionForm.tsx b/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionForm.tsx index 5470405..9f24af6 100644 --- a/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionForm.tsx +++ b/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionForm.tsx @@ -21,6 +21,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -34,13 +35,7 @@ import { } from '../../shared/attachments/attachments.service'; import { IRequestFormProps } from '../../models/types'; import { IFormData, EMPTY_DATA, SetField } from './model'; -import { - saveSelectionRequest, - selectionApprovalService, - RECRUITING_SELECTION_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { saveSelectionRequest, selectionApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { loadSelectionRequest } from './load'; import { ApplicationSection } from './sections/ApplicationSection'; import { DepartmentSection } from './sections/DepartmentSection'; @@ -92,26 +87,32 @@ const RecruitingSelectionForm: React.FC = ({ requestType, con // затереть правки инициатора при первом сохранении. const approval = useMemo(() => selectionApprovalService(), []); const userEmail = context.pageContext.user.email; + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; const loadMatrix = useCallback( () => editId ? approval.getMatrixForRequest(editId) - : getPlannedMatrix(context, RECRUITING_SELECTION_REQUEST_TYPE_ID, userEmail, approval, 'Approval'), - [context, userEmail, approval, editId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Approval'), + [context, userEmail, approval, editId, requestTypeId] ); const matrix = useMatrixEditor( loadMatrix, `${userEmail}-${editId ?? 'new'}`, !readOnly, - true, + !!editId || !!requestTypeId, (login) => approval.getJobTitle(login) ); const isValid = !!(data.Responsible && data.DepartmentName.trim() && data.Position.trim()); - const validate = (): string | undefined => - isValid - ? undefined - : 'Заполните обязательные поля: ответственный за подачу, название отдела, должность.'; + const validate = (): string | undefined => { + if (!isValid) return 'Заполните обязательные поля: ответственный за подачу, название отдела, должность.'; + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => rows.map((r) => ({ @@ -135,7 +136,14 @@ const RecruitingSelectionForm: React.FC = ({ requestType, con }; const save = (status: string): (() => Promise) => async (): Promise => { - const { id, code } = await saveSelectionRequest(requestId, data, toApprovers(matrix.rows), status); + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } + const { id, code } = await saveSelectionRequest(requestId, rtId, data, toApprovers(matrix.rows), status); setRequestId(id); setRequestStatus(status); if (code) { @@ -152,7 +160,7 @@ const RecruitingSelectionForm: React.FC = ({ requestType, con // Отправка на согласование — уведомляем первый активный шаг (поток оповещения). if (status === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: RECRUITING_SELECTION_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }); diff --git a/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionView.tsx b/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionView.tsx index f0357dc..81ca4e0 100644 --- a/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionView.tsx +++ b/src/webparts/requestPortal/forms/RecruitingSelection/RecruitingSelectionView.tsx @@ -9,17 +9,13 @@ import { StatusBadge, ApprovalMatrix, useMatrixEditor, - notifyActiveStep + notifyActiveStep, + useRequestTypeId } from '../../shared/approval'; import { getAttachments, IAttachmentInfo } from '../../shared/attachments/attachments.service'; import { IFormData, EMPTY_DATA, SetField } from './model'; import { loadSelectionRequest } from './load'; -import { - selectionApprovalService, - RECRUITING_SELECTION_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { selectionApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { ApplicationSection } from './sections/ApplicationSection'; import { DepartmentSection } from './sections/DepartmentSection'; import { VacancySection } from './sections/VacancySection'; @@ -29,6 +25,8 @@ import { ConditionsSection } from './sections/ConditionsSection'; interface IRecruitingSelectionViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } // В режиме чтения поля не редактируются, поэтому обновление — пустышка. @@ -36,7 +34,7 @@ const NOOP_SET: SetField = () => undefined; // Заявка «на подбор» только для просмотра (режим согласующего). Переиспользует те же // секции, что и форма подачи; редактирование выключено через . -const RecruitingSelectionView: React.FC = ({ context, itemId }) => { +const RecruitingSelectionView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -45,6 +43,7 @@ const RecruitingSelectionView: React.FC = ({ cont const [error, setError] = useState(null); const approval = useMemo(() => selectionApprovalService(), []); + const { id: requestTypeId } = useRequestTypeId(context, formKey); const approverService = useMemo(() => new ApproverService(approval, DATA_LIST_TITLE), [approval]); // Матрица маршрута из фактических строк заявки (read-only). decisionSeq в @@ -94,8 +93,9 @@ const RecruitingSelectionView: React.FC = ({ cont const sectionProps = { data, set: NOOP_SET, context }; const notify = (): void => { + if (!requestTypeId) return; notifyActiveStep(context, approval, itemId, { - requestTypeId: RECRUITING_SELECTION_REQUEST_TYPE_ID, + requestTypeId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }).catch(() => undefined); diff --git a/src/webparts/requestPortal/forms/RecruitingSelection/submit.ts b/src/webparts/requestPortal/forms/RecruitingSelection/submit.ts index 26f63c5..59dc753 100644 --- a/src/webparts/requestPortal/forms/RecruitingSelection/submit.ts +++ b/src/webparts/requestPortal/forms/RecruitingSelection/submit.ts @@ -5,8 +5,6 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'Заявка на подбор — данные'; export const APPROVAL_LIST_TITLE = 'Заявка на подбор — согласование'; -// Тип заявки во внешнем approval-route API (GET /api/request-types → id 4). -export const RECRUITING_SELECTION_REQUEST_TYPE_ID = 4; // Поля-люди (один человек): логин → ensureUser → Id в колонку Id. const PERSON_FIELDS: (keyof IFormData)[] = ['Responsible', 'ProposedMentor', 'DirectManager']; @@ -75,6 +73,7 @@ export interface ISaveResult { */ export async function saveSelectionRequest( existingId: number | undefined, + requestTypeId: number, data: IFormData, approvers: IApproverToWrite[], status: string @@ -93,7 +92,7 @@ export async function saveSelectionRequest( const added = await list.items.add(item); id = (added as { Id: number }).Id; // Код известен только после создания элемента (нужен его Id). - code = buildRequestUniqueCode(RECRUITING_SELECTION_REQUEST_TYPE_ID, id, new Date()); + code = buildRequestUniqueCode(requestTypeId, id, new Date()); await list.items.getById(id).update({ RequestUniqueCode: code }); } diff --git a/src/webparts/requestPortal/forms/Ticket1C/Ticket1CForm.tsx b/src/webparts/requestPortal/forms/Ticket1C/Ticket1CForm.tsx index f3df323..a6a2693 100644 --- a/src/webparts/requestPortal/forms/Ticket1C/Ticket1CForm.tsx +++ b/src/webparts/requestPortal/forms/Ticket1C/Ticket1CForm.tsx @@ -21,6 +21,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -38,7 +39,6 @@ import { saveTicketRequest, setPlannerTaskId, ticket1cApprovalService, - TICKET1C_REQUEST_TYPE_ID, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; @@ -133,6 +133,10 @@ const Ticket1CForm: React.FC = ({ requestType, context, onCan const approval = useMemo(() => ticket1cApprovalService(), []); const userEmail = context.pageContext.user.email; + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; // Маршрут полностью и однозначно определяется видом работ — настроен в админке // маршрутов (условные шаги по WorkType, см. fieldValues), а не выбирается @@ -143,16 +147,16 @@ const Ticket1CForm: React.FC = ({ requestType, context, onCan () => editId ? approval.getMatrixForRequest(editId) - : getPlannedMatrix(context, TICKET1C_REQUEST_TYPE_ID, userEmail, approval, 'Approval', null, [ + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Approval', null, [ { fieldName: 'WorkType', value: data.WorkType } ]), - [context, userEmail, approval, editId, data.WorkType] + [context, userEmail, approval, editId, data.WorkType, requestTypeId] ); const matrix = useMatrixEditor( loadMatrix, `${userEmail}-${editId ?? 'new'}-${data.WorkType}`, false, - !!editId || !!data.WorkType, + !!editId || (!!data.WorkType && !!requestTypeId), (login) => approval.getJobTitle(login) ); @@ -163,10 +167,14 @@ const Ticket1CForm: React.FC = ({ requestType, context, onCan data.DB1C && data.Question.trim() ); - const validate = (): string | undefined => - isValid - ? undefined - : 'Заполните обязательные поля: наименование задачи, вид работ, организация, база 1С, вопрос.'; + const validate = (): string | undefined => { + if (!isValid) { + return 'Заполните обязательные поля: наименование задачи, вид работ, организация, база 1С, вопрос.'; + } + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => rows.map((r) => ({ @@ -190,6 +198,13 @@ const Ticket1CForm: React.FC = ({ requestType, context, onCan }; const save = (intendedStatus: string): (() => Promise) => async (): Promise => { + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } // Пустой маршрут — согласование не требуется по правилам этого вида работ. const hasApprovers = matrix.rows.length > 0; const finalStatus = @@ -220,7 +235,7 @@ const Ticket1CForm: React.FC = ({ requestType, context, onCan if (finalStatus === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: TICKET1C_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }); diff --git a/src/webparts/requestPortal/forms/Ticket1C/Ticket1CView.tsx b/src/webparts/requestPortal/forms/Ticket1C/Ticket1CView.tsx index 8ea7448..015be31 100644 --- a/src/webparts/requestPortal/forms/Ticket1C/Ticket1CView.tsx +++ b/src/webparts/requestPortal/forms/Ticket1C/Ticket1CView.tsx @@ -22,6 +22,7 @@ import { ApprovalMatrix, useMatrixEditor, notifyActiveStep, + useRequestTypeId, isEditableStatus, REQUEST_STATUS } from '../../shared/approval'; @@ -32,7 +33,6 @@ import { ticket1cApprovalService, getPlannerTaskId, setPlannerTaskId, - TICKET1C_REQUEST_TYPE_ID, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; @@ -44,6 +44,8 @@ import { ContactSection } from './sections/ContactSection'; interface ITicket1CViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } /** Текст карточки Planner из данных заявки; rejection — автор и причина отклонения, если есть. */ @@ -75,7 +77,7 @@ function buildPlannerNotes( } // Заявка в отдел 1С только для просмотра (режим согласующего). -const Ticket1CView: React.FC = ({ context, itemId }) => { +const Ticket1CView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -83,6 +85,7 @@ const Ticket1CView: React.FC = ({ context, itemId }) => { const [loading, setLoading] = useState(true); const [error, setError] = useState(null); + const { id: requestTypeId } = useRequestTypeId(context, formKey); const set: SetField = (key, value) => setData((prev) => ({ ...prev, [key]: value })); const approverService = useMemo( @@ -172,13 +175,22 @@ const Ticket1CView: React.FC = ({ context, itemId }) => { const sectionProps = { data, set, context }; + const notify = (): void => { + if (!requestTypeId) return; + notifyActiveStep(context, approval, itemId, { + requestTypeId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE + }).catch(() => undefined); + }; + return (
{isEditableStatus(status) && (
diff --git a/src/webparts/requestPortal/forms/Ticket1C/submit.ts b/src/webparts/requestPortal/forms/Ticket1C/submit.ts index 23e74e0..73a06ff 100644 --- a/src/webparts/requestPortal/forms/Ticket1C/submit.ts +++ b/src/webparts/requestPortal/forms/Ticket1C/submit.ts @@ -5,14 +5,9 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'Заявка в отдел 1С — данные'; export const APPROVAL_LIST_TITLE = 'Заявка в отдел 1С — согласование'; -// Тип заявки для кода заявки, резолва маршрута (resolveRoute) и уведомлений -// (notifyActiveStep) — подтверждён через GET /api/request-types: id 10, -// name "Заявка в отдел 1С" (не путать с «Маршрут #8» — это id самого маршрута -// в админке, отдельная сущность от id типа заявки). -export const TICKET1C_REQUEST_TYPE_ID = 10; // Префикс уникального кода заявки — буквенная метка «1С» вместо числового id -// типа (10). Только для отображения/идентификации; на резолв маршрута и -// уведомления (там нужен числовой TICKET1C_REQUEST_TYPE_ID) не влияет. +// типа заявки. Числовой RequestTypeId (для резолва маршрута/уведомлений) +// больше не хардкодится — см. useRequestTypeId в Form.tsx/View.tsx. export const TICKET1C_CODE_PREFIX = '1С'; /** Экземпляр сервиса списка согласования этой формы. */ diff --git a/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferForm.tsx b/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferForm.tsx index 3672f90..e59102c 100644 --- a/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferForm.tsx +++ b/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferForm.tsx @@ -21,6 +21,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -35,13 +36,7 @@ import { } from '../../shared/attachments/attachments.service'; import { IRequestFormProps } from '../../models/types'; import { IFormData, EMPTY_DATA, SetField } from './model'; -import { - saveTmcTransferRequest, - tmcTransferApprovalService, - TMC_TRANSFER_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { saveTmcTransferRequest, tmcTransferApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { loadTmcTransferRequest } from './load'; import { HeaderSection } from './sections/HeaderSection'; import { ItemsSection } from './sections/ItemsSection'; @@ -101,6 +96,10 @@ const TmcTransferForm: React.FC = ({ requestType, context, on const approval = useMemo(() => tmcTransferApprovalService(), []); const userEmail = context.pageContext.user.email; + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; // У новой заявки маршрут ПЛАНИРУЕТСЯ (resolve по внешнему API), у открытой на // редактирование — читается из уже записанных строк. // projectId = oid проекта-источника: по нему бэкенд подставляет согласующего @@ -110,18 +109,18 @@ const TmcTransferForm: React.FC = ({ requestType, context, on // Маршрут согласования зависит от проекта-источника («Откуда»): пока он не // выбран, resolve вернёт 404 — не грузим, показываем заглушку. У открытой на // редактирование заявки маршрут уже записан, источник не нужен. - const matrixReady = !!editId || !!projectId; + const matrixReady = !!editId || (!!projectId && !!requestTypeId); // Исполнитель определяется проектом-получателем («Куда»): показываем блок с // заглушкой, пока «Куда» не выбрано. Источник тоже нужен, иначе resolve 404. - const executorsReady = !!editId || (!!projectId && !!targetProjectId); + const executorsReady = !!editId || (!!projectId && !!targetProjectId && !!requestTypeId); const loadMatrix = useCallback( () => editId ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase !== APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, TMC_TRANSFER_REQUEST_TYPE_ID, userEmail, approval, 'Approval', null, undefined, projectId), - [context, userEmail, approval, editId, projectId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Approval', null, undefined, projectId), + [context, userEmail, approval, editId, projectId, requestTypeId] ); const matrix = useMatrixEditor(loadMatrix, `${userEmail}-${editId ?? 'new'}-${projectId}`, !readOnly, matrixReady, (login) => approval.getJobTitle(login) @@ -134,8 +133,8 @@ const TmcTransferForm: React.FC = ({ requestType, context, on ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase === APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, TMC_TRANSFER_REQUEST_TYPE_ID, userEmail, approval, 'Execution', null, undefined, projectId), - [context, userEmail, approval, editId, projectId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Execution', null, undefined, projectId), + [context, userEmail, approval, editId, projectId, requestTypeId] ); const executors = useMatrixEditor( loadExecutors, @@ -148,10 +147,14 @@ const TmcTransferForm: React.FC = ({ requestType, context, on const hasItems = data.Items.some((i) => i.name.trim().length > 0); const isValid = data.SourceProject.trim().length > 0 && data.TargetProject.trim().length > 0 && hasItems; - const validate = (): string | undefined => - isValid - ? undefined - : 'Заполните обязательные поля: проект-источник, проект-получатель и хотя бы одну позицию ТМЦ.'; + const validate = (): string | undefined => { + if (!isValid) { + return 'Заполните обязательные поля: проект-источник, проект-получатель и хотя бы одну позицию ТМЦ.'; + } + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => rows.map((r) => ({ @@ -172,8 +175,16 @@ const TmcTransferForm: React.FC = ({ requestType, context, on }; const save = (status: string): (() => Promise) => async (): Promise => { + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } const { id, code } = await saveTmcTransferRequest( requestId, + rtId, data, toApprovers(matrix.rows), toApprovers(executors.rows), @@ -194,7 +205,7 @@ const TmcTransferForm: React.FC = ({ requestType, context, on } if (status === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: TMC_TRANSFER_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }); diff --git a/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferView.tsx b/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferView.tsx index 45d6cd0..3456e50 100644 --- a/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferView.tsx +++ b/src/webparts/requestPortal/forms/TmcTransfer/TmcTransferView.tsx @@ -22,28 +22,26 @@ import { ApprovalMatrix, useMatrixEditor, notifyActiveStep, + useRequestTypeId, isEditableStatus, APPROVAL_PHASE } from '../../shared/approval'; import { getAttachments, IAttachmentInfo } from '../../shared/attachments/attachments.service'; import { IFormData, EMPTY_DATA, SetField } from './model'; import { loadTmcTransferRequest } from './load'; -import { - tmcTransferApprovalService, - TMC_TRANSFER_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { tmcTransferApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { HeaderSection } from './sections/HeaderSection'; import { ItemsSection } from './sections/ItemsSection'; interface ITmcTransferViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } // Заявка «СЗ на перемещение ТМЦ» только для просмотра (режим согласующего). -const TmcTransferView: React.FC = ({ context, itemId }) => { +const TmcTransferView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -51,6 +49,7 @@ const TmcTransferView: React.FC = ({ context, itemId }) = const [loading, setLoading] = useState(true); const [error, setError] = useState(null); + const { id: requestTypeId } = useRequestTypeId(context, formKey); // В просмотре форма read-only, но секции ждут set — даём no-op. const set: SetField = () => undefined; @@ -120,13 +119,22 @@ const TmcTransferView: React.FC = ({ context, itemId }) = const sectionProps = { data, set, context }; + const notify = (): void => { + if (!requestTypeId) return; + notifyActiveStep(context, approval, itemId, { + requestTypeId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE + }).catch(() => undefined); + }; + return (
{isEditableStatus(status) && (
diff --git a/src/webparts/requestPortal/forms/TmcTransfer/submit.ts b/src/webparts/requestPortal/forms/TmcTransfer/submit.ts index 17a5eb7..894c116 100644 --- a/src/webparts/requestPortal/forms/TmcTransfer/submit.ts +++ b/src/webparts/requestPortal/forms/TmcTransfer/submit.ts @@ -6,9 +6,6 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'СЗ на перемещение ТМЦ — данные'; export const APPROVAL_LIST_TITLE = 'СЗ на перемещение ТМЦ — согласование'; -// Тип заявки во внешнем approval-route API — подтверждён через GET /api/request-types: -// id 12, name «Заявка на перемещение», pageUrl «wh-tmc-transfer», section «Снабжение». -export const TMC_TRANSFER_REQUEST_TYPE_ID = 12; /** Экземпляр сервиса списка согласования этой формы. */ export function tmcTransferApprovalService(): ApprovalListService { @@ -52,6 +49,7 @@ export interface ISaveResult { */ export async function saveTmcTransferRequest( existingId: number | undefined, + requestTypeId: number, data: IFormData, approvers: IApproverToWrite[], executors: IApproverToWrite[], @@ -69,7 +67,7 @@ export async function saveTmcTransferRequest( const added = await list.items.add(item); id = (added as { Id: number }).Id; // Код известен только после создания элемента (нужен его Id). - code = buildRequestUniqueCode(TMC_TRANSFER_REQUEST_TYPE_ID, id, new Date()); + code = buildRequestUniqueCode(requestTypeId, id, new Date()); await list.items.getById(id).update({ RequestUniqueCode: code }); } diff --git a/src/webparts/requestPortal/forms/Vacation/VacationForm.tsx b/src/webparts/requestPortal/forms/Vacation/VacationForm.tsx index c67e31c..0c5dac7 100644 --- a/src/webparts/requestPortal/forms/Vacation/VacationForm.tsx +++ b/src/webparts/requestPortal/forms/Vacation/VacationForm.tsx @@ -17,6 +17,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -24,13 +25,7 @@ import { } from '../../shared/approval'; import { IRequestFormProps } from '../../models/types'; import { IFormData, EMPTY_DATA, SetField } from './model'; -import { - saveVacationRequest, - vacationApprovalService, - VACATION_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { saveVacationRequest, vacationApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { loadVacationRequest } from './load'; import { RequestSection } from './sections/RequestSection'; @@ -45,6 +40,10 @@ const VacationForm: React.FC = ({ requestType, context, onCan const set: SetField = (key, value) => setData((prev) => ({ ...prev, [key]: value })); const approval = useMemo(() => vacationApprovalService(), []); + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; // Открыта СВОЯ заявка на редактирование (черновик/возврат) — подгружаем содержимое и статус. useEffect(() => { @@ -112,14 +111,14 @@ const VacationForm: React.FC = ({ requestType, context, onCan () => editId ? approval.getMatrixForRequest(editId) - : getPlannedMatrix(context, VACATION_REQUEST_TYPE_ID, data.Employee, approval, 'Approval'), - [context, approval, editId, data.Employee] + : getPlannedMatrix(context, requestTypeId as number, data.Employee, approval, 'Approval'), + [context, approval, editId, data.Employee, requestTypeId] ); const matrix = useMatrixEditor( loadMatrix, `${data.Employee}-${editId ?? 'new'}`, !readOnly, - !!editId || !!data.Employee, + !!editId || (!!data.Employee && !!requestTypeId), (login) => approval.getJobTitle(login) ); @@ -131,8 +130,12 @@ const VacationForm: React.FC = ({ requestType, context, onCan Number.isFinite(daysNum) && daysNum > 0 ); - const validate = (): string | undefined => - isValid ? undefined : 'Заполните обязательные поля: сотрудник, должность, дата начала, количество дней (> 0).'; + const validate = (): string | undefined => { + if (!isValid) return 'Заполните обязательные поля: сотрудник, должность, дата начала, количество дней (> 0).'; + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => rows.map((r) => ({ @@ -147,7 +150,14 @@ const VacationForm: React.FC = ({ requestType, context, onCan })); const save = (status: string): (() => Promise) => async (): Promise => { - const { id, code } = await saveVacationRequest(requestId, data, toApprovers(matrix.rows), status); + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } + const { id, code } = await saveVacationRequest(requestId, rtId, data, toApprovers(matrix.rows), status); setRequestId(id); setRequestStatus(status); if (code) { @@ -159,7 +169,7 @@ const VacationForm: React.FC = ({ requestType, context, onCan // Отправка на согласование — уведомляем первый активный шаг (поток оповещения). if (status === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: VACATION_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }); diff --git a/src/webparts/requestPortal/forms/Vacation/VacationView.tsx b/src/webparts/requestPortal/forms/Vacation/VacationView.tsx index 1ea6e7f..d498638 100644 --- a/src/webparts/requestPortal/forms/Vacation/VacationView.tsx +++ b/src/webparts/requestPortal/forms/Vacation/VacationView.tsx @@ -9,21 +9,19 @@ import { StatusBadge, ApprovalMatrix, useMatrixEditor, - notifyActiveStep + notifyActiveStep, + useRequestTypeId } from '../../shared/approval'; import { IFormData, EMPTY_DATA, SetField } from './model'; import { loadVacationRequest } from './load'; -import { - vacationApprovalService, - VACATION_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { vacationApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { RequestSection } from './sections/RequestSection'; interface IVacationViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } // В режиме чтения поля не редактируются, поэтому обновление — пустышка. @@ -32,7 +30,7 @@ const NOOP_SET: SetField = () => undefined; // Заявка на отпуск только для просмотра (режим согласующего). Переиспользует ту // же секцию, что и форма подачи; редактирование выключено через // — поля сами становятся неизменяемыми. -const VacationView: React.FC = ({ context, itemId }) => { +const VacationView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -40,6 +38,7 @@ const VacationView: React.FC = ({ context, itemId }) => { const [error, setError] = useState(null); const approval = useMemo(() => vacationApprovalService(), []); + const { id: requestTypeId } = useRequestTypeId(context, formKey); const approverService = useMemo(() => new ApproverService(approval, DATA_LIST_TITLE), [approval]); // Матрица маршрута из фактических строк заявки (read-only). decisionSeq в @@ -88,8 +87,12 @@ const VacationView: React.FC = ({ context, itemId }) => { const sectionProps = { data, set: NOOP_SET, context }; const notify = (): void => { + // requestTypeId ещё не резолвился — решение уже применилось (строка списка + // обновлена), просто нечего уведомлять на следующем шаге; не проваливаем + // весь UI ради необязательного оповещения. + if (!requestTypeId) return; notifyActiveStep(context, approval, itemId, { - requestTypeId: VACATION_REQUEST_TYPE_ID, + requestTypeId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE }).catch(() => undefined); diff --git a/src/webparts/requestPortal/forms/Vacation/submit.ts b/src/webparts/requestPortal/forms/Vacation/submit.ts index 75e6237..a6a5fff 100644 --- a/src/webparts/requestPortal/forms/Vacation/submit.ts +++ b/src/webparts/requestPortal/forms/Vacation/submit.ts @@ -5,11 +5,6 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'Заявка на отпуск — данные'; export const APPROVAL_LIST_TITLE = 'Заявка на отпуск — согласование'; -// Тип заявки «отпуск» во внешнем approval-route API (GET /api/request-types → id 11, -// name «Заявка на отпуск», pageUrl «hr-vacation»). Захардкожен, как -// HIRE_REQUEST_TYPE_ID=5 в RecruitingHire. NB: в старой web part pto-tracker было 1 — -// при заведении типа в админке портала он получил новый id 11, берём его. -export const VACATION_REQUEST_TYPE_ID = 11; /** Экземпляр сервиса списка согласования этой формы. */ export function vacationApprovalService(): ApprovalListService { @@ -67,6 +62,7 @@ export interface ISaveResult { */ export async function saveVacationRequest( existingId: number | undefined, + requestTypeId: number, data: IFormData, approvers: IApproverToWrite[], status: string @@ -84,7 +80,7 @@ export async function saveVacationRequest( const added = await list.items.add(item); id = (added as { Id: number }).Id; // Код известен только после создания элемента (нужен его Id). - code = buildRequestUniqueCode(VACATION_REQUEST_TYPE_ID, id, new Date()); + code = buildRequestUniqueCode(requestTypeId, id, new Date()); await list.items.getById(id).update({ RequestUniqueCode: code }); } diff --git a/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupForm.tsx b/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupForm.tsx index a65016d..7155f22 100644 --- a/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupForm.tsx +++ b/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupForm.tsx @@ -21,6 +21,7 @@ import { useMatrixEditor, getPlannedMatrix, notifyActiveStep, + useRequestTypeId, REQUEST_STATUS, isEditableStatus, IApproverToWrite, @@ -35,13 +36,7 @@ import { } from '../../shared/attachments/attachments.service'; import { IRequestFormProps } from '../../models/types'; import { IFormData, EMPTY_DATA, SetField } from './model'; -import { - saveWorkplaceRequest, - workplaceApprovalService, - WORKPLACE_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { saveWorkplaceRequest, workplaceApprovalService, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { loadPrefillFromHire } from './prefill'; import { loadWorkplaceRequest } from './load'; import { EmployeeSection } from './sections/EmployeeSection'; @@ -114,6 +109,10 @@ const WorkplaceSetupForm: React.FC = ({ requestType, context, const approval = useMemo(() => workplaceApprovalService(), []); const userEmail = context.pageContext.user.email; + // pageUrl формы (requestType.id совпадает с ним по конвенции портала, см. + // data/requests.ts) → числовой RequestTypeId, нужный approval-route API. + const requestTypeIdState = useRequestTypeId(context, requestType.id); + const requestTypeId = requestTypeIdState.id; // У новой заявки маршрут ПЛАНИРУЕТСЯ (resolve по внешнему API), у открытой на // редактирование — читается из уже записанных строк: их могли поправить // вручную при первом сохранении, и планирование затёрло бы эти правки. @@ -123,15 +122,15 @@ const WorkplaceSetupForm: React.FC = ({ requestType, context, ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase !== APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, WORKPLACE_REQUEST_TYPE_ID, userEmail, approval, 'Approval', data.DepartmentId), - [context, userEmail, approval, editId, data.DepartmentId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Approval', data.DepartmentId), + [context, userEmail, approval, editId, data.DepartmentId, requestTypeId] ); const matrix = useMatrixEditor( loadMatrix, // Отдел влияет на резолв маршрута — смена отдела у НОВОЙ заявки должна перепланировать матрицу. `${userEmail}-${editId ?? 'new'}-${data.DepartmentId}`, !readOnly, - true, + !!editId || !!requestTypeId, (login) => approval.getJobTitle(login) ); @@ -142,20 +141,24 @@ const WorkplaceSetupForm: React.FC = ({ requestType, context, ? approval .getMatrixForRequest(editId) .then((rows) => rows.filter((r) => r.phase === APPROVAL_PHASE.execution)) - : getPlannedMatrix(context, WORKPLACE_REQUEST_TYPE_ID, userEmail, approval, 'Execution', data.DepartmentId), - [context, userEmail, approval, editId, data.DepartmentId] + : getPlannedMatrix(context, requestTypeId as number, userEmail, approval, 'Execution', data.DepartmentId), + [context, userEmail, approval, editId, data.DepartmentId, requestTypeId] ); const executors = useMatrixEditor( loadExecutors, `${userEmail}-${editId ?? 'new'}-${data.DepartmentId}`, false, - true, + !!editId || !!requestTypeId, (login) => approval.getJobTitle(login) ); const isValid = data.FullName.trim().length > 0 && data.Position.trim().length > 0; - const validate = (): string | undefined => - isValid ? undefined : 'Заполните обязательные поля: Ф.И.О., Должность.'; + const validate = (): string | undefined => { + if (!isValid) return 'Заполните обязательные поля: Ф.И.О., Должность.'; + if (requestTypeIdState.error) return `Не удалось определить тип заявки: ${requestTypeIdState.error}`; + if (!requestTypeId) return 'Тип заявки ещё определяется — подождите немного и повторите.'; + return undefined; + }; const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => // order обязателен: параллельные шаги делят один номер, по позиции в @@ -182,8 +185,16 @@ const WorkplaceSetupForm: React.FC = ({ requestType, context, }; const save = (status: string): (() => Promise) => async (): Promise => { + // validate()/validateSubmit() уже гарантировали requestTypeId к моменту вызова + // (см. useRequestSubmit.run — валидирует ДО save), но TS этого через границу + // замыкания не видит — локальный const и явная проверка снимают both проблемы. + const rtId = requestTypeId; + if (!rtId) { + throw new Error('Тип заявки не определён — обновите страницу и повторите.'); + } const { id, code } = await saveWorkplaceRequest( requestId, + rtId, data, toApprovers(matrix.rows), toApprovers(executors.rows), @@ -206,7 +217,7 @@ const WorkplaceSetupForm: React.FC = ({ requestType, context, // Отправка на согласование — уведомляем первый активный шаг (поток оповещения). if (status === REQUEST_STATUS.onApproval) { await notifyActiveStep(context, approval, id, { - requestTypeId: WORKPLACE_REQUEST_TYPE_ID, + requestTypeId: rtId, dataListName: DATA_LIST_TITLE, approvalListName: APPROVAL_LIST_TITLE, }); diff --git a/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupView.tsx b/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupView.tsx index 1f6e59a..3b5a9a3 100644 --- a/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupView.tsx +++ b/src/webparts/requestPortal/forms/WorkplaceSetup/WorkplaceSetupView.tsx @@ -22,25 +22,22 @@ import { ApprovalMatrix, useMatrixEditor, notifyActiveStep, + useRequestTypeId, isEditableStatus, APPROVAL_PHASE } from '../../shared/approval'; import { getAttachments, IAttachmentInfo } from '../../shared/attachments/attachments.service'; import { IFormData, EMPTY_DATA, SetField } from './model'; import { loadWorkplaceRequest } from './load'; -import { - workplaceApprovalService, - updateReviewFields, - WORKPLACE_REQUEST_TYPE_ID, - DATA_LIST_TITLE, - APPROVAL_LIST_TITLE -} from './submit'; +import { workplaceApprovalService, updateReviewFields, DATA_LIST_TITLE, APPROVAL_LIST_TITLE } from './submit'; import { EmployeeSection } from './sections/EmployeeSection'; import { EquipmentSection } from './sections/EquipmentSection'; interface IWorkplaceSetupViewProps { context: WebPartContext; itemId: number; + /** pageUrl формы — см. IApprovalViewProps в forms/viewRegistry.ts. */ + formKey: string; } type SaveState = 'idle' | 'saving' | 'saved' | 'error'; @@ -48,7 +45,7 @@ type SaveState = 'idle' | 'saving' | 'saved' | 'error'; // Заявка «орг. рабочего места» только для просмотра (режим согласующего). // Исключение: первый согласующий шаг (по маршруту — непосредственный руководитель) // на своём этапе может заполнить «Необходимое ПО» и «Доступы» (инициатор их не знает). -const WorkplaceSetupView: React.FC = ({ context, itemId }) => { +const WorkplaceSetupView: React.FC = ({ context, itemId, formKey }) => { const [data, setData] = useState(EMPTY_DATA); const [status, setStatus] = useState(''); const [code, setCode] = useState(''); @@ -58,6 +55,7 @@ const WorkplaceSetupView: React.FC = ({ context, itemI const [canEditFields, setCanEditFields] = useState(false); const [saveState, setSaveState] = useState('idle'); + const { id: requestTypeId } = useRequestTypeId(context, formKey); const set: SetField = (key, value) => setData((prev) => ({ ...prev, [key]: value })); const approverService = useMemo( @@ -172,15 +170,22 @@ const WorkplaceSetupView: React.FC = ({ context, itemI const sectionProps = { data, set, context }; + const notify = (): void => { + if (!requestTypeId) return; + notifyActiveStep(context, approval, itemId, { + requestTypeId, + dataListName: DATA_LIST_TITLE, + approvalListName: APPROVAL_LIST_TITLE, + }).catch(() => undefined); + }; + return (
{isEditableStatus(status) && (
diff --git a/src/webparts/requestPortal/forms/WorkplaceSetup/submit.ts b/src/webparts/requestPortal/forms/WorkplaceSetup/submit.ts index 60e1f95..ec2f3f0 100644 --- a/src/webparts/requestPortal/forms/WorkplaceSetup/submit.ts +++ b/src/webparts/requestPortal/forms/WorkplaceSetup/submit.ts @@ -5,8 +5,6 @@ import { IFormData } from './model'; export const DATA_LIST_TITLE = 'Организация рабочего места — данные'; export const APPROVAL_LIST_TITLE = 'Организация рабочего места — согласование'; -// Тип заявки во внешнем approval-route API (GET /api/request-types → id 7). -export const WORKPLACE_REQUEST_TYPE_ID = 7; /** Экземпляр сервиса списка согласования этой формы. */ export function workplaceApprovalService(): ApprovalListService { @@ -66,6 +64,7 @@ export interface ISaveResult { */ export async function saveWorkplaceRequest( existingId: number | undefined, + requestTypeId: number, data: IFormData, approvers: IApproverToWrite[], executors: IApproverToWrite[], @@ -84,7 +83,7 @@ export async function saveWorkplaceRequest( const added = await list.items.add(item); id = (added as { Id: number }).Id; // Код известен только после создания элемента (нужен его Id). - code = buildRequestUniqueCode(WORKPLACE_REQUEST_TYPE_ID, id, new Date()); + code = buildRequestUniqueCode(requestTypeId, id, new Date()); await list.items.getById(id).update({ RequestUniqueCode: code }); } diff --git a/src/webparts/requestPortal/forms/registry.ts b/src/webparts/requestPortal/forms/registry.ts index 5f92615..0ac0bdb 100644 --- a/src/webparts/requestPortal/forms/registry.ts +++ b/src/webparts/requestPortal/forms/registry.ts @@ -30,6 +30,9 @@ const formRegistry: Record> = { ), 'wh-tmc-transfer': React.lazy( () => import(/* webpackChunkName: 'form-tmc-transfer' */ './TmcTransfer/TmcTransferForm') + ), + 'clearance-form': React.lazy( + () => import(/* webpackChunkName: 'form-clearance' */ './Clearance/ClearanceForm') ) }; diff --git a/src/webparts/requestPortal/forms/viewRegistry.ts b/src/webparts/requestPortal/forms/viewRegistry.ts index 5d43b19..d146894 100644 --- a/src/webparts/requestPortal/forms/viewRegistry.ts +++ b/src/webparts/requestPortal/forms/viewRegistry.ts @@ -4,6 +4,13 @@ import type { WebPartContext } from '@microsoft/sp-webpart-base'; export interface IApprovalViewProps { context: WebPartContext; itemId: number; + /** + * Ключ формы (= pageUrl типа заявки в approval-route API) — тот же, что и + * rptype в ссылке (см. ApproveFeature). Нужен режиму согласующего, чтобы + * самому резолвить числовой RequestTypeId через useRequestTypeId (форма + * подачи получает готовый requestType.id, а View — только itemId/context). + */ + formKey: string; } type ViewComponent = React.ComponentType; @@ -30,6 +37,9 @@ const viewRegistry: Record> = { ), 'wh-tmc-transfer': React.lazy( () => import(/* webpackChunkName: 'view-tmc-transfer' */ './TmcTransfer/TmcTransferView') + ), + 'clearance-form': React.lazy( + () => import(/* webpackChunkName: 'view-clearance' */ './Clearance/ClearanceView') ) }; diff --git a/src/webparts/requestPortal/shared/approval/approvalRoute.service.ts b/src/webparts/requestPortal/shared/approval/approvalRoute.service.ts index 78c6c60..975ff17 100644 --- a/src/webparts/requestPortal/shared/approval/approvalRoute.service.ts +++ b/src/webparts/requestPortal/shared/approval/approvalRoute.service.ts @@ -24,6 +24,43 @@ export async function getRequestTypes(context: WebPartContext): Promise | undefined; + +async function getRequestTypesCached(context: WebPartContext): Promise { + if (!requestTypesPromise) { + requestTypesPromise = getRequestTypes(context).catch((e) => { + requestTypesPromise = undefined; + throw e; + }); + } + return requestTypesPromise; +} + +/** + * Числовой RequestTypeId по pageUrl (rptype) формы. Id типа заявки выдаётся + * сервером при регистрации типа в админке approval-route и НЕПРЕДСКАЗУЕМ + * заранее (см. README форм — «в старой pto-tracker было 1, при заведении типа + * в админке получил новый id 11»), а pageUrl задаём мы сами в data/requests.ts + * и он стабилен. Поэтому формы резолвят числовой id отсюда (через + * useRequestTypeId), а не хардкодят его. + */ +export async function getRequestTypeIdByPageUrl(context: WebPartContext, pageUrl: string): Promise { + const types = await getRequestTypesCached(context); + const found = types.filter((t) => t.pageUrl === pageUrl)[0]; + if (!found) { + throw new Error( + `Тип заявки с pageUrl «${pageUrl}» не найден в approval-route API — зарегистрируйте его в админке маршрутов (Типы заявок).` + ); + } + return found.id; +} + // POST /api/template-routes/resolve — планируемый маршрут для типа заявки и // пользователя. Бэкенд сам подставляет исполнителей шага (executorEmail/position) // по ролям исходя из userEmail. Параметр status выбирает ФАЗУ маршрута: diff --git a/src/webparts/requestPortal/shared/approval/index.ts b/src/webparts/requestPortal/shared/approval/index.ts index 440b9db..08a35bc 100644 --- a/src/webparts/requestPortal/shared/approval/index.ts +++ b/src/webparts/requestPortal/shared/approval/index.ts @@ -19,7 +19,9 @@ export type { IResolveFieldValue, } from './models/IApprovalRouteApi'; -export { getRequestTypes, resolveRoute } from './approvalRoute.service'; +export { getRequestTypes, getRequestTypeIdByPageUrl, resolveRoute } from './approvalRoute.service'; +export { useRequestTypeId } from './useRequestTypeId'; +export type { IRequestTypeIdState } from './useRequestTypeId'; export { ApprovalListService } from './approvalList.service'; export type { IApproverToWrite } from './approvalList.service'; export { ApproverService } from './approverDecision.service'; diff --git a/src/webparts/requestPortal/shared/approval/useRequestTypeId.ts b/src/webparts/requestPortal/shared/approval/useRequestTypeId.ts new file mode 100644 index 0000000..6880679 --- /dev/null +++ b/src/webparts/requestPortal/shared/approval/useRequestTypeId.ts @@ -0,0 +1,46 @@ +import { useEffect, useState } from 'react'; +import type { WebPartContext } from '@microsoft/sp-webpart-base'; +import { getRequestTypeIdByPageUrl } from './approvalRoute.service'; + +export interface IRequestTypeIdState { + /** undefined, пока не резолвился (или если резолв упал — см. error). */ + id: number | undefined; + loading: boolean; + error: string | undefined; +} + +/** + * Резолвит числовой RequestTypeId по pageUrl формы (см. getRequestTypeIdByPageUrl). + * Формы больше не хардкодят число — используют этот хук с уже известным pageUrl: + * в форме подачи это requestType.id (проп IRequestFormProps), в режиме + * согласующего — formKey (проп IApprovalViewProps), см. ApproveFeature. + */ +export function useRequestTypeId(context: WebPartContext, pageUrl: string): IRequestTypeIdState { + const [id, setId] = useState(undefined); + const [loading, setLoading] = useState(true); + const [error, setError] = useState(undefined); + + useEffect(() => { + let alive = true; + setLoading(true); + setError(undefined); + getRequestTypeIdByPageUrl(context, pageUrl) + .then((v) => { + if (alive) { + setId(v); + setLoading(false); + } + }) + .catch((e: Error) => { + if (alive) { + setError(e.message); + setLoading(false); + } + }); + return () => { + alive = false; + }; + }, [context, pageUrl]); + + return { id, loading, error }; +}