From 87c9f92d48aa43f2af9ed027659ffc9dad77cc4d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=A2=D0=B8=D0=BC=D1=83=D1=80=20=D0=93=D0=BE=D1=80=D0=BE?= =?UTF-8?q?=D0=B4=D0=B5=D1=86=D0=BA=D0=B8=D0=B9?= Date: Thu, 29 Oct 2026 14:20:41 +0500 Subject: [PATCH] =?UTF-8?q?feat:=20=D0=BF=D0=BE=D0=BB=D1=8F=20=D0=B2=D0=B8?= =?UTF-8?q?=D0=B4=D0=B0=20=C2=AB=D0=9F=D0=B5=D1=80=D0=B5=D0=B2=D0=BE=D0=B4?= =?UTF-8?q?=20=D0=B2=20=D0=B4=D1=80=D1=83=D0=B3=D0=BE=D0=B5=20=D0=BF=D0=BE?= =?UTF-8?q?=D0=B4=D1=80=D0=B0=D0=B7=D0=B4=D0=B5=D0=BB=D0=B5=D0=BD=D0=B8?= =?UTF-8?q?=D0=B5=C2=BB=20=D0=B2=20=D0=B8=D0=B7=D0=BC=D0=B5=D0=BD=D0=B5?= =?UTF-8?q?=D0=BD=D0=B8=D0=B8=20=D1=83=D1=81=D0=BB=D0=BE=D0=B2=D0=B8=D0=B9?= =?UTF-8?q?=20=D0=A2=D0=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Четвёртый вид: новое подразделение из справочника, дата, причина а–г и обоснование. По ТЗ согласие принимающего руководителя — шаг маршрута, а approval-route руководителя нового подразделения найти не умеет, поэтому пока форма сама ставит выбранного принимающего руководителя первым шагом согласования (временно, до шага в админке маршрутов). Логика маршрута вынесена в useContractRoute — форма перерастала 300 строк. Co-Authored-By: Claude Opus 5.5 (1M context) --- .../ContractChange/ContractChangeForm.tsx | 65 +++++-------- .../forms/ContractChange/README.md | 18 +++- .../byType/ChangeTypeSections.tsx | 4 +- .../DepartmentTransferSection.tsx | 64 +++++++++++++ .../byType/DepartmentTransfer/options.ts | 13 +++ .../DepartmentTransfer/receivingHead.test.ts | 55 +++++++++++ .../DepartmentTransfer/receivingHead.ts | 68 ++++++++++++++ .../forms/ContractChange/load.ts | 20 +++- .../forms/ContractChange/model.ts | 28 +++++- .../forms/ContractChange/provisioning.json | 14 ++- .../forms/ContractChange/submit.ts | 40 +++++++- .../forms/ContractChange/useContractRoute.ts | 91 +++++++++++++++++++ 12 files changed, 427 insertions(+), 53 deletions(-) create mode 100644 src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/DepartmentTransferSection.tsx create mode 100644 src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/options.ts create mode 100644 src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/receivingHead.test.ts create mode 100644 src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/receivingHead.ts create mode 100644 src/webparts/requestPortal/forms/ContractChange/useContractRoute.ts diff --git a/src/webparts/requestPortal/forms/ContractChange/ContractChangeForm.tsx b/src/webparts/requestPortal/forms/ContractChange/ContractChangeForm.tsx index 4b357a9..b937283 100644 --- a/src/webparts/requestPortal/forms/ContractChange/ContractChangeForm.tsx +++ b/src/webparts/requestPortal/forms/ContractChange/ContractChangeForm.tsx @@ -1,5 +1,5 @@ import * as React from 'react'; -import { useState, useMemo, useCallback, useEffect } from 'react'; +import { useState, useMemo, useEffect } from 'react'; import styles from './ContractChangeForm.module.scss'; import { Alert, @@ -18,20 +18,14 @@ import { ExecutorsMatrix, ApprovalHistory, StatusBadge, - useMatrixEditor, - getPlannedMatrix, - loadRoutePhase, - isOpenedRouteReady, notifyActiveStep, useRequestTypeId, REQUEST_STATUS, isEditableStatus, - IApproverToWrite, unassignedStepMessage, resolveWriteListId, executorsToWrite, - initialSubmitStatus, - useFormExecutors + initialSubmitStatus } from '../../shared/approval'; import { effectiveUserEmail } from '../../shared/impersonation'; import { IRequestFormProps } from '../../models/types'; @@ -42,13 +36,22 @@ import { dataListTitle, dataListTitleForListId } from '../../shared/listTitles'; -import { FORM_ID, IFormData, EMPTY_DATA, SetField, REQUIRED_FIELDS_MESSAGE, routeEmail } from './model'; +import { + FORM_ID, + IFormData, + EMPTY_DATA, + SetField, + REQUIRED_FIELDS_MESSAGE, + RECEIVING_HEAD_REQUIRED_MESSAGE, + routeEmail +} from './model'; import { saveContractChangeRequest, contractChangeApprovalService } from './submit'; import { loadContractChangeRequest } from './load'; import { useContractEmployee } from './useContractEmployee'; import { RequestSection } from './sections/RequestSection'; import { ChangeTypeSections } from './byType/ChangeTypeSections'; import { missingRequiredAttachments, prepareFilesFor } from './byType/attachmentSlots'; +import { useContractRoute, matrixToApprovers } from './useContractRoute'; import { useAttachmentEditor } from '../../shared/attachments/useAttachmentEditor'; import { getAttachments } from '../../shared/attachments/attachments.service'; import { useStableCallback } from '../../shared/useStableCallback'; @@ -114,32 +117,18 @@ const ContractChangeForm: React.FC = ({ set('Employee', record) ); - // Маршрут — по сотруднику, а без учётной записи — по заявителю (см. routeEmail). - // У заявки на редактировании маршрут читается из уже записанных строк. - const routeUserEmail = routeEmail(employeeControl.person, applicantEmail); - const loadMatrix = useCallback( - () => - loadRoutePhase(approval, editId, requestStatus, 'Approval', () => - getPlannedMatrix(context, requestTypeId as number, routeUserEmail, approval, 'Approval') - ), - [context, approval, editId, requestStatus, routeUserEmail, requestTypeId] - ); - const matrix = useMatrixEditor( - loadMatrix, - `${routeUserEmail}-${editId ?? 'new'}`, - !readOnly, - isOpenedRouteReady(editId, requestStatus, requestTypeId) && (!!editId || (!!routeUserEmail && !!requestTypeId)), - (login) => approval.getJobTitle(login) - ); - - const executors = useFormExecutors({ - readOnly, + // Маршрут — по сотруднику, без учётной записи — по заявителю (см. routeEmail); при + // переводе в другое подразделение принимающий руководитель — первым шагом. + const isDepartmentTransfer = data.ChangeTypes.includes('department-transfer'); + const { matrix, executors } = useContractRoute({ context, approval, requestTypeId, - userEmail: routeUserEmail, + routeUserEmail: routeEmail(employeeControl.person, applicantEmail), editId, - requestStatus + requestStatus, + readOnly, + receivingHeadEmail: !readOnly && isDepartmentTransfer ? data.DepartmentTransferReceivingHead : '' }); const isValid = !!data.Employee && data.ChangeTypes.length > 0; @@ -150,17 +139,6 @@ const ContractChangeForm: React.FC = ({ return undefined; }; - const toApprovers = (rows: typeof matrix.rows): IApproverToWrite[] => - rows.map((r) => ({ - email: r.email, - position: r.position, - allowReassign: r.allowReassign, - order: r.order, - slaHours: r.slaHours, - askOverdueReason: r.askOverdueReason, - autoApprove: r.autoApprove - })); - const save = (status: string): (() => Promise) => async (): Promise => { // validate() уже гарантировал requestTypeId, но TS этого через замыкание не видит. const rtId = requestTypeId; @@ -173,7 +151,7 @@ const ContractChangeForm: React.FC = ({ requestId, rtId, data, - toApprovers(matrix.rows), + matrixToApprovers(matrix.rows), executorsToWrite(executors.rows), status, writeListId @@ -210,6 +188,7 @@ const ContractChangeForm: React.FC = ({ // Обязательные документы — только на отправку: черновик можно сохранить и без файла. const missingFiles = missingRequiredAttachments(data, attachments); if (missingFiles.length > 0) return `Приложите обязательные документы: ${missingFiles.join(', ')}.`; + if (isDepartmentTransfer && !data.DepartmentTransferReceivingHead) return RECEIVING_HEAD_REQUIRED_MESSAGE; if (matrix.error) { return `Не удалось определить маршрут согласования: ${matrix.error}. Попробуйте ещё раз или обратитесь к администратору.`; } diff --git a/src/webparts/requestPortal/forms/ContractChange/README.md b/src/webparts/requestPortal/forms/ContractChange/README.md index 20308a9..75d5d7e 100644 --- a/src/webparts/requestPortal/forms/ContractChange/README.md +++ b/src/webparts/requestPortal/forms/ContractChange/README.md @@ -60,7 +60,23 @@ RequestTypeId» в `docs/forms.md`). Тип и маршрут в админке | Повышение должностного оклада | `SalaryIncrease` | текущий оклад, новый оклад (`MoneyInput`, Number), дата изменения, причина (испытательный срок / расширение функционала / иное), обоснование — при «иное» это и есть текст причины | | Установление надбавки / доплаты | `Allowance` | основание (а–д: надбавка или доплата видно из пункта), дата начала, дата окончания или «бессрочно» (флажок прячет и очищает дату), размер + переключатель «% / ₸» (`UnitSwitch`, по умолчанию %), обоснование — в т.ч. объём доп. работы при совмещении и текст «иное». Письменное согласие работника на совмещение оформляет кадровик после согласования — не в форме. Вложение **«Результаты аттестации»** — только при основании «г», обязательно | | Перевод на другую должность (постоянный) | `PositionTransfer` | новая должность (`PositionPicker`: справочник `Dictionary/Positions` + ручной ввод), дата перевода, причина (карьерный рост / вакантная должность / новый объект, перераспределение функций / иное), обоснование. Работника и работодателя не различаем — причины видны всем, работник пишет текст заявления в обоснование. Вложение **«Должностная инструкция»** — всегда, обязательно | -| Остальные 7 | — | ещё не разобраны с кадрами | +| Перевод в другое подразделение | `DepartmentTransfer` | новое подразделение (`DepartmentPicker`, имя + id), дата перевода, причина (усиление подразделения / перераспределение функций / вакансия / иное), обоснование, **принимающий руководитель** (People, обязателен при отправке) — см. ниже | +| Остальные 6 | — | ещё не разобраны с кадрами | + +**Принимающий руководитель — временное решение (2026-10-29).** По ТЗ (лист +«Маршрут»: А шаг 2, Б шаг 3 параллельно с непосредственным руководителем) его +согласие — шаг маршрута. Approval-route строит маршрут по сотруднику и найти +руководителя НОВОГО подразделения не умеет, поэтому шаг вставляет форма: +`byType/DepartmentTransfer/receivingHead.ts` (`withReceivingHead`, с тестами) — +первым шагом, с должностью-пометкой «Принимающий руководитель» (по ней шаг +узнаётся у черновика); вызывается из `useContractRoute`. Смена человека/вида +пересобирает маршрут. Ограничения: человек выбирается вручную; шаг всегда первым и +последовательным (для маршрута Б параллельность инициатор может включить в матрице); +SLA и настроек из админки у шага нет. **Правильное решение** — тип шага в +approval-route «руководитель подразделения из заявки» по `departmentId` (его +форма уже хранит в `DepartmentTransferNewDepartmentId`). Когда появится — удалить +поле, `receivingHead.ts` и `receivingHeadEmail` в `useContractRoute`, передать +`departmentId` в резолв. **Вложения по колонке ТЗ «Что приложить»** (решение 2026-10-28): фиксированные подписанные слоты по виду (подпись — что за документ); обязательны там, где ТЗ diff --git a/src/webparts/requestPortal/forms/ContractChange/byType/ChangeTypeSections.tsx b/src/webparts/requestPortal/forms/ContractChange/byType/ChangeTypeSections.tsx index 5185752..fa69061 100644 --- a/src/webparts/requestPortal/forms/ContractChange/byType/ChangeTypeSections.tsx +++ b/src/webparts/requestPortal/forms/ContractChange/byType/ChangeTypeSections.tsx @@ -4,6 +4,7 @@ import { ISectionProps } from '../model'; import { SalaryIncreaseSection } from './SalaryIncrease/SalaryIncreaseSection'; import { AllowanceSection } from './Allowance/AllowanceSection'; import { PositionTransferSection } from './PositionTransfer/PositionTransferSection'; +import { DepartmentTransferSection } from './DepartmentTransfer/DepartmentTransferSection'; /** * Вид изменения → секция с его полями. Partial: поля видов добавляются по мере @@ -13,7 +14,8 @@ import { PositionTransferSection } from './PositionTransfer/PositionTransferSect const SECTION_BY_TYPE: Partial>> = { 'salary-increase': SalaryIncreaseSection, allowance: AllowanceSection, - 'position-transfer': PositionTransferSection + 'position-transfer': PositionTransferSection, + 'department-transfer': DepartmentTransferSection }; /** Секции всех выбранных видов, в порядке выбора (сейчас вид один — см. MAX_CHANGE_TYPES). */ diff --git a/src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/DepartmentTransferSection.tsx b/src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/DepartmentTransferSection.tsx new file mode 100644 index 0000000..01f624e --- /dev/null +++ b/src/webparts/requestPortal/forms/ContractChange/byType/DepartmentTransfer/DepartmentTransferSection.tsx @@ -0,0 +1,64 @@ +import * as React from 'react'; +import { CollapseSection, FormField, TextInput, TextArea, Select, PeopleField } from '../../../../shared/ui'; +import { DepartmentPicker } from '../../../../shared/dictionary'; +import { CHANGE_TYPE_LABELS } from '../../changeTypes'; +import { ISectionProps } from '../../model'; +import { DEPARTMENT_TRANSFER_REASONS } from './options'; + +/** + * Поля вида «Перевод в другое подразделение». Состав согласован с Тимуром по ТЗ + * кадров: новое подразделение (справочник), дата, причина (а–г), обоснование и + * принимающий руководитель. Его согласие по ТЗ — шаг маршрута: форма сама ставит + * его первым шагом согласования (см. receivingHead.ts). Обязателен при отправке. + * + * Поля — прямо в CollapseSection, без FormGrid (тело секции само сетка). + */ +export const DepartmentTransferSection: React.FC = ({ data, set, context }) => ( + + + { + set('DepartmentTransferNewDepartment', name); + set('DepartmentTransferNewDepartmentId', id); + }} + placeholder="Начните вводить название подразделения…" + /> + + + set('DepartmentTransferDate', v)} + /> + + + set('DepartmentTransferReceivingHead', v)} + placeholder="Начните вводить Ф.И.О. руководителя, который принимает работника…" + /> + + +