fix(money-input): уважать AllowFractionalPrices в формах редактирования
Some checks are pending
CI / POS (WPF, Windows) (push) Waiting to run
CI / Backend (.NET 8) (push) Successful in 27s
CI / Web (React + Vite) (push) Successful in 24s
Docker Images / API image (push) Successful in 33s
Docker Images / Web image (push) Successful in 26s
Docker Images / Deploy stage (push) Successful in 18s

MoneyInput теперь сам читает useOrgSettings().allowFractionalPrices,
а не только полагается на prop из вызова. Это закрывает два бага:

1. Когда настройка известна и запрещает дробное, но в state товара
   лежит дробная цена (например исторические данные из MoySklad) —
   useEffect синхронизирует округлённое значение наружу, чтобы
   при сохранении не уходило значение, которого юзер не видит.

2. Пока org.data ещё не загружено, MoneyInput не режет дробь
   (fractional трактуется как true до приезда настройки), иначе
   в момент гидрации формы из ProductDto с дробной ценой компонент
   успевал её обрезать до того как настройка приходила.

Все вызовы MoneyInput в ProductEdit / SupplyEdit / RetailSaleEdit /
ProductsPage filters очищены от избыточного prop allowFractional —
компонент берёт настройку сам. Override через prop остаётся доступным.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
nns 2026-04-25 12:30:00 +05:00
parent 1ee4b84e53
commit 52a420ea3d
5 changed files with 37 additions and 19 deletions

View file

