Compare commits

14 Commits

Author SHA1 Message Date
gorbenko.pavel 06dff47f70 Реестр пакетов: организация tansu, адреса по https 2026-10-29 12:22:31 +00:00
gorbenko.pavel 842b9ad678 Обновлён @tnsu/ui-kit до 0.14.0, реестр @tnsu — организация Tansu
Все версии ui-kit скопированы в публичный реестр организации Tansu (контрольные суммы совпадают),
токен для установки больше не нужен. Сборка проекта проверена.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-28 16:47:30 +05:00
Тимур Городецкий 2591d94414 docs: CHANGELOG — заметное добавление поставщика (#12)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-28 08:59:42 +05:00
Тимур Городецкий 2eaa75e876 feat(ct): заметное добавление потенциального поставщика на Шаге 3 (#12)
Блок добавления не замечали: поле без подписи и серая заблокированная кнопка
читались как «не работает», а сам блок стоял в зоне настроек. Теперь у блока
заголовок и подсказка, кнопка «Добавить» не блокируется (с пустым названием
подсказывает, что ввести), «Из базы поставщиков» рядом. И вторая точка входа
там, где ищут, — колонка «+ Добавить поставщика» в шапке таблицы с окном
«вручную или из базы». Добавление вручную — одна функция на обе кнопки.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-28 08:59:10 +05:00
Тимур Городецкий eaef011b73 docs: CHANGELOG — своя единица у доп. расходов (#11)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 17:36:01 +05:00
Тимур Городецкий 99c720f26c feat(ct): своя единица у логистики и доп. расходов на Шаге 3 (#11)
У логистики единица была зашита — «рейс», у доп. расходов её задавали только
при добавлении. Теперь единица строки — поле ввода; поставщики, у которых её
не меняли, получают новую, своя единица поставщика остаётся. Подсказка в
количестве логистики — единица строки. Строка уже в SP — единица обновляется
в «Позиции СТ» при сохранении.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 17:35:53 +05:00
Тимур Городецкий 107110b2b8 docs: CHANGELOG — заглушки КП и двойная колонка (#8, #10)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 17:03:06 +05:00
Тимур Городецкий da2c86b4a9 fix(ct): файлы-заглушки КП не видны в списках и не идут в пакет INCORE (#10)
Поставщику, добавленному на Шаге 3 вручную, система создаёт в библиотеке КП
файл-заглушку .txt — оффер в SP должен ссылаться на файл КП. Он показывался
в «Коммерческих предложениях», в «Загруженных КП» Шага 2, в подписи колонки
и скачивался в пакет INCORE. Теперь при показе и выгрузке заглушки скрыты
(узнаём по .txt — на Шаге 2 загружают только pdf/docx/doc), в подписи колонки —
«Добавлен вручную». Данные формы не меняются: по ним сохранение находит файл.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 17:02:58 +05:00
Тимур Городецкий 09a0889741 fix(ct): поставщик Шага 3 после черновика не задваивается при открытии СТ (#8)
«Сохранить черновик» оставляет поставщика, добавленного на Шаге 3, в списке
шага, а в SP он уже лежит колонкой «поставщик_заглушка». Если других
поставщиков в СТ нет, при открытии из реестра колонки собирались из SP — и
поставщик приходил дважды: из SP без цен и из сессии с ценами (подписи
колонок именем файла-заглушки .txt и «КП 2»). Теперь колонки из SP, которые
уже есть поставщиком Шага 3, не добавляются, а их офферы идут к нему.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:52:37 +05:00
Тимур Городецкий 5c21718914 docs: CHANGELOG — подсветка незаполненного (#9)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:43:39 +05:00
Тимур Городецкий 78e5eec9b6 feat(ct): подсветка незаполненного на Шаге 3, условия оплаты обязательны (#9)
Красная рамка — то, что не пускает дальше Шага 3: «Выбрать», пока нет
победителя, пустой километраж логистики, пустые условия оплаты. Условия
оплаты были помечены обязательными, но сохранять не мешали — теперь
проверяются. Пустая цена — бледной заливкой: заметно, но сохранению не мешает,
поставщик мог не предложить позицию. Подсветка видна сразу, пока заполняют.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:43:31 +05:00
Тимур Городецкий ad76e59ca0 feat(ct): «Сохранить черновик» на Шагах 2 и 3 (#8)
Запись шага в SharePoint без проверок и без перехода — правки не пропадают при
закрытии вкладки. Проверено в воркбенче, дублей поставщиков в SP нет.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:24:03 +05:00
Тимур Городецкий 053e4d4c3e docs: CHANGELOG — черновик на Шагах 2 и 3 (#8)
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:24:03 +05:00
Тимур Городецкий 09b1df87c3 feat(ct): «Сохранить черновик» на Шагах 2 и 3 (#8)
Несохранённые правки жили в памяти вкладки и пропадали при её закрытии, а
«Сохранить и продолжить» на Шаге 3 не пускает без победителя и километража.
Черновик пишет шаг в SharePoint так же, без проверок, и оставляет на шаге;
рядом — время последнего сохранения.

На Шаге 3 черновик не переносит поставщиков, добавленных на шаге, в общий
список: таблица держит их в своём, перенос задвоил бы колонку и обнулил бы
суммы. Повторные черновики дублей в SP не создают — SP Id уже запомнены.
На Шаге 2 черновик доступен, когда заданы курсы валют: они пишутся в общий
список «Курс Валют».

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
2026-10-27 16:16:23 +05:00
30 changed files with 848 additions and 120 deletions
+1 -1
View File
@@ -1 +1 @@
@tnsu:registry=http://git.tnsu.kz/api/packages/timur.gorodetskiy/npm/
@tnsu:registry=https://git.tnsu.kz/api/packages/tansu/npm/
+18
View File
@@ -7,6 +7,17 @@
## [Не выпущено]
### Добавлено
- СТ ОМТС, Шаги 2 и 3: кнопка «Сохранить черновик» — записывает шаг в SharePoint без проверок
и оставляет на шаге, чтобы правки не пропали при закрытии вкладки. (#8)
- СТ ОМТС, Шаг 3: незаполненное подсвечивается сразу. Красная рамка — то, без чего дальше не
пустит (победитель, километраж, условия оплаты); бледная заливка — пустая цена. (#9)
- СТ ОМТС, Шаг 3: у логистики и доп. расходов можно вписать свою единицу измерения — у логистики
вместо «рейс» может быть «тонна», «машина» и т.п. (#11)
- СТ ОМТС, Шаг 3: добавить потенциального поставщика проще найти — блок над таблицей с заголовком
и активной кнопкой, и кнопка «+ Добавить поставщика» прямо в шапке таблицы. (#12)
### Изменено
- СТ ОМТС, Шаг 3: крестик у поставщика не удаляет его, а исключает из сравнения с обязательной
@@ -16,9 +27,16 @@
периода. Старую потребность можно найти, не загружая весь период. (#5)
- СТ ОМТС, Шаг 3: без выбранного поставщика-победителя «Сохранить и продолжить» не пускает
дальше. (#7)
- СТ ОМТС, Шаг 3: условия оплаты обязательны для «Сохранить и продолжить» — раньше были помечены
обязательными, но сохранять не мешали. (#9)
### Исправлено
- СТ ОМТС: у поставщика, добавленного на Шаге 3 вручную, не видно служебного файла `.txt` —
ни в списках КП на Шагах 2 и 3, ни в подписи колонки, ни в пакете для INCORE. (#10)
- СТ ОМТС, Шаг 3: поставщик, добавленный на шаге и сохранённый черновиком, не задваивается при
открытии СТ из реестра. (#8)
- СТ ОМТС, Шаг 1: позиции, выбранные до поиска или смены периода, больше не пропадают при
«Сохранить и продолжить». (#5)
- СТ ОМТС: цена, скопированная с пробелами между разрядами («100 000»), считается целиком, а не
+2 -2
View File
@@ -269,7 +269,7 @@ Info "Подключаем @tnsu/ui-kit..."
# .npmrc: адрес registry для скоупа @tnsu (НЕ секрет, можно коммитить).
# Токен на чтение — у каждого разработчика свой, в ~/.npmrc (в проект не кладём).
$npmrc = @'
@tnsu:registry=http://git.tnsu.kz/api/packages/timur.gorodetskiy/npm/
@tnsu:registry=https://git.tnsu.kz/api/packages/tansu/npm/
'@
$npmrc | Set-Content -Encoding UTF8 ".npmrc"
Success ".npmrc создан (адрес registry @tnsu)."
@@ -282,7 +282,7 @@ try {
} catch {
Warn "Не удалось поставить @tnsu/ui-kit. Скорее всего нет токена @tnsu в ~/.npmrc."
Warn "Создай токен (Gitea → Настройки → Приложения → package: Read) и пропиши в ~/.npmrc:"
Warn " //git.tnsu.kz/api/packages/timur.gorodetskiy/npm/:_authToken=ТВОЙ_ТОКЕН"
Warn " //git.tnsu.kz/api/packages/tansu/npm/:_authToken=ТВОЙ_ТОКЕН"
Warn "Потом выполни: npm install @tnsu/ui-kit"
}
+4 -4
View File
@@ -15,7 +15,7 @@
"@microsoft/sp-office-ui-fabric-core": "1.22.1",
"@microsoft/sp-property-pane": "1.22.1",
"@microsoft/sp-webpart-base": "1.22.1",
"@tnsu/ui-kit": "^0.11.0",
"@tnsu/ui-kit": "^0.14.0",
"ncalayer-js-client": "^1.6.0",
"react": "17.0.1",
"react-dom": "17.0.1",
@@ -6269,9 +6269,9 @@
"license": "0BSD"
},
"node_modules/@tnsu/ui-kit": {
"version": "0.11.0",
"resolved": "http://git.tnsu.kz/api/packages/timur.gorodetskiy/npm/%40tnsu%2Fui-kit/-/0.11.0/ui-kit-0.11.0.tgz",
"integrity": "sha512-Yv4hUmCsDNvGMj32ygNhBZiurhe5lU0IHvWSZMkG15h+xmCBmsTzgrItmKenKkKiOM2R4cjOo4D2W8SsTNHleQ==",
"version": "0.14.0",
"resolved": "https://git.tnsu.kz/api/packages/tansu/npm/%40tnsu%2Fui-kit/-/0.14.0/ui-kit-0.14.0.tgz",
"integrity": "sha512-Vun9AUM9jL8rttbsbKo5Yh6TUz1srkNqxiO6mmzebrSo1JOASNJqNjJPMTS+8Jt3MTrJPklvh0TbeqGQzw7qxQ==",
"license": "ISC",
"peerDependencies": {
"@fluentui/react": "^8.106.4",
+1 -1
View File
@@ -19,7 +19,7 @@
"@microsoft/sp-office-ui-fabric-core": "1.22.1",
"@microsoft/sp-property-pane": "1.22.1",
"@microsoft/sp-webpart-base": "1.22.1",
"@tnsu/ui-kit": "^0.11.0",
"@tnsu/ui-kit": "^0.14.0",
"ncalayer-js-client": "^1.6.0",
"react": "17.0.1",
"react-dom": "17.0.1",
@@ -9,7 +9,7 @@ import { Step1PositionSelection } from './Step1PositionSelection/Step1PositionSe
import { Step2CommercialOffers } from './Step2CommercialOffers/Step2CommercialOffers';
import { Step3ComparisonTable } from './Step3ComparisonTable/Step3ComparisonTable';
import { Step4Approval } from './Step4Approval/Step4Approval';
import { buildColumnsFromSpOffers, buildSupplierColumnId, findColumnForRecord, findWinnerColumnId } from './utils/supplierColumns';
import { buildColumnsFromSpOffers, buildSupplierColumnId, findColumnForRecord, findWinnerColumnId, withoutSessionExtras } from './utils/supplierColumns';
import { buildOffersFromSpRecords } from './utils/spOffers';
import { excludedItemsToFormData } from './utils/excludedColumns';
import { isUserInApprovalChain } from './utils/approvalAccess';
@@ -122,16 +122,22 @@ const CtCreateForm: React.FC<ICtCreateFormProps> = ({ context, userDisplayName,
// КП в разных валютах. Используются, только когда в сессии колонок нет.
const currencyCodeOf = (valutaSpId?: number): string =>
valutaSpId ? (rateById.get(valutaSpId)?.code ?? 'KZT') : 'KZT';
const spSuppliers = buildColumnsFromSpOffers(rawOffers.map(o => ({
// Поставщики Шага 3 из сессии (после «Сохранить черновик») уже лежат в SP — их колонки из SP
// не дублируем.
const sessionExtras = formData.step3ExtraSuppliers ?? [];
const spSuppliers = withoutSessionExtras(buildColumnsFromSpOffers(rawOffers.map(o => ({
supplierSpId: o.supplierSpId,
supplierName: o.supplierName,
offerFileSpId: o.offerFileSpId,
currencyCode: currencyCodeOf(o.valutaSpId),
})));
}))), sessionExtras);
const supplierMap = new Map<number, string>();
rawOffers.forEach(o => { if (!supplierMap.has(o.supplierSpId)) supplierMap.set(o.supplierSpId, o.supplierName); });
// Колонка оффера: из сессии, если она есть (в т.ч. старая колонка без файла КП), иначе — из SP.
const visibleColumns = formData.suppliers.length > 0 ? formData.suppliers : spSuppliers;
// Поставщики Шага 3 — тоже колонки: их офферы из SP должны попасть к ним, а не в убранную колонку.
const visibleColumns = formData.suppliers.length > 0
? formData.suppliers
: [...spSuppliers, ...sessionExtras.map(es => ({ id: es.id, spId: es.spId, name: es.name, hasOffer: true }))];
const columnIdOf = (supplierSpId: number, offerFileSpId?: number): string =>
findColumnForRecord(visibleColumns, supplierSpId, offerFileSpId)?.id
?? buildSupplierColumnId(supplierSpId, offerFileSpId);
@@ -0,0 +1,13 @@
@import '../common/theme';
.root {
display: flex;
align-items: center;
gap: 12px;
}
.savedAt {
font-size: 12px;
color: $muted;
white-space: nowrap;
}
@@ -0,0 +1,35 @@
import * as React from 'react';
import { Button, Spinner } from '@tnsu/ui-kit';
import styles from './SaveDraftButton.module.scss';
interface Props {
onSave: () => void;
isSaving: boolean;
disabled?: boolean;
// Подсказка, почему кнопка недоступна.
disabledReason?: string;
// Время последнего сохранения черновика в этой вкладке.
savedAt?: Date;
}
const TIME_FORMAT: Intl.DateTimeFormatOptions = { hour: '2-digit', minute: '2-digit' };
// «Сохранить черновик» на шагах СТ: записать недоделанный шаг в SharePoint без проверок и остаться
// на шаге — чтобы правки не пропали при закрытии вкладки и были видны на другом компьютере.
export const SaveDraftButton: React.FC<Props> = ({ onSave, isSaving, disabled = false, disabledReason, savedAt }) => (
<div className={styles.root}>
{savedAt && (
<span className={styles.savedAt}>
Черновик сохранён в {savedAt.toLocaleTimeString('ru-RU', TIME_FORMAT)}
</span>
)}
<Button
variant="secondary"
onClick={onSave}
disabled={disabled || isSaving}
title={disabled ? disabledReason : 'Сохранить в SharePoint и остаться на этом шаге'}
>
{isSaving ? <Spinner size="small" /> : 'Сохранить черновик'}
</Button>
</div>
);
@@ -13,6 +13,8 @@ import { SuppliersWithoutKp } from './SuppliersWithoutKp';
import { UploadedKpList } from './UploadedKpList';
import { removeKpFileFromComparison } from './kpRemoval';
import { applySupplierCurrency, getSupplierCurrency } from './kpCurrency';
import { SaveDraftButton } from '../SaveDraftButton/SaveDraftButton';
import { withoutPlaceholders } from '../utils/placeholderKpFile';
import styles from './Step2CommercialOffers.module.scss';
interface Props {
@@ -50,6 +52,7 @@ export const Step2CommercialOffers: React.FC<Props> = ({
);
const [isSaving, setIsSaving] = useState(false);
const [saveError, setSaveError] = useState<string | undefined>(undefined);
const [draftSavedAt, setDraftSavedAt] = useState<Date | undefined>(undefined);
const kp = useKpProcessing(service, formData, onSave);
@@ -136,12 +139,18 @@ export const Step2CommercialOffers: React.FC<Props> = ({
kp.handleFileRemove(localId);
};
const handleSave = async (): Promise<void> => {
// 'draft' — записать шаг в SP и остаться: распознанные КП попадают в SP только здесь, и без
// черновика пропадали при закрытии вкладки.
const handleSave = async (mode: 'next' | 'draft' = 'next'): Promise<void> => {
setIsSaving(true);
setSaveError(undefined);
try {
const patch = await persistStep2({ service, formData, kpFiles: kp.kpFiles, currencyRates, noOfferSuppliers });
onSave(patch);
if (mode === 'draft') {
setDraftSavedAt(new Date());
return;
}
onStepComplete(2);
onNext();
} catch (err) {
@@ -152,6 +161,8 @@ export const Step2CommercialOffers: React.FC<Props> = ({
};
const canProceed = !isSaving && currencyRates.every(r => r.rateToKzt > 0);
// Заглушки поставщиков, добавленных на Шаге 3 вручную, — не КП: в списке не показываем.
const realKpFiles = withoutPlaceholders(formData.uploadedKpFiles ?? []);
const currencyOptions = ['KZT', ...currencyRates.filter(r => r.code !== 'KZT').map(r => r.code)];
const uploadedFiles: UploadedFile[] = kp.kpFiles.map(f => ({
@@ -193,9 +204,9 @@ export const Step2CommercialOffers: React.FC<Props> = ({
<SuppliersWithoutKp items={noOfferSuppliers} onChange={handleNoOfferChange} disabled={isReadOnly} />
{!isRestrictedView && (formData.uploadedKpFiles?.length ?? 0) > 0 && (
{!isRestrictedView && realKpFiles.length > 0 && (
<UploadedKpList
files={formData.uploadedKpFiles}
files={realKpFiles}
service={service}
currencyOptions={currencyOptions}
getCurrency={file => getSupplierCurrency(formData.suppliers ?? [], file)}
@@ -212,9 +223,19 @@ export const Step2CommercialOffers: React.FC<Props> = ({
<div className={styles.footerRight}>
{saveError && <Alert variant="err">{saveError}</Alert>}
{canEdit && !isReadOnly && (
<Button variant="accent" onClick={handleSave} disabled={!canProceed}>
{isSaving ? <Spinner size="small" /> : 'Сохранить и продолжить'}
</Button>
<>
<SaveDraftButton
onSave={() => { handleSave('draft').catch(() => undefined /* ошибку показывает handleSave */); }}
isSaving={isSaving}
// Курсы пишутся в общий список «Курс Валют» — нулевой курс туда писать нельзя.
disabled={!canProceed}
disabledReason="Укажите курсы всех валют"
savedAt={draftSavedAt}
/>
<Button variant="accent" onClick={() => { handleSave().catch(() => undefined /* ошибку показывает handleSave */); }} disabled={!canProceed}>
{isSaving ? <Spinner size="small" /> : 'Сохранить и продолжить'}
</Button>
</>
)}
</div>
</div>
@@ -59,6 +59,8 @@ export function AddExtraRow({ matrix, isRestrictedView, colSpan }: Props): JSX.E
{suppliersWithOffer.map(s => (
<td key={s.id} colSpan={getSubCols(supplierCurrencies[s.id] ?? 'KZT', isRestrictedView)} />
))}
{/* Под колонкой «+ Добавить поставщика» (видна там же, где эта строка, — не в режиме просмотра). */}
<td className={styles.addSupplierFillerCell} />
</tr>
);
}
@@ -0,0 +1,69 @@
import * as React from 'react';
import { useState } from 'react';
import { Button, Dialog, Icon, TextInput } from '@tnsu/ui-kit';
import styles from './Step3ComparisonTable.module.scss';
interface Props {
currencyOptions: string[];
onAddManual: (name: string, currency: string) => void;
onOpenDatabase: () => void;
onClose: () => void;
}
const DEFAULT_CURRENCY = 'KZT';
// Окно кнопки «+ Добавить поставщика» в шапке таблицы: вручную (название и валюта) или из базы
// поставщиков. Добавление — те же функции, что у блока над таблицей.
export const AddSupplierDialog: React.FC<Props> = ({ currencyOptions, onAddManual, onOpenDatabase, onClose }) => {
const [name, setName] = useState('');
const [currency, setCurrency] = useState(DEFAULT_CURRENCY);
const canAdd = name.trim().length > 0;
const handleAdd = (): void => {
if (!canAdd) return;
onAddManual(name, currency);
onClose();
};
return (
<Dialog
open
title="Добавить потенциального поставщика"
subText="Поставщик, у которого нет загруженного КП: цены вносятся в таблицу вручную."
onDismiss={onClose}
footer={<Button variant="secondary" onClick={onClose}>Отмена</Button>}
>
<div className={styles.addSupplierDialogBody}>
<div className={styles.addSupplierRow}>
<div className={styles.addInput}>
<TextInput
value={name}
onChange={setName}
placeholder="Наименование поставщика"
ariaLabel="Наименование поставщика"
autoFocus
onKeyDown={e => { if (e.key === 'Enter') handleAdd(); }}
/>
</div>
<select
className={styles.currencySelect}
value={currency}
onChange={e => setCurrency(e.target.value)}
aria-label="Валюта поставщика"
>
{currencyOptions.map(c => <option key={c} value={c}>{c}</option>)}
</select>
<Button variant="accent" onClick={handleAdd} disabled={!canAdd}>
<Icon name="plus" size="sm" />
Добавить
</Button>
</div>
<div className={styles.addSupplierDivider}>или</div>
<Button variant="secondary" onClick={() => { onClose(); onOpenDatabase(); }}>
<Icon name="building" size="sm" />
Выбрать из базы поставщиков
</Button>
</div>
</Dialog>
);
};
@@ -0,0 +1,82 @@
import * as React from 'react';
import { useState } from 'react';
import { Button, Icon, TextInput } from '@tnsu/ui-kit';
import styles from './Step3ComparisonTable.module.scss';
interface Props {
name: string;
currency: string;
currencyOptions: string[];
onNameChange: (value: string) => void;
onCurrencyChange: (value: string) => void;
onAdd: () => void;
onOpenDatabase: () => void;
}
const NAME_INPUT_ID = 'step3-new-supplier-name';
// Блок «Потенциальные поставщики» над таблицей. Раньше это были поле, выпадашка и серая кнопка без
// подписи — их не замечали: заблокированная кнопка читалась как «не работает». Теперь у блока есть
// заголовок, кнопка не блокируется: с пустым названием подсказывает, что ввести.
export const AddSupplierToolbar: React.FC<Props> = ({
name, currency, currencyOptions, onNameChange, onCurrencyChange, onAdd, onOpenDatabase,
}) => {
const [isNameMissing, setIsNameMissing] = useState(false);
const handleAdd = (): void => {
if (!name.trim()) {
setIsNameMissing(true);
// Поле — в ui-kit, ref он не отдаёт; id задаём сами.
document.getElementById(NAME_INPUT_ID)?.focus();
return;
}
setIsNameMissing(false);
onAdd();
};
const handleNameChange = (value: string): void => {
if (value.trim()) setIsNameMissing(false);
onNameChange(value);
};
return (
<section className={styles.addSupplierBlock}>
<div className={styles.addSupplierHeading}>
<h4 className={styles.addSupplierTitle}>Потенциальные поставщики</h4>
<span className={styles.addSupplierHint}>
Добавьте поставщика, у которого нет загруженного КП, — вручную или из базы поставщиков
</span>
</div>
<div className={styles.addSupplierRow}>
<div className={styles.addInput}>
<TextInput
id={NAME_INPUT_ID}
placeholder="Наименование поставщика"
value={name}
onChange={handleNameChange}
ariaLabel="Наименование нового потенциального поставщика"
onKeyDown={e => { if (e.key === 'Enter') handleAdd(); }}
/>
</div>
<select
className={styles.currencySelect}
value={currency}
onChange={e => onCurrencyChange(e.target.value)}
aria-label="Валюта нового потенциального поставщика"
>
{currencyOptions.map(c => <option key={c} value={c}>{c}</option>)}
</select>
<Button variant="accent" onClick={handleAdd}>
<Icon name="plus" size="sm" />
Добавить
</Button>
<span className={styles.addSupplierOr}>или</span>
<Button variant="secondary" onClick={onOpenDatabase}>
<Icon name="building" size="sm" />
Из базы поставщиков
</Button>
</div>
{isNameMissing && <span className={styles.addSupplierError}>Введите наименование поставщика</span>}
</section>
);
};
@@ -19,12 +19,14 @@ interface Props {
isRestrictedView: boolean;
adata: UseAdataCheckResult;
recommendedId?: string; // победитель формулы 80% цена / 20% условия оплаты (useRecommendedPick)
// Кнопка «+ Добавить поставщика» в последней колонке шапки.
onAddSupplierClick: () => void;
}
// Таблица сравнения целиком: шапка (фикс-колонки + заголовки поставщиков) + строки позиций
// (MatrixRowView/ExtraPositionRow) + итоговые строки + строка добавления (AddExtraRow).
// Перенесено 1:1 из Step3ComparisonTable (§стайл-гайд: чистая вёрстка, логика — в useStep3Matrix).
export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adata, recommendedId }: Props): JSX.Element {
export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adata, recommendedId, onAddSupplierClick }: Props): JSX.Element {
const {
isReadOnly, suppliersWithOffer, supplierCurrencies, selectedSupplierId,
supplierStatuses, nameColWidth, regularPositions, extraPositionsRows, allPositions, computedOffers,
@@ -34,8 +36,15 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
// Колонка, для которой открыто окно «Исключить из сравнения».
const [excludingSupplierId, setExcludingSupplierId] = React.useState<string | undefined>(undefined);
const excludingSupplier = suppliersWithOffer.find(s => s.id === excludingSupplierId);
// Как в проверке перед сохранением (getStep3SaveError): выбор на исключённом — не победитель.
const hasWinner = suppliersWithOffer.some(s => s.id === selectedSupplierId);
const totalColSpan = FIXED_COLS + suppliersWithOffer.reduce((sum, s) => sum + getSubCols(supplierCurrencies[s.id] ?? 'KZT', isRestrictedView), 0);
// Последняя колонка «+ Добавить поставщика» — там, где ищут, как добавить колонку. Во всех строках
// под ней пустая ячейка (addSupplierFillerCell), иначе строки таблицы были бы разной длины.
const showAddColumn = !isReadOnly;
const totalColSpan = FIXED_COLS
+ suppliersWithOffer.reduce((sum, s) => sum + getSubCols(supplierCurrencies[s.id] ?? 'KZT', isRestrictedView), 0)
+ (showAddColumn ? 1 : 0);
// Высота row1 (шапка поставщика) переменная (теги валют, выбор поставщика, состояние Adata),
// а row2 прибит sticky сразу под ней. Измеряем реальную высоту и отдаём в CSS-переменную,
@@ -113,7 +122,12 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
{!isReadOnly ? (
<>
<button
className={[styles.selectSupplierBtn, isSel ? styles.selectSupplierBtnActive : ''].filter(Boolean).join(' ')}
className={[
styles.selectSupplierBtn,
isSel ? styles.selectSupplierBtnActive : '',
// Победитель не выбран — красная рамка у всех «Выбрать»: без него дальше Шага 3 не пускает.
!hasWinner ? styles.selectSupplierBtnRequired : '',
].filter(Boolean).join(' ')}
onClick={() => setSelectedSupplierId(toggleWinner(selectedSupplierId, s.id))}
aria-pressed={isSel}
title={isSel ? 'Снять выбор' : 'Выбрать победителем'}
@@ -135,6 +149,13 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
</th>
);
})}
{showAddColumn && (
<th rowSpan={2} className={styles.addSupplierHeaderCell}>
<button className={styles.addSupplierHeaderBtn} onClick={onAddSupplierClick} title="Добавить потенциального поставщика">
+ Добавить поставщика
</button>
</th>
)}
</tr>
{/* Row 2: sub-headers — 4 cols for KZT, 6 cols for foreign currency */}
@@ -199,6 +220,7 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
</React.Fragment>
);
})}
{showAddColumn && <td className={styles.addSupplierFillerCell} />}
</tr>
{/* Доп. расходы positions — single row each (no bundle support needed) */}
@@ -235,6 +257,7 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
</React.Fragment>
);
})}
{showAddColumn && <td className={styles.addSupplierFillerCell} />}
</tr>
)}
@@ -258,6 +281,7 @@ export function ComparisonMatrix({ matrix, currencyRates, isRestrictedView, adat
</React.Fragment>
);
})}
{showAddColumn && <td className={styles.addSupplierFillerCell} />}
</tr>
{/* Inline add-position row */}
@@ -5,6 +5,7 @@ import { PositionRow } from './matrixTypes';
import { Step3Matrix } from './useStep3Matrix';
import { getUnitStatus } from '../utils/unitConversion';
import { unitStatusClass } from './unitStatusStyles';
import { isPriceMissing } from './step3Validation';
import { formatKzt, formatNum, formatQty, round2Str, buildOfferKey, isMinPositionTotal } from './matrixHelpers';
import styles from './Step3ComparisonTable.module.scss';
@@ -24,7 +25,7 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
isReadOnly, edits, extraPositions, translateUnits, rawValues, deliveryKm, selectedSupplierId, nameColWidth,
suppliersWithOffer, supplierCurrencies, computedOffers, minTotalByPosition,
updateSupplierProductName, updateUnit, updateEdit, updatePriceKzt, handleDeliveryKmChange,
handleDeleteExtraPosition,
handleDeleteExtraPosition, handleExtraPositionUnitChange,
} = matrix;
const ep = extraPositions.find(e => e.id === pos.id);
@@ -46,7 +47,22 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
/>
)}
</td>
<td className={styles.unitCell}>{pos.unit}</td>
{/* Единица строки — своя (у логистики по умолчанию «рейс»: бывает «тонна», «машина», «услуга»). */}
<td className={styles.unitCell}>
{!isReadOnly ? (
<div className={styles.offerInput}>
<TextInput
value={pos.unit}
onChange={val => handleExtraPositionUnitChange(pos.id, val)}
placeholder="ед. изм."
borderless
ariaLabel={`Ед. изм. строки ${pos.materialName}`}
/>
</div>
) : (
pos.unit
)}
</td>
<td className={styles.qtyCell}>—</td>
{suppliersWithOffer.map(s => {
const key = key0(s);
@@ -75,6 +91,10 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
const dispPrice = (tRatio !== null && price !== undefined) ? price / tRatio : price;
const dispPriceKzt = tRatio !== null ? priceKztEffective / tRatio : priceKztEffective;
const showInput = !isReadOnly && !translateUnits;
// Подсветка сразу, пока заполняют: пустая цена — заливкой (сохранять не мешает),
// пустой километраж логистики — рамкой (не пускает дальше Шага 3).
const priceMissingCls = !isReadOnly && isPriceMissing(price, item?.priceKztOverride) ? styles.missingPriceCell : '';
const kmMissingCls = !isReadOnly && deliveryKm[`${pos.id}::${s.id}`] === undefined ? styles.requiredMissingBox : '';
return (
<React.Fragment key={s.id}>
@@ -115,16 +135,16 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
<TextInput
value={rawValues[`${s.id}::${pos.id}::0::quantity`] ?? (qty !== undefined ? String(qty) : '')}
onChange={val => updateEdit(s.id, pos.id, 0, 'quantity', val)}
placeholder="рейсы"
placeholder={pos.unit || 'кол-во'}
borderless
ariaLabel={`Кол-во рейсов ${s.name} ${pos.materialName}`}
ariaLabel={`Кол-во (${pos.unit || 'ед.'}) ${s.name} ${pos.materialName}`}
/>
</div>
) : (
<span className={styles.computedValue}>{dispQty !== undefined ? formatQty(dispQty) : '—'}</span>
)}
</div>
<div className={styles.deliveryKmBox}>
<div className={[styles.deliveryKmBox, kmMissingCls].filter(Boolean).join(' ')}>
{showInput ? (
<div className={styles.offerInput}>
<TextInput
@@ -160,7 +180,7 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
</td>
{isForeign && (
<>
<td className={[styles.dataCell, selCls].filter(Boolean).join(' ')}>
<td className={[styles.dataCell, priceMissingCls, selCls].filter(Boolean).join(' ')}>
{showInput ? (
<div className={styles.offerInput}>
<TextInput
@@ -180,7 +200,7 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
</td>
</>
)}
<td className={[styles.dataCell, selCls].filter(Boolean).join(' ')}>
<td className={[styles.dataCell, priceMissingCls, selCls].filter(Boolean).join(' ')}>
{showInput ? (
<div className={styles.offerInput}>
<TextInput
@@ -214,6 +234,8 @@ export function ExtraPositionRow({ pos, posIndex, regularPositionsCount, matrix,
</React.Fragment>
);
})}
{/* Под колонкой «+ Добавить поставщика». */}
{!isReadOnly && <td className={styles.addSupplierFillerCell} />}
</tr>
);
}
@@ -6,6 +6,7 @@ import { Step3Matrix } from './useStep3Matrix';
import { getUnitStatus } from '../utils/unitConversion';
import { parseNumberInput } from '../utils/parseNumberInput';
import { unitStatusClass } from './unitStatusStyles';
import { isPriceMissing } from './step3Validation';
import { getSubCols, formatKzt, formatNum, formatQty, round2Str, buildOfferKey, isMinPositionTotal } from './matrixHelpers';
import styles from './Step3ComparisonTable.module.scss';
@@ -98,6 +99,8 @@ export function MatrixRowView({ pos, posIndex, matrix, currencyRates, isRestrict
const dispPrice = (tRatio !== null && price !== undefined) ? price / tRatio : price;
const dispPriceKzt = tRatio !== null ? priceKztEffective / tRatio : priceKztEffective;
const showInput = !isReadOnly && !translateUnits;
// Пустая цена — подсветка сразу, пока заполняют; сохранению не мешает (см. isPriceMissing).
const priceMissingCls = !isReadOnly && isPriceMissing(price, item.priceKztOverride) ? styles.missingPriceCell : '';
return (
<React.Fragment key={s.id}>
@@ -160,7 +163,7 @@ export function MatrixRowView({ pos, posIndex, matrix, currencyRates, isRestrict
</td>
{!isRestrictedView && isForeign && (
<>
<td className={[styles.dataCell, valueCls, selCls].filter(Boolean).join(' ')}>
<td className={[styles.dataCell, valueCls, priceMissingCls, selCls].filter(Boolean).join(' ')}>
{showInput ? (
<div className={styles.offerInput}>
<TextInput
@@ -180,7 +183,7 @@ export function MatrixRowView({ pos, posIndex, matrix, currencyRates, isRestrict
</td>
</>
)}
{!isRestrictedView && <td className={[styles.dataCell, valueCls, selCls].filter(Boolean).join(' ')}>
{!isRestrictedView && <td className={[styles.dataCell, valueCls, priceMissingCls, selCls].filter(Boolean).join(' ')}>
{showInput ? (
<div className={styles.offerInput}>
<TextInput
@@ -215,6 +218,8 @@ export function MatrixRowView({ pos, posIndex, matrix, currencyRates, isRestrict
</React.Fragment>
);
})}
{/* Под колонкой «+ Добавить поставщика»: одна ячейка на позицию, включая строки комплекта. */}
{rowIdx === 0 && !isReadOnly && <td className={styles.addSupplierFillerCell} rowSpan={posRowSpan} />}
</tr>
);
}
@@ -866,20 +866,6 @@ $delivery-km-min-width: 76px;
color: $text;
}
.supplierToolbar {
display: flex;
align-items: center;
gap: 8px;
flex-wrap: wrap;
padding: 4px 0 8px;
max-width: 30%;
> :first-child {
flex: 1 1 0;
min-width: 160px;
}
}
.addRow {
display: flex;
align-items: flex-end;
@@ -1349,3 +1335,114 @@ $delivery-km-min-width: 76px;
align-items: center;
gap: 12px;
}
// ─── Незаполненное (решение от 2026-10-27: подсвечивать сразу, пока заполняют) ─────
// Красная рамка — то, что не пускает дальше Шага 3 (см. step3Validation): победитель, километраж,
// условия оплаты (.criteriaInputRequired). Бледная заливка — пустая цена: сохранять не мешает.
// !important — фон выбранной колонки (.colSelected) тоже с !important, а пустая цена в выбранной
// колонке должна оставаться заметной.
.missingPriceCell {
background: $danger-bg !important;
}
.requiredMissingBox {
box-shadow: inset 0 0 0 1px $danger;
border-radius: 2px;
}
.selectSupplierBtnRequired {
border-color: $danger;
color: $danger;
}
// ─── Добавление потенциального поставщика ───────────────────────────────────
// Блок над таблицей (AddSupplierToolbar): раньше без заголовка и с серой кнопкой — его не замечали.
.addSupplierBlock {
display: flex;
flex-direction: column;
gap: 8px;
padding: 12px 16px;
border: 1px solid $accent-border;
border-radius: 4px;
background: $accent-bg;
}
.addSupplierHeading {
display: flex;
flex-wrap: wrap;
align-items: baseline;
gap: 4px 12px;
}
.addSupplierTitle {
margin: 0;
font-size: 14px;
font-weight: 600;
color: $accent-dark;
}
.addSupplierHint {
font-size: 12px;
color: $muted;
}
.addSupplierRow {
display: flex;
flex-wrap: wrap;
align-items: center;
gap: 8px;
}
.addSupplierOr {
font-size: 12px;
color: $muted;
}
.addSupplierError {
font-size: 12px;
color: $danger;
}
.addSupplierDialogBody {
display: flex;
flex-direction: column;
align-items: flex-start;
gap: 12px;
> .addSupplierRow { align-self: stretch; }
}
.addSupplierDivider {
font-size: 12px;
color: $muted;
}
// Последняя колонка таблицы «+ Добавить поставщика» — там, где ищут, как добавить колонку.
.addSupplierHeaderCell {
min-width: 140px;
vertical-align: middle;
text-align: center;
background: $head-bg;
border-left: 2px dashed $accent-border !important;
}
.addSupplierHeaderBtn {
padding: 8px 12px;
border: 1px dashed $accent;
border-radius: 4px;
background: $surface;
color: $accent;
font-family: inherit;
font-size: 13px;
font-weight: 600;
white-space: nowrap;
cursor: pointer;
&:hover { background: $accent-bg; }
}
.addSupplierFillerCell {
border-left: 2px dashed $accent-border !important;
}
@@ -1,6 +1,6 @@
import * as React from 'react';
import { useMemo, useState } from 'react';
import { Button, Alert, TextInput, Toggle, Spinner, Dialog, Icon } from '@tnsu/ui-kit';
import { Button, Alert, Toggle, Spinner, Dialog, Icon } from '@tnsu/ui-kit';
import { FormStep } from '../types/domain';
import { CtService } from '../../services/CtService';
import { VendorDatabaseService, IVendorRecord } from '../../services/VendorDatabaseService';
@@ -19,6 +19,10 @@ import { useAdataCheck } from './useAdataCheck';
import { useRecommendedPick } from './useRecommendedPick';
import { VendorPickerDialog } from './VendorPickerDialog';
import { ExcludedSuppliersList } from './ExcludedSuppliersList';
import { AddSupplierToolbar } from './AddSupplierToolbar';
import { AddSupplierDialog } from './AddSupplierDialog';
import { SaveDraftButton } from '../SaveDraftButton/SaveDraftButton';
import { withoutPlaceholders } from '../utils/placeholderKpFile';
// Исключённых поставщиков в режиме просмотра (согласующий, отправленная СТ) пока не показываем —
// решение от 2026-10-27 (docs/adr/0001). Данные грузятся и так: включить — поменять на true.
@@ -56,8 +60,8 @@ export const Step3ComparisonTable: React.FC<Props> = ({
const matrix = useStep3Matrix({ service, formData, isReadOnly: !canEdit, onSave, onNext, onStepComplete });
const {
isReadOnly, newSupplierName, newSupplierCurrency, setNewSupplierName, setNewSupplierCurrency,
handleAddSupplier, handleAddSupplierFromDatabase, translateUnits, setTranslateUnits, isSaving, saveError, setSaveError,
validateBeforeSave, handleSave,
addManualSupplier, handleAddSupplier, handleAddSupplierFromDatabase, translateUnits, setTranslateUnits, isSaving, saveError, setSaveError,
validateBeforeSave, handleSave, draftSavedAt,
extraPositions, suppliersWithOffer, columnDisplayNames, allPositions, computedOffers, selectedSupplierId,
excludedColumns, handleRestoreSupplier,
} = matrix;
@@ -68,6 +72,8 @@ export const Step3ComparisonTable: React.FC<Props> = ({
// Модалка выбора поставщика из «База данных поставщиков» (см. VendorPickerDialog).
const [showVendorPicker, setShowVendorPicker] = useState(false);
// Окно кнопки «+ Добавить поставщика» в шапке таблицы.
const [showAddSupplierDialog, setShowAddSupplierDialog] = useState(false);
const vendorDbService = useMemo(() => new VendorDatabaseService(service.webPartContext), [service]);
const handleAddVendorFromDatabase = (vendor: IVendorRecord): void => {
const id = handleAddSupplierFromDatabase(vendor);
@@ -115,35 +121,17 @@ export const Step3ComparisonTable: React.FC<Props> = ({
return (
<div className={styles.root}>
{/* Toolbar: add supplier — always visible above the table */}
{/* Добавление потенциального поставщика: блок над таблицей + кнопка в шапке таблицы. */}
{!isReadOnly && (
<div className={styles.supplierToolbar}>
<div className={styles.addInput}>
<TextInput
placeholder="Наименование потенциального поставщика"
value={newSupplierName}
onChange={setNewSupplierName}
ariaLabel="Наименование нового потенциального поставщика"
onKeyDown={e => { if (e.key === 'Enter') handleAddSupplier(); }}
/>
</div>
<select
className={styles.currencySelect}
value={newSupplierCurrency}
onChange={e => setNewSupplierCurrency(e.target.value)}
aria-label="Валюта нового потенциального поставщика"
>
{currencyOptions.map(c => <option key={c} value={c}>{c}</option>)}
</select>
<Button variant="secondary" onClick={handleAddSupplier} disabled={!newSupplierName.trim()}>
<Icon name="plus" size="sm" />
Добавить потенциального поставщика
</Button>
<Button variant="secondary" onClick={() => setShowVendorPicker(true)}>
<Icon name="building" size="sm" />
Добавить из базы поставщиков
</Button>
</div>
<AddSupplierToolbar
name={newSupplierName}
currency={newSupplierCurrency}
currencyOptions={currencyOptions}
onNameChange={setNewSupplierName}
onCurrencyChange={setNewSupplierCurrency}
onAdd={handleAddSupplier}
onOpenDatabase={() => setShowVendorPicker(true)}
/>
)}
{/* Тумблер пересчёта единиц измерения поставщика в ед. изм. заявки */}
@@ -183,7 +171,14 @@ export const Step3ComparisonTable: React.FC<Props> = ({
{recPick.pickError && <Alert variant="err">{recPick.pickError}</Alert>}
<ComparisonMatrix matrix={matrix} currencyRates={currencyRates} isRestrictedView={isRestrictedView} adata={adata} recommendedId={recPick.recommendedId} />
<ComparisonMatrix
matrix={matrix}
currencyRates={currencyRates}
isRestrictedView={isRestrictedView}
adata={adata}
recommendedId={recPick.recommendedId}
onAddSupplierClick={() => setShowAddSupplierDialog(true)}
/>
<CriteriaTable matrix={matrix} />
@@ -201,7 +196,8 @@ export const Step3ComparisonTable: React.FC<Props> = ({
)}
{/* В КП — цены: показываем тем же, кому видны цены (участникам согласования и автору). */}
{!isRestrictedView && <KpFilesList files={formData.uploadedKpFiles ?? []} service={service} />}
{/* Заглушки поставщиков, добавленных вручную, — не КП: не показываем. */}
{!isRestrictedView && <KpFilesList files={withoutPlaceholders(formData.uploadedKpFiles ?? [])} service={service} />}
<AiCommentBlock
comment={ai.aiComment}
@@ -226,6 +222,9 @@ export const Step3ComparisonTable: React.FC<Props> = ({
</Button>
<div className={styles.footerRight}>
{saveError && <Alert variant="err">{saveError}</Alert>}
{!isReadOnly && (
<SaveDraftButton onSave={() => { handleSave('draft').catch(() => undefined /* ошибку показывает handleSave */); }} isSaving={isSaving} savedAt={draftSavedAt} />
)}
<Button
variant="secondary"
onClick={ai.handleGeneratePdfReport}
@@ -247,7 +246,7 @@ export const Step3ComparisonTable: React.FC<Props> = ({
// Сначала проверка (победитель, километраж), потом предложение комментария ИИ.
if (!validateBeforeSave()) return;
if (!ai.aiComment && !ai.isAiLoading) ai.setShowAiPromptDialog(true);
else void handleSave();
else handleSave().catch(() => undefined /* ошибку показывает handleSave */);
}}
disabled={isSaving || isReadOnly}
>
@@ -263,6 +262,15 @@ export const Step3ComparisonTable: React.FC<Props> = ({
</div>
</div>
{showAddSupplierDialog && (
<AddSupplierDialog
currencyOptions={currencyOptions}
onAddManual={addManualSupplier}
onOpenDatabase={() => setShowVendorPicker(true)}
onClose={() => setShowAddSupplierDialog(false)}
/>
)}
{showVendorPicker && (
<VendorPickerDialog
vendorDbService={vendorDbService}
@@ -1,7 +1,34 @@
import { applyCreatedIds, buildOfferKey, computeOffer, getSubCols, isMinPositionTotal, round2Str, sumPositionTotals } from './matrixHelpers';
import { ExtraPosition } from './matrixTypes';
import { applyCreatedIds, applyPositionUnitChange, buildOfferKey, computeOffer, getSubCols, isMinPositionTotal, round2Str, sumPositionTotals } from './matrixHelpers';
import { EditableOffers, ExtraPosition } from './matrixTypes';
import { SupplierOffer } from '../types/domain';
describe('applyPositionUnitChange (единица доп. позиции стала своей вместо «рейс»)', () => {
const edits: EditableOffers = {
'sup_a::extra_pos_1': [{ supplierId: 'sup_a', positionId: 'extra_pos_1', itemIndex: 0, unit: 'рейс', pricePerUnit: 50000 }],
'sup_b::extra_pos_1': [{ supplierId: 'sup_b', positionId: 'extra_pos_1', itemIndex: 0, unit: 'машина' }],
'sup_c::extra_pos_1': [{ supplierId: 'sup_c', positionId: 'extra_pos_1', itemIndex: 0 }],
'sup_a::pos_1': [{ supplierId: 'sup_a', positionId: 'pos_1', itemIndex: 0, unit: 'рейс' }],
};
const result = applyPositionUnitChange(edits, 'extra_pos_1', 'рейс', 'тонна');
it('у поставщика с прежней единицей позиции — новая, цена не теряется', () => {
expect(result['sup_a::extra_pos_1'][0]).toMatchObject({ unit: 'тонна', pricePerUnit: 50000 });
});
it('у поставщика без своей единицы — новая', () => {
expect(result['sup_c::extra_pos_1'][0].unit).toBe('тонна');
});
it('свою единицу поставщика не трогает', () => {
expect(result['sup_b::extra_pos_1'][0].unit).toBe('машина');
});
it('другие позиции не трогает, даже с той же единицей', () => {
expect(result['sup_a::pos_1'][0].unit).toBe('рейс');
});
});
describe('computeOffer', () => {
it('считает сумму в тенге для КП в тенге', () => {
const offer = computeOffer(
@@ -87,6 +87,25 @@ export function buildOfferKey(supplierId: string, positionId: string): string {
return `${supplierId}::${positionId}`;
}
// Поменяли единицу доп. позиции (логистика, доп. расходы): у поставщиков, у которых единица та же,
// что была у позиции (её не трогали), — новая. Свою единицу поставщика не меняем. Ячейки без правок
// и так показывают единицу позиции.
export function applyPositionUnitChange(
edits: EditableOffers,
positionId: string,
oldUnit: string,
newUnit: string,
): EditableOffers {
const next: EditableOffers = {};
Object.keys(edits).forEach(key => {
const isThisPosition = key.split('::')[1] === positionId;
next[key] = isThisPosition
? edits[key].map(item => (!item.unit || item.unit === oldUnit ? { ...item, unit: newUnit } : item))
: edits[key];
});
return next;
}
export interface ColumnOffersInput {
columns: Array<{ id: string; isExtraSupplier: boolean }>;
positions: PositionRow[];
@@ -0,0 +1,55 @@
import { Supplier } from '../types/domain';
import { ExtraSupplier } from './matrixTypes';
import { buildSuppliersPatch } from './step3SavePatch';
const kpColumn: Supplier = { id: '282_320', spId: 282, kpFileSpId: 320, name: 'Каспий Дорс', hasOffer: true };
const extra: ExtraSupplier = { id: 'extra_sup_1', name: 'ООО Новый', currency: 'KZT' };
describe('buildSuppliersPatch', () => {
it('статусы поставщиков — в данные формы в обоих режимах', () => {
for (const mode of ['next', 'draft'] as const) {
const patch = buildSuppliersPatch({
mode, suppliers: [kpColumn], extraSuppliers: [], statuses: { '282_320': 'Производитель' }, extraSupplierSpIds: {},
});
expect(patch.suppliers[0].status).toBe('Производитель');
}
});
it('«Сохранить и продолжить»: поставщик Шага 3 переезжает в suppliers с SP Id, список Шага 3 очищается', () => {
const patch = buildSuppliersPatch({
mode: 'next', suppliers: [kpColumn], extraSuppliers: [extra], statuses: {}, extraSupplierSpIds: { extra_sup_1: 501 },
});
expect(patch.suppliers.map(s => s.id)).toEqual(['282_320', 'extra_sup_1']);
expect(patch.suppliers[1]).toMatchObject({ spId: 501, hasOffer: true });
expect(patch.step3ExtraSuppliers).toEqual([]);
});
// Остаёмся на шаге: таблица держит поставщика в своём списке Шага 3 — перенос задвоил бы колонку.
it('черновик: поставщик Шага 3 остаётся в своём списке, suppliers его не получают', () => {
const patch = buildSuppliersPatch({
mode: 'draft', suppliers: [kpColumn], extraSuppliers: [extra], statuses: {}, extraSupplierSpIds: { extra_sup_1: 501 },
});
expect(patch.suppliers.map(s => s.id)).toEqual(['282_320']);
expect(patch).not.toHaveProperty('step3ExtraSuppliers');
});
it('повторное «Сохранить и продолжить» не дублирует уже перенесённого поставщика', () => {
const alreadyMoved: Supplier = { id: 'extra_sup_1', spId: 501, name: 'ООО Новый', hasOffer: true };
const patch = buildSuppliersPatch({
mode: 'next', suppliers: [kpColumn, alreadyMoved], extraSuppliers: [extra], statuses: {}, extraSupplierSpIds: { extra_sup_1: 501 },
});
expect(patch.suppliers.filter(s => s.id === 'extra_sup_1')).toHaveLength(1);
});
it('исключение поставщика Шага 3 переезжает вместе с ним', () => {
const patch = buildSuppliersPatch({
mode: 'next', suppliers: [], extraSuppliers: [{ ...extra, exclusion: { reason: 'Дорого' } }], statuses: {}, extraSupplierSpIds: {},
});
expect(patch.suppliers[0].exclusion).toEqual({ reason: 'Дорого' });
});
});
@@ -0,0 +1,49 @@
import { Supplier } from '../types/domain';
import { ExtraSupplier } from './matrixTypes';
// «Сохранить и продолжить» — переход на Шаг 4; «Сохранить черновик» — запись в SP, остаёмся на Шаге 3.
export type Step3SaveMode = 'next' | 'draft';
export interface SuppliersPatchInput {
mode: Step3SaveMode;
suppliers: Supplier[];
extraSuppliers: ExtraSupplier[];
statuses: Record<string, string>;
// SP Id поставщиков, добавленных на Шаге 3 (созданы при этом сохранении или раньше).
extraSupplierSpIds: Record<string, number>;
}
export interface SuppliersPatch {
suppliers: Supplier[];
// Есть только при переходе на Шаг 4: поставщики Шага 3 переезжают в suppliers.
step3ExtraSuppliers?: ExtraSupplier[];
}
// Поставщики в данных формы после сохранения Шага 3.
//
// При переходе на Шаг 4 поставщики, добавленные на Шаге 3, переезжают в suppliers (с тем же id —
// по нему к ним привязаны цены в сессии), а список Шага 3 очищается: таблица размонтируется и
// при возврате соберётся заново.
//
// Черновик оставляет нас на шаге, и таблица по-прежнему держит их в своём списке Шага 3. Перенос
// задвоил бы колонку (она в обоих списках), а без списка Шага 3 у такого поставщика пропала бы
// подстановка количества из позиции — суммы обнулились бы. Поэтому черновик их не переносит:
// SP Id уже запомнены (rememberCreatedExtras), повторное сохранение дублей в SP не создаст.
export function buildSuppliersPatch(input: SuppliersPatchInput): SuppliersPatch {
const { mode, suppliers, extraSuppliers, statuses, extraSupplierSpIds } = input;
const withStatuses = suppliers.map(s => ({ ...s, status: statuses[s.id] ?? s.status }));
if (mode === 'draft') return { suppliers: withStatuses };
const movedExtras: Supplier[] = extraSuppliers
// Уже в suppliers после прежнего сохранения — не дублируем колонку.
.filter(es => !suppliers.some(s => s.id === es.id))
.map(es => ({
id: es.id,
spId: extraSupplierSpIds[es.id] ?? es.spId,
name: es.name,
hasOffer: true,
status: statuses[es.id] ?? '',
exclusion: es.exclusion,
}));
return { suppliers: [...withStatuses, ...movedExtras], step3ExtraSuppliers: [] };
}
@@ -1,4 +1,4 @@
import { STEP3_SAVE_ERRORS, getStep3SaveError, toggleWinner } from './step3Validation';
import { STEP3_SAVE_ERRORS, getStep3SaveError, isPriceMissing, toggleWinner } from './step3Validation';
describe('toggleWinner', () => {
it('нажали «Выбрать» — поставщик выбран', () => {
@@ -19,23 +19,44 @@ describe('getStep3SaveError', () => {
const compared = ['259_278', '282_320'];
it('победитель выбран — сохранять можно', () => {
expect(getStep3SaveError({ selectedSupplierId: '259_278', comparedColumnIds: compared, hasDeliveryWithoutKm: false }))
expect(getStep3SaveError({ selectedSupplierId: '259_278', comparedColumnIds: compared, hasMissingPaymentTerms: false, hasDeliveryWithoutKm: false }))
.toBeUndefined();
});
it('победитель не выбран — нельзя', () => {
expect(getStep3SaveError({ selectedSupplierId: undefined, comparedColumnIds: compared, hasDeliveryWithoutKm: false }))
expect(getStep3SaveError({ selectedSupplierId: undefined, comparedColumnIds: compared, hasMissingPaymentTerms: false, hasDeliveryWithoutKm: false }))
.toBe(STEP3_SAVE_ERRORS.noWinner);
});
// Выбор из сессии мог остаться на поставщике, которого исключили или чей КП убрали.
it('выбран поставщик не из сравниваемых — как будто не выбран', () => {
expect(getStep3SaveError({ selectedSupplierId: '300_400', comparedColumnIds: compared, hasDeliveryWithoutKm: false }))
expect(getStep3SaveError({ selectedSupplierId: '300_400', comparedColumnIds: compared, hasMissingPaymentTerms: false, hasDeliveryWithoutKm: false }))
.toBe(STEP3_SAVE_ERRORS.noWinner);
});
it('логистика без километража — нельзя', () => {
expect(getStep3SaveError({ selectedSupplierId: '259_278', comparedColumnIds: compared, hasDeliveryWithoutKm: true }))
expect(getStep3SaveError({ selectedSupplierId: '259_278', comparedColumnIds: compared, hasMissingPaymentTerms: false, hasDeliveryWithoutKm: true }))
.toBe(STEP3_SAVE_ERRORS.deliveryKmMissing);
});
// Решение от 2026-10-27: «Условия оплаты *» были помечены обязательными, но сохранять не мешали.
it('условия оплаты не у всех — нельзя', () => {
expect(getStep3SaveError({ selectedSupplierId: '259_278', comparedColumnIds: compared, hasMissingPaymentTerms: true, hasDeliveryWithoutKm: false }))
.toBe(STEP3_SAVE_ERRORS.paymentTermsMissing);
});
});
describe('isPriceMissing', () => {
it('цены нет ни в валюте, ни в ₸ — пусто', () => {
expect(isPriceMissing(undefined, undefined)).toBe(true);
expect(isPriceMissing(0, undefined)).toBe(true);
});
it('цена в валюте КП есть — не пусто', () => {
expect(isPriceMissing(635, undefined)).toBe(false);
});
it('цену ввели вручную в ₸ — не пусто', () => {
expect(isPriceMissing(undefined, 286575.5)).toBe(false);
});
});
@@ -2,6 +2,7 @@
export const STEP3_SAVE_ERRORS = {
noWinner: 'Выберите поставщика-победителя: кнопка «Выбрать» в шапке его колонки.',
paymentTermsMissing: 'Заполните условия оплаты у всех поставщиков в «Дополнительной информации».',
deliveryKmMissing: 'Укажите километраж для каждого потенциального поставщика в строке логистики',
};
@@ -9,21 +10,32 @@ export interface Step3SaveCheckInput {
selectedSupplierId: string | undefined;
// Колонки, которые сравниваются (без исключённых).
comparedColumnIds: string[];
hasMissingPaymentTerms: boolean;
hasDeliveryWithoutKm: boolean;
}
// Победитель обязателен (решение от 2026-10-27): он уходит в шапку СТ (winSupplier) — без него
// отчёт и согласование не знают, кого выбрали. Выбранный должен быть среди сравниваемых:
// исключённый или удалённый поставщик победителем не считается.
// Что не пускает дальше Шага 3 (решения от 2026-10-27). Всё это подсвечено красной рамкой сразу,
// пока заполняют, — подсветка показывает ровно то, что проверяется здесь.
// - Победитель: уходит в шапку СТ (winSupplier) — без него отчёт и согласование не знают, кого
// выбрали. Должен быть среди сравниваемых: исключённый поставщик победителем не считается.
// - Условия оплаты: по ним считается рекомендация (80% цена / 20% условия оплаты).
// - Километраж логистики.
export function getStep3SaveError(input: Step3SaveCheckInput): string | undefined {
const { selectedSupplierId, comparedColumnIds, hasDeliveryWithoutKm } = input;
const { selectedSupplierId, comparedColumnIds, hasMissingPaymentTerms, hasDeliveryWithoutKm } = input;
if (!selectedSupplierId || !comparedColumnIds.includes(selectedSupplierId)) {
return STEP3_SAVE_ERRORS.noWinner;
}
if (hasMissingPaymentTerms) return STEP3_SAVE_ERRORS.paymentTermsMissing;
if (hasDeliveryWithoutKm) return STEP3_SAVE_ERRORS.deliveryKmMissing;
return undefined;
}
// Цена в ячейке не введена: ни в валюте КП, ни вручную в ₸. Подсвечивается, но сохранению не
// мешает — поставщик мог не предложить позицию, и 0 вместо пустоты исказил бы сравнение.
export function isPriceMissing(price: number | undefined, priceKztOverride: number | undefined): boolean {
return !(price !== undefined && price > 0) && !(priceKztOverride !== undefined && priceKztOverride > 0);
}
// Кнопка «Выбрать» / «Выбран»: повторное нажатие на выбранного снимает выбор.
export function toggleWinner(currentId: string | undefined, clickedId: string): string | undefined {
return currentId === clickedId ? undefined : clickedId;
@@ -4,10 +4,11 @@ import { SupplierOffer, CurrencyRate, FormStep, SupplierExclusion } from '../typ
import { CtService } from '../../services/CtService';
import { CTFormData } from '../hooks/useCtForm';
import { ExtraSupplier, ExtraPosition, PositionRow, CellEdit, EditableOffers, CriteriaEntry } from './matrixTypes';
import { applyCreatedIds, buildOfferKey, computeColumnsOffers, sumPositionTotals } from './matrixHelpers';
import { applyCreatedIds, applyPositionUnitChange, buildOfferKey, computeColumnsOffers, sumPositionTotals } from './matrixHelpers';
import { buildExcludedRecordsForSave } from '../utils/excludedColumns';
import { parseNumberInput } from '../utils/parseNumberInput';
import { getStep3SaveError } from './step3Validation';
import { Step3SaveMode, buildSuppliersPatch } from './step3SavePatch';
import { IVendorRecord } from '../../services/VendorDatabaseService';
import { buildColumnDisplayNames, buildColumnLabels, findColumnForRecord } from '../utils/supplierColumns';
import { getNoOfferSuppliers } from '../utils/noOfferSuppliers';
@@ -106,6 +107,8 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
}, [onSave]);
const [isSaving, setIsSaving] = useState(false);
const [saveError, setSaveError] = useState<string | undefined>(undefined);
// Когда последний раз сохранили черновик — показываем рядом с кнопкой.
const [draftSavedAt, setDraftSavedAt] = useState<Date | undefined>(undefined);
// Тумблер «Перевести в ед. изм. заявки»: переводит кол-во и цену поставщика в ед. заявки.
// При включении блоки поставщиков показываются как вычисленные значения (без редактирования).
const [translateUnits, setTranslateUnits] = useState(false);
@@ -584,16 +587,23 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
}
};
const handleAddSupplier = (): void => {
if (!newSupplierName.trim()) return;
// Поставщик, добавленный на Шаге 3 вручную (без КП). Общая точка для обеих кнопок добавления:
// блока над таблицей и колонки «+ Добавить поставщика» в самой таблице.
const addManualSupplier = (name: string, currency: string): void => {
if (isReadOnly || !name.trim()) return;
const id = `extra_sup_${++extraIdCounter}`;
const newEntry = { id, name: newSupplierName.trim(), currency: newSupplierCurrency };
const updated = [...extraSuppliers, newEntry];
const updated = [...extraSuppliers, { id, name: name.trim(), currency }];
setExtraSuppliers(updated);
setSupplierStatuses(prev => ({ ...prev, [id]: '' }));
onSave({ step3ExtraSuppliers: updated });
};
// Блок над таблицей: добавить из своих полей и очистить их.
const handleAddSupplier = (): void => {
if (!newSupplierName.trim()) return;
addManualSupplier(newSupplierName, newSupplierCurrency);
setNewSupplierName('');
setNewSupplierCurrency('KZT');
onSave({ step3ExtraSuppliers: updated });
};
// Добавление поставщика из VendorPickerDialog («База данных поставщиков»): та же механика
@@ -678,6 +688,17 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
onSave({ step3DeliveryKm: updated });
};
// Своя единица доп. позиции вместо заданной при добавлении (у логистики — «рейс»). Поставщики, у
// которых единицу не меняли, получают новую. В SP уходит при «Сохранить» (см. handleSave).
const handleExtraPositionUnitChange = (positionId: string, unit: string): void => {
if (isReadOnly) return;
const oldUnit = extraPositions.find(ep => ep.id === positionId)?.unit ?? '';
const updated = extraPositions.map(ep => (ep.id === positionId ? { ...ep, unit } : ep));
setExtraPositions(updated);
setEdits(prev => applyPositionUnitChange(prev, positionId, oldUnit, unit));
onSave({ step3ExtraPositions: updated });
};
const handleDeleteExtraPosition = (positionId: string): void => {
if (isReadOnly) return;
const updated = extraPositions.filter(ep => ep.id !== positionId);
@@ -718,6 +739,7 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
const error = getStep3SaveError({
selectedSupplierId,
comparedColumnIds: suppliersWithOffer.map(s => s.id),
hasMissingPaymentTerms: suppliersWithOffer.some(s => !(criteriaMap[s.id]?.paymentType ?? '').trim()),
hasDeliveryWithoutKm: extraPositions.some(ep =>
ep.isDelivery && suppliersWithOffer.some(s => !deliveryKm[`${ep.id}::${s.id}`])
),
@@ -726,9 +748,11 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
return error === undefined;
};
const handleSave = async (): Promise<void> => {
// Запись Шага 3 в SP. 'next' — с проверками и переходом на Шаг 4; 'draft' — без проверок,
// остаёмся на шаге (черновик: сохранить недоделанное, чтобы не потерять при закрытии вкладки).
const handleSave = async (mode: Step3SaveMode = 'next'): Promise<void> => {
if (!ct) return;
if (!validateBeforeSave()) return;
if (mode === 'next' && !validateBeforeSave()) return;
setIsSaving(true);
setSaveError(undefined);
@@ -754,6 +778,9 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
if (ep.spId) {
// Already created in SP on a previous save — reuse to avoid duplicate Позиции СТ rows.
extraPosSpIds[ep.id] = ep.spId;
// Единицу могли поменять после создания. Пишем всегда — строк доп. расходов единицы,
// а отслеживать «менялась ли» — лишнее состояние.
await service.updateCTPositionUnit(ep.spId, ep.unit);
continue;
}
const saved = await service.createCTPositions(ct.id, [{
@@ -880,28 +907,15 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
onSave({
offers: offersForSession,
suppliers: [
...suppliers.map(s => ({
...s,
status: supplierStatuses[s.id] ?? s.status,
})),
// Move extras into suppliers (with React ID kept as `id` — required so formData.offers
// entries with React IDs still match sup.id on remount). Skip extras that are already in
// formData.suppliers from a prior save (avoids duplicate columns).
...extraSuppliers
.filter(es => !suppliers.some(s => s.id === es.id))
.map(es => ({
id: es.id,
spId: extraSupSpIds[es.id],
name: es.name,
hasOffer: true as const,
status: supplierStatuses[es.id] ?? '',
exclusion: es.exclusion,
})),
],
// Поставщики Шага 3 переезжают в suppliers только при переходе на Шаг 4 (см. buildSuppliersPatch).
...buildSuppliersPatch({
mode,
suppliers,
extraSuppliers,
statuses: supplierStatuses,
extraSupplierSpIds: extraSupSpIds,
}),
recommendedSupplierId: selectedSupplierId,
// Extras moved into `suppliers` — clear extras state so `suppliersWithOffer` doesn't duplicate columns.
step3ExtraSuppliers: [],
// Positions don't migrate into selectedPositions — keep them in extras state so the
// "доп.расходы" rows continue to render after Step 4 → breadcrumb back. Track spId to
// skip re-creating SP rows on subsequent saves.
@@ -918,6 +932,10 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
step3CriteriaEdits: criteriaMap,
step3CustomCriteria: customCriteria,
});
if (mode === 'draft') {
setDraftSavedAt(new Date());
return;
}
saveCompletedRef.current = true;
onStepComplete(3);
onNext();
@@ -989,14 +1007,17 @@ export function useStep3Matrix(args: UseStep3MatrixArgs) {
handleRemoveCustomCriteria,
handleNameColResizeMouseDown,
handleStatusChange,
addManualSupplier,
handleAddSupplier,
handleAddSupplierFromDatabase,
handleAddPosition,
handleAddDelivery,
handleDeliveryKmChange,
handleDeleteExtraPosition,
handleExtraPositionUnitChange,
validateBeforeSave,
handleSave,
draftSavedAt,
};
}
@@ -2,6 +2,7 @@ import { useEffect, useState } from 'react';
import { CtService } from '../../services/CtService';
import { SigexSignService, SigexSignStep } from '../../services/SigexSignService';
import { CTFormData } from '../hooks/useCtForm';
import { withoutPlaceholders } from '../utils/placeholderKpFile';
// Этапы выгрузки пакета в INCORE: формирование отчёта + этапы подписания SIGEX.
export type IncoreStep = 'report' | SigexSignStep;
@@ -72,9 +73,10 @@ export function useIncore(args: UseIncoreArgs) {
setCardAttachment(saved);
// 4. Пакет для INCORE: карточка с ЭЦП + файлы КП — скачиваем пользователю.
// Заглушки поставщиков, добавленных вручную, — не КП, в пакет не идут.
service.downloadBlob(cardBlob, incoreCardName);
const kpFiles = formData.uploadedKpFiles ?? [];
const kpFiles = withoutPlaceholders(formData.uploadedKpFiles ?? []);
for (const f of kpFiles) {
await service.downloadKpFile(f.name);
await new Promise<void>((resolve) => setTimeout(resolve, 300));
@@ -0,0 +1,27 @@
import { isPlaceholderKpFile, withoutPlaceholders } from './placeholderKpFile';
describe('isPlaceholderKpFile', () => {
// СТ 375: поставщик «fghfg» добавлен на Шаге 3 — заглушка видна в «Коммерческих предложениях».
it('заглушка поставщика, добавленного вручную', () => {
expect(isPlaceholderKpFile('fghfg_1793100061349.txt')).toBe(true);
expect(isPlaceholderKpFile('ООО Альфа_1793100061349.TXT')).toBe(true);
});
it('настоящие КП — не заглушки', () => {
expect(isPlaceholderKpFile('372_KP_SteelGate_Export_USD.pdf')).toBe(false);
expect(isPlaceholderKpFile('372_KP_TOO_Kaspiy_Dors.docx')).toBe(false);
expect(isPlaceholderKpFile('КП.doc')).toBe(false);
});
});
describe('withoutPlaceholders', () => {
it('оставляет только настоящие КП, порядок сохраняется', () => {
const files = [
{ spId: 318, name: '372_KP_SteelGate.pdf' },
{ spId: 322, name: 'fghfg_1793100061349.txt' },
{ spId: 320, name: '372_KP_Kaspiy.docx' },
];
expect(withoutPlaceholders(files).map(f => f.spId)).toEqual([318, 320]);
});
});
@@ -0,0 +1,18 @@
// Файл-заглушка КП: создаётся для поставщика, добавленного на Шаге 3 вручную, — оффер в SP должен
// ссылаться на файл КП (CtService._createPlaceholderKpFile). Людям он не нужен: в списках КП и в
// пакете INCORE его не показываем. Узнаём по расширению: на Шаге 2 загрузить можно только
// .pdf/.docx/.doc (FileUploadArea), а заглушка всегда .txt.
const PLACEHOLDER_EXTENSION = '.txt';
// Подпись колонки поставщика, у которого вместо КП — заглушка.
export const PLACEHOLDER_COLUMN_LABEL = 'Добавлен вручную';
export function isPlaceholderKpFile(fileName: string): boolean {
return fileName.toLowerCase().endsWith(PLACEHOLDER_EXTENSION);
}
// Настоящие КП — для показа и выгрузки. Данные формы (uploadedKpFiles) не фильтруем: по ним
// сохранение находит файл КП колонки, в том числе заглушку.
export function withoutPlaceholders<T extends { name: string }>(files: T[]): T[] {
return files.filter(file => !isPlaceholderKpFile(file.name));
}
@@ -1,6 +1,31 @@
import { buildColumnDisplayNames, buildColumnLabels, buildColumnsFromSpOffers, buildSupplierColumnId, findColumnForRecord, findSiblingColumnIds, findWinnerColumnId } from './supplierColumns';
import { buildColumnDisplayNames, buildColumnLabels, buildColumnsFromSpOffers, buildSupplierColumnId, findColumnForRecord, findSiblingColumnIds, findWinnerColumnId, withoutSessionExtras } from './supplierColumns';
import { Supplier } from '../types/domain';
describe('withoutSessionExtras (поставщик Шага 3 после «Сохранить черновик»)', () => {
// СТ 375: единственный поставщик «fghfg» добавлен на Шаге 3, сохранён черновиком. При открытии из
// реестра колонка собиралась из SP («285_322», без цен) и ещё раз приходила из сессии (с ценами).
const fromSp: Supplier[] = [
{ id: '285_322', spId: 285, kpFileSpId: 322, name: 'fghfg', hasOffer: true },
{ id: '282_320', spId: 282, kpFileSpId: 320, name: 'Каспий Дорс', hasOffer: true },
];
it('колонку поставщика Шага 3 из SP не добавляет — она уже есть в сессии', () => {
const result = withoutSessionExtras(fromSp, [{ spId: 285, offerId: 322 }]);
expect(result.map(c => c.id)).toEqual(['282_320']);
});
it('другой КП того же поставщика — отдельная колонка, остаётся', () => {
const result = withoutSessionExtras(fromSp, [{ spId: 285, offerId: 999 }]);
expect(result.map(c => c.id)).toEqual(['285_322', '282_320']);
});
it('поставщик Шага 3 ещё не сохранён в SP — ничего не убирает', () => {
expect(withoutSessionExtras(fromSp, [{}])).toHaveLength(2);
});
});
describe('findWinnerColumnId (победитель из шапки СТ при открытии без сессии)', () => {
const columns: Supplier[] = [
{ id: '281_318', spId: 281, kpFileSpId: 318, name: 'SteelGate', hasOffer: true, exclusion: { reason: 'Дорого' } },
@@ -68,6 +93,19 @@ describe('buildColumnsFromSpOffers (открытие по ссылке / без
});
});
describe('подпись колонки с файлом-заглушкой', () => {
// СТ 375: в шапке колонки показывалось имя заглушки «fghfg_1793100061349.txt».
it('вместо имени заглушки — «Добавлен вручную»', () => {
const columns: Supplier[] = [
{ id: '285_322', spId: 285, kpFileSpId: 322, name: 'fghfg', hasOffer: true },
{ id: '285_330', spId: 285, kpFileSpId: 330, name: 'fghfg', hasOffer: true },
];
const files = [{ spId: 322, name: 'fghfg_1793100061349.txt' }, { spId: 330, name: '375_KP_fghfg.pdf' }];
expect(buildColumnLabels(columns, files)).toEqual({ '285_322': 'Добавлен вручную', '285_330': '375_KP_fghfg.pdf' });
});
});
describe('подписи колонок поставщика с несколькими КП', () => {
const ARLAN = 'ТОО «Арлан Офис Снаб»';
const columns: Supplier[] = [
@@ -1,4 +1,5 @@
import { Supplier } from '../types/domain';
import { PLACEHOLDER_COLUMN_LABEL, isPlaceholderKpFile } from './placeholderKpFile';
// Колонка таблицы сравнения = поставщик + файл КП: у одного поставщика может быть несколько
// предложений (напр. в тенге и в долларах). Разделитель «_», а не «::» — «::» уже служит
@@ -44,6 +45,22 @@ export function buildColumnsFromSpOffers(records: SpOfferColumnSource[]): Suppli
return Array.from(columnsById.values());
}
// Поставщик, добавленный на Шаге 3, в сессии: SP Id поставщика и его файла-заглушки КП.
export interface SessionExtraColumn {
spId?: number;
offerId?: number;
}
// Колонки из SP без тех, что в сессии уже есть поставщиком Шага 3. «Сохранить черновик» оставляет
// такого поставщика в списке Шага 3 (см. buildSuppliersPatch), а в SP он уже лежит колонкой
// «поставщик_заглушка». Без этого при открытии СТ, где других поставщиков нет, колонка задваивалась:
// одна из SP без цен, вторая из сессии с ценами.
export function withoutSessionExtras(spColumns: Supplier[], extras: SessionExtraColumn[]): Supplier[] {
return spColumns.filter(column => !extras.some(extra =>
extra.spId !== undefined && extra.spId === column.spId && extra.offerId === column.kpFileSpId,
));
}
// Колонка победителя по winSupplier из шапки СТ (СТ открыта без сессии). В SP хранится поставщик,
// а не колонка: у поставщика с несколькими КП берём первую неисключённую колонку.
export function findWinnerColumnId(columns: Supplier[], winSupplierSpId: number | undefined): string | undefined {
@@ -75,6 +92,11 @@ export function buildColumnLabels(
if (group.length < 2) return;
group.forEach((c, index) => {
const fileName = uploadedKpFiles.find(f => f.spId === c.kpFileSpId)?.name;
// Имя файла-заглушки (поставщик добавлен вручную) людям ничего не говорит.
if (fileName !== undefined && isPlaceholderKpFile(fileName)) {
labels[c.id] = PLACEHOLDER_COLUMN_LABEL;
return;
}
labels[c.id] = fileName ?? `КП ${index + 1}`;
});
});
@@ -397,6 +397,21 @@ export class CtService {
}));
}
// Единица доп. позиции Шага 3 (логистика, доп. расходы) — её можно поменять после создания.
// MERGE на «Позиции СТ» ресивер не блокирует (блокирует только на «Оценка поставщиков»).
async updateCTPositionUnit(positionSpId: number, unit: string): Promise<void> {
const res = await this.http.fetch(this._liById(LISTS.CT_POSITIONS, positionSpId), SPHttpClient.configurations.v1, {
method: "POST",
headers: {
"Content-Type": "application/json;odata=nometadata",
"IF-MATCH": "*",
"X-HTTP-Method": "MERGE",
},
body: JSON.stringify({ unit }),
});
if (!res.ok) throw new Error(`Ошибка обновления единицы позиции: ${res.status}`);
}
// Пакетное создание элементов списка через SharePoint OData $batch: вместо N POST —
// один HTTP-запрос (до 100 операций/changeset, при большем числе бьём на чанки).
// Возвращает Id созданных элементов В ТОМ ЖЕ порядке, что и bodies. Changeset атомарен: