From 888a02f39722c68e4beddefbd1e39c402dfa51cb Mon Sep 17 00:00:00 2001
From: kewldan <kewldanil1@gmail.com>
Date: Tue, 28 Jul 2026 21:23:08 +0300
Subject: [PATCH] =?UTF-8?q?fix(subscription):=20=D0=BA=D0=BE=D1=80=D1=80?=
 =?UTF-8?q?=D0=B5=D0=BA=D1=82=D0=BD=D0=B0=D1=8F=20=D1=86=D0=B5=D0=BD=D0=B0?=
 =?UTF-8?q?=20=D0=B7=D0=B0=20=D0=BC=D0=B5=D1=81=D1=8F=D1=86=20=D0=BF=D1=80?=
 =?UTF-8?q?=D0=B8=20=D0=B2=D1=8B=D0=B1=D0=BE=D1=80=D0=B5=20=D0=BF=D0=B5?=
 =?UTF-8?q?=D1=80=D0=B8=D0=BE=D0=B4=D0=B0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Для периодов короче месяца в карточке периода под ценой выводилась
та же сумма с подписью «/мес»: цена за 7 дней показывалась как цена
за месяц. Причина — деление на max(1, days / 30), которое для коротких
периодов даёт делитель 1.

Расчёт вынесен в getMonthlyPriceKopeks: месячная ставка считается
пропорцией (price * 30 / days) и не показывается для периодов в месяц
и короче, где она либо дублирует цену, либо вводит в заблуждение.
Тот же хелпер применён на экране продления, где месячная цена для
периодов вроде 45 дней считалась делением на округлённое число
месяцев.
---
 .../purchase/TariffPurchaseForm.tsx           | 14 +++++-----
 src/pages/RenewSubscription.tsx               |  6 ++---
 src/utils/pricing.test.ts                     | 26 +++++++++++++++++++
 src/utils/pricing.ts                          | 15 +++++++++++
 4 files changed, 51 insertions(+), 10 deletions(-)
 create mode 100644 src/utils/pricing.test.ts
 create mode 100644 src/utils/pricing.ts

diff --git a/src/components/subscription/purchase/TariffPurchaseForm.tsx b/src/components/subscription/purchase/TariffPurchaseForm.tsx
index d49a918..a234f11 100644
--- a/src/components/subscription/purchase/TariffPurchaseForm.tsx
+++ b/src/components/subscription/purchase/TariffPurchaseForm.tsx
@@ -8,6 +8,7 @@ import { useCurrency } from '../../../hooks/useCurrency';
 import { usePromoDiscount } from '../../../hooks/usePromoDiscount';
 import { usePlatform } from '../../../platform';
 import { openPaymentUrl } from '../../../utils/openPaymentUrl';
+import { getMonthlyPriceKopeks } from '../../../utils/pricing';
 import InsufficientBalancePrompt from '../../InsufficientBalancePrompt';
 import type { Tariff, TariffPeriod } from '../../../types';
 
@@ -279,10 +280,7 @@ export function TariffPurchaseForm({
                   const displayDiscount = promoPeriod.percent;
                   const displayOriginal = promoPeriod.original;
                   const displayPrice = promoPeriod.price;
-                  const displayPerMonth =
-                    displayPrice !== period.price_kopeks
-                      ? Math.round(displayPrice / Math.max(1, period.days / 30))
-                      : period.price_per_month_kopeks;
+                  const displayPerMonth = getMonthlyPriceKopeks(displayPrice, period.days);
 
                   return (
                     <button
@@ -317,9 +315,11 @@ export function TariffPurchaseForm({
                           </span>
                         )}
                       </div>
-                      <div className="mt-1 text-xs text-dark-500">
-                        {formatPrice(displayPerMonth)}/{t('subscription.month')}
-                      </div>
+                      {displayPerMonth !== null && (
+                        <div className="mt-1 text-xs text-dark-500">
+                          {formatPrice(displayPerMonth)}/{t('subscription.month')}
+                        </div>
+                      )}
                     </button>
                   );
                 })}