@ -1,5 +1,6 @@
import type { InputHTMLAttributes, SelectHTMLAttributes, ReactNode, TextareaHTMLAttributes } from 'react'
import { useEffect, type InputHTMLAttributes, type SelectHTMLAttributes, type ReactNode, type TextareaHTMLAttributes } from 'react'
import { cn } from '@/lib/utils'
import { useOrgSettings } from '@/lib/useOrgSettings'
interface FieldProps {
label: string
@ -38,7 +39,8 @@ interface MoneyInputProps {
onChange: (v: number | null) => void
currencyCode?: string | null
currencySymbol?: string | null
/** Разрешать ли две цифры после запятой. По умолчанию false (целые). */
/** Если задан переопределяет настройку организации. Если не задан
* читается из useOrgSettings().allowFractionalPrices. */
allowFractional?: boolean
disabled?: boolean
placeholder?: string
@ -47,34 +49,50 @@ interface MoneyInputProps {
/** Денежное поле: только цифры (+ точка/запятая если allowFractional),
* справа символ валюты (/$/). При allowFractional=false дробная часть
* отбрасывается на лету и отображается целым; иначе два знака после
* запятой. Пустое поле null. */
* отбрасывается на лету и сразу синхронизируется в state наружу; иначе
* две цифры после запятой. Пустое поле null. */
export function MoneyInput({
value, onChange, currencyCode, currencySymbol, allowFractional = false,
value, onChange, currencyCode, currencySymbol, allowFractional,
disabled, placeholder = '0', className,
}: MoneyInputProps) {
const org = useOrgSettings()
// Пока org.data не загружен и prop не задан — фактическое поведение
// оставляем «как сейчас в state», чтобы не резать дробь до приезда настройки.
const settingKnown = allowFractional !== undefined || org.data !== undefined
const fractional = allowFractional ?? org.data?.allowFractionalPrices ?? true
// Когда настройка известна и она запрещает дробное, а в state лежит дробь —
// синхронизируем округлённое значение наружу. Это нужно чтобы при сохранении
// не уходило значение, которое юзер не видит на экране.
useEffect(() => {
if (!settingKnown || fractional) return
if (value == null) return
const rounded = Math.round(value)
if (rounded !== value) onChange(rounded)
}, [settingKnown, fractional, value, onChange])
const suffix = currencySymbol || currencyCode || '₸'
const display = value == null ? '' : (allowFractional ? String(value) : String(Math.round(value)))
const display = value == null ? '' : (fractional ? String(value) : String(Math.round(value)))
return (
<div className={cn('relative', className)}>
<input
type="text"
inputMode={allowFractional ? 'decimal' : 'numeric'}
inputMode={fractional ? 'decimal' : 'numeric'}
disabled={disabled}
value={display}
placeholder={placeholder}
onFocus={(e) => e.currentTarget.select()}
onChange={(e) => {
let raw = e.target.value.replace(',', '.')
raw = allowFractional ? raw.replace(/[^\d.]/g, '') : raw.replace(/[^\d]/g, '')
raw = fractional ? raw.replace(/[^\d.]/g, '') : raw.replace(/[^\d]/g, '')
if (raw === '') return onChange(null)
if (allowFractional) {
if (fractional) {
const parts = raw.split('.')
raw = parts.length > 1 ? parts[0] + '.' + parts.slice(1).join('') : raw
}
const n = Number(raw)
if (!Number.isFinite(n)) return
onChange(allowFractional ? n : Math.round(n))
onChange(fractional ? n : Math.round(n))
}}
className={cn(inputClass, 'pr-10 text-right tabular-nums')}
/>

View file

@ -323,7 +323,7 @@ export function ProductEditPage() {
onChange={(n) => setForm({ ...form, purchasePrice: n == null ? '' : String(n) })}
currencyCode={currencies.data?.find((c) => c.id === form.purchaseCurrencyId)?.code ?? org.data?.defaultCurrencyCode ?? undefined}
currencySymbol={currencies.data?.find((c) => c.id === form.purchaseCurrencyId)?.symbol ?? org.data?.defaultCurrencySymbol ?? undefined}
allowFractional={org.data?.allowFractionalPrices ?? false}
/>
</Field>
{org.data?.multiCurrencyEnabled && (
@ -379,7 +379,7 @@ export function ProductEditPage() {
onChange={(n) => updatePrice(i, { amount: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === p.currencyId)?.code ?? org.data?.defaultCurrencyCode ?? undefined}
currencySymbol={currencies.data?.find((c) => c.id === p.currencyId)?.symbol ?? org.data?.defaultCurrencySymbol ?? undefined}
allowFractional={org.data?.allowFractionalPrices ?? false}
/>
</div>
{org.data?.multiCurrencyEnabled && (

View file

@ -230,7 +230,7 @@ export function ProductsPage() {
onChange={(n) => { setFilters({ ...filters, purchasePriceFrom: n }); setPage(1) }}
currencyCode={org.data?.defaultCurrencyCode ?? undefined}
currencySymbol={org.data?.defaultCurrencySymbol ?? undefined}
allowFractional={org.data?.allowFractionalPrices ?? false}
placeholder="от"
/>
</div>
@ -240,7 +240,7 @@ export function ProductsPage() {
onChange={(n) => { setFilters({ ...filters, purchasePriceTo: n }); setPage(1) }}
currencyCode={org.data?.defaultCurrencyCode ?? undefined}
currencySymbol={org.data?.defaultCurrencySymbol ?? undefined}
allowFractional={org.data?.allowFractionalPrices ?? false}
placeholder="до"
/>
</div>

View file

@ -289,14 +289,14 @@ export function RetailSaleEditPage() {
onChange={(n) => setForm({ ...form, paidCash: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === form.currencyId)?.code}
currencySymbol={currencies.data?.find((c) => c.id === form.currencyId)?.symbol}
allowFractional={org.data?.allowFractionalPrices ?? false} />
/>
</Field>
<Field label="Получено картой">
<MoneyInput value={form.paidCard} disabled={isPosted}
onChange={(n) => setForm({ ...form, paidCard: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === form.currencyId)?.code}
currencySymbol={currencies.data?.find((c) => c.id === form.currencyId)?.symbol}
allowFractional={org.data?.allowFractionalPrices ?? false} />
/>
</Field>
<Field label="Примечание" className="md:col-span-3">
<TextArea rows={2} value={form.notes} disabled={isPosted}
@ -348,7 +348,7 @@ export function RetailSaleEditPage() {
onChange={(n) => updateLine(i, { unitPrice: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === form.currencyId)?.code}
currencySymbol={currencies.data?.find((c) => c.id === form.currencyId)?.symbol}
allowFractional={org.data?.allowFractionalPrices ?? false} />
/>
</td>
<td className="py-2 px-3">
<MoneyInput disabled={isPosted}
@ -356,7 +356,7 @@ export function RetailSaleEditPage() {
onChange={(n) => updateLine(i, { discount: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === form.currencyId)?.code}
currencySymbol={currencies.data?.find((c) => c.id === form.currencyId)?.symbol}
allowFractional={org.data?.allowFractionalPrices ?? false} />
/>
</td>
<td className="py-2 px-3 text-right font-mono font-semibold">
{lineTotal(l).toLocaleString('ru', { maximumFractionDigits: 2 })}

View file

@ -322,7 +322,7 @@ export function SupplyEditPage() {
onChange={(n) => updateLine(i, { unitPrice: n ?? 0 })}
currencyCode={currencies.data?.find((c) => c.id === form.currencyId)?.code}
currencySymbol={currencies.data?.find((c) => c.id === form.currencyId)?.symbol}
allowFractional={org.data?.allowFractionalPrices ?? false} />
/>
</td>
<td className="py-2 px-3 text-right font-mono font-semibold">
{lineTotal(l).toLocaleString('ru', { maximumFractionDigits: 2 })}