Формы итогов партии: дублирующийся разбор целей и миров, двойной расчёт мест #89

Closed
opened 2026-09-18 23:21:07 +03:00 by Agent · 0 comments
Collaborator

Упрощения по диффу вехи v1.5, поведение не меняется.

  • Разбор поля «цели/миры» (пусто → null, иначе целое 0..99) написан дважды: parseCount в frontend/src/pages/admin/AdminMatchEdit.tsx и inline в countInput из frontend/src/components/PlaceEditor.tsx. Вынести одну функцию в domain/ вместе с верхней границей 99.
  • В frontend/src/pages/MatchDetailPage.tsx места по блокам (competition ranking 1,2,2,4, выбывшие — после) считаются дважды: в outcomeSeats() для предупреждений и в resultRows() для API. Считать один раз и строить обе проекции из общего результата.

Обнаружено при работе над #82.

Упрощения по диффу вехи v1.5, поведение не меняется. - [ ] Разбор поля «цели/миры» (пусто → `null`, иначе целое 0..99) написан дважды: `parseCount` в `frontend/src/pages/admin/AdminMatchEdit.tsx` и inline в `countInput` из `frontend/src/components/PlaceEditor.tsx`. Вынести одну функцию в `domain/` вместе с верхней границей 99. - [ ] В `frontend/src/pages/MatchDetailPage.tsx` места по блокам (competition ranking 1,2,2,4, выбывшие — после) считаются дважды: в `outcomeSeats()` для предупреждений и в `resultRows()` для API. Считать один раз и строить обе проекции из общего результата. Обнаружено при работе над #82.
Agent added this to the v1.5 — изменение системы подсчёта рейтинга milestone 2026-09-18 23:21:07 +03:00
Agent added the Kind/Enhancement
Reviewed
Confirmed
1
labels 2026-09-18 23:21:16 +03:00
Agent added the
Status
In Review
label 2026-09-18 23:30:12 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NotBigGhost/ForbiddenStarsApp#89