diff --git a/src/pages/RenewSubscription.tsx b/src/pages/RenewSubscription.tsx
index 8b9ea7e..ebcc461 100644
--- a/src/pages/RenewSubscription.tsx
+++ b/src/pages/RenewSubscription.tsx
@@ -5,6 +5,7 @@ import { Navigate, useNavigate, useParams } from 'react-router';
 import { subscriptionApi } from '../api/subscription';
 import { useTheme } from '../hooks/useTheme';
 import { getGlassColors } from '../utils/glassTheme';
+import { getMonthlyPriceKopeks } from '../utils/pricing';
 import { useCurrency } from '../hooks/useCurrency';
 import { useHaptic } from '../platform';
 import InsufficientBalancePrompt from '../components/InsufficientBalancePrompt';
@@ -143,8 +144,7 @@ export default function RenewSubscription() {
           {options.map((option) => {
             const isSelected = selectedPeriod === option.period_days;
             const canAfford = balanceKopeks >= option.price_kopeks;
-            const months = Math.max(1, Math.round(option.period_days / 30));
-            const perMonth = option.price_kopeks / months;
+            const perMonth = getMonthlyPriceKopeks(option.price_kopeks, option.period_days);
 
             return (
               <button
@@ -181,7 +181,7 @@ export default function RenewSubscription() {
                         ? t('subscription.free', 'Бесплатно')
                         : `${formatAmount(option.price_kopeks / 100)} ${currencySymbol}`}
                     </div>
-                    {months > 1 && (
+                    {perMonth !== null && (
                       <div className="text-[11px]" style={{ color: g.textSecondary }}>
                         {formatAmount(perMonth / 100)} {currencySymbol}/
                         {t('common.units.mo', 'мес')}
diff --git a/src/utils/pricing.test.ts b/src/utils/pricing.test.ts
new file mode 100644
index 0000000..feb4489
--- /dev/null
+++ b/src/utils/pricing.test.ts
@@ -0,0 +1,26 @@
+import { describe, expect, it } from 'vitest';
+import { getMonthlyPriceKopeks } from './pricing';
+
+describe('getMonthlyPriceKopeks', () => {
+  it('hides the monthly rate for periods of a month or shorter', () => {
+    expect(getMonthlyPriceKopeks(4830, 7)).toBeNull();
+    expect(getMonthlyPriceKopeks(9900, 14)).toBeNull();
+    expect(getMonthlyPriceKopeks(16030, 30)).toBeNull();
+  });
+
+  it('divides by whole months for multiples of 30 days', () => {
+    expect(getMonthlyPriceKopeks(30000, 90)).toBe(10000);
+    expect(getMonthlyPriceKopeks(60000, 180)).toBe(10000);
+  });
+
+  it('prorates periods that are not whole months', () => {
+    expect(getMonthlyPriceKopeks(15000, 45)).toBe(10000);
+    expect(getMonthlyPriceKopeks(100000, 365)).toBe(8219);
+  });
+
+  it('returns null for non-finite input', () => {
+    expect(getMonthlyPriceKopeks(Number.NaN, 90)).toBeNull();
+    expect(getMonthlyPriceKopeks(30000, Number.NaN)).toBeNull();
+    expect(getMonthlyPriceKopeks(30000, Number.POSITIVE_INFINITY)).toBeNull();
+  });
+});
diff --git a/src/utils/pricing.ts b/src/utils/pricing.ts
new file mode 100644
index 0000000..5f12fd5
--- /dev/null
+++ b/src/utils/pricing.ts
@@ -0,0 +1,15 @@
+/** Длина «месяца» в днях — тот же множитель, что использует бэкенд при расчёте цены за месяц. */
+const DAYS_IN_MONTH = 30;
+
+/**
+ * Цена за месяц (в копейках) для периода длиной `days` дней.
+ *
+ * Возвращает `null`, когда месячная ставка не имеет смысла: для периода
+ * в месяц и короче она либо повторяет цену периода (30 дней), либо
+ * выдаёт цену периода за месячную (7 дней → цена семи дней «за месяц»).
+ */
+export function getMonthlyPriceKopeks(priceKopeks: number, days: number): number | null {
+  if (!Number.isFinite(priceKopeks) || !Number.isFinite(days)) return null;
+  if (days <= DAYS_IN_MONTH) return null;
+  return Math.round((priceKopeks * DAYS_IN_MONTH) / days);
+}