From 415a76f7cff4e0e06edb15b91a24467f0465e2e9 Mon Sep 17 00:00:00 2001 From: Fringg Date: Thu, 6 Aug 2026 02:25:02 +0300 Subject: [PATCH] =?UTF-8?q?fix(users):=20=D0=BF=D0=BE=D0=BA=D0=B0=D0=B7?= =?UTF-8?q?=D1=8B=D0=B2=D0=B0=D1=82=D1=8C=20=D0=BF=D1=80=D0=B8=D1=87=D0=B8?= =?UTF-8?q?=D0=BD=D1=83=20=D0=BE=D1=82=D0=BA=D0=B0=D0=B7=D0=B0=20=D0=BF?= =?UTF-8?q?=D1=80=D0=B8=20=D1=83=D0=B4=D0=B0=D0=BB=D0=B5=D0=BD=D0=B8=D0=B8?= =?UTF-8?q?=20=D0=BF=D0=BE=D0=B4=D0=BF=D0=B8=D1=81=D0=BA=D0=B8?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Дополнение к PR #542 при мерже. Обработчик глушил ошибку целиком и показывал общее «Ошибка». Между тем сервер отвечает ровно теми словами, которые админу и нужны: открытый временный доступ (409, «сначала заверши или восстанови grace»), активная платная без force (409), подписки нет (404). Первый случай автор сам описал в PR как штатный сценарий — и именно он выглядел как сломанная кнопка. Прокинул getApiErrorMessage, он для этого в репозитории и заведён и уже используется в восьми других местах. Логику force не трогал: она совпадает с сервером. Сериализуемое is_active собирается в _build_subscription_info как `status == active AND end_date > now`, то есть в точности серверное свойство Subscription.is_active, по которому и стоит защита. Разъехаться значения могут только в безопасную сторону: подписка успела истечь между загрузкой страницы и нажатием — тогда защиты нет и на сервере. --- src/pages/AdminUserDetail.tsx | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/pages/AdminUserDetail.tsx b/src/pages/AdminUserDetail.tsx index 7902e06..09a1e18 100644 --- a/src/pages/AdminUserDetail.tsx +++ b/src/pages/AdminUserDetail.tsx @@ -28,6 +28,7 @@ import { ActivityTab } from '../components/admin/userDetail/ActivityTab'; import { TicketsTab } from '../components/admin/userDetail/TicketsTab'; import { InfoTab } from '../components/admin/userDetail/InfoTab'; import { SubscriptionTab } from '../components/admin/userDetail/SubscriptionTab'; +import { getApiErrorMessage } from '../utils/api-error'; import { toNumber } from '../utils/inputHelpers'; import { usePermissionStore } from '../store/permissions'; @@ -673,8 +674,12 @@ export default function AdminUserDetail() { notify.success(t('admin.users.detail.subscription.deleted'), t('common.success')); setSubscriptionDetailView(false); await loadUser(); - } catch { - notify.error(t('admin.users.userActions.error'), t('common.error')); + } catch (err) { + // Отказы тут осмысленные и действенные: открытый временный доступ + // (409, «сначала заверши или восстанови grace»), активная платная без + // force (409), подписки нет (404). Общее «Ошибка» оставило бы админа + // гадать, почему кнопка не сработала, — показываем текст сервера. + notify.error(getApiErrorMessage(err, t('admin.users.userActions.error')), t('common.error')); } finally { setActionLoading(false); }