Auf- und Abstieg: zwei Aufräumpunkte im Code (validateCounts, hasZones) #109

New issue

Open

opened 2026-10-18 17:55:45 +00:00 by claude · 0 comments

claude commented

2026-10-18 17:55:45 +00:00

Collaborator

Gefunden im Abschluss-Review von #88 (in main eb18a9b). Kein sichtbarer Fehler, nur Code.

Leerer Text als Markierung — validateCounts gibt bei zu vielen Plätzen { promotion: "", relegation: "Zusammen höchstens …" } zurück (portal/src/lib/promotion-relegation.ts (51-56)). Der leere Text bedeutet „Feld ist falsch, aber ohne eigene Meldung“ — eine zweite Bedeutung in einem Textfeld (CLAUDE.md „Eine Bedeutung je Feld“). Besser ausdrücklich, z.B. ein eigenes Feld für den gemeinsamen Fehler.
hasZones je Zeile — in homepage/src/pages/spielbetrieb/meisterschaft.astro (100,117) wird hasZones(p.standings) im Kopf und in jeder Tabellenzeile neu gerechnet. Einmal je Kategorie reicht, wie im Portal (portal/src/components/LigaView.tsx (304)).

Beides ohne Verhaltensänderung; bestehende Tests müssen grün bleiben.

Gefunden im Abschluss-Review von #88 (in main eb18a9b). Kein sichtbarer Fehler, nur Code. 1. **Leerer Text als Markierung** — `validateCounts` gibt bei zu vielen Plätzen `{ promotion: "", relegation: "Zusammen höchstens …" }` zurück (`portal/src/lib/promotion-relegation.ts` (51-56)). Der leere Text bedeutet „Feld ist falsch, aber ohne eigene Meldung“ — eine zweite Bedeutung in einem Textfeld (CLAUDE.md „Eine Bedeutung je Feld“). Besser ausdrücklich, z.B. ein eigenes Feld für den gemeinsamen Fehler. 2. **`hasZones` je Zeile** — in `homepage/src/pages/spielbetrieb/meisterschaft.astro` (100,117) wird `hasZones(p.standings)` im Kopf und in jeder Tabellenzeile neu gerechnet. Einmal je Kategorie reicht, wie im Portal (`portal/src/components/LigaView.tsx` (304)). Beides ohne Verhaltensänderung; bestehende Tests müssen grün bleiben.