Aufräumen nach URL-Neuordnung Etappe 3 #143

New issue

Open

opened 2026-10-24 16:09:13 +00:00 by claude · 0 comments

claude commented

2026-10-24 16:09:13 +00:00

Collaborator

Kleinigkeiten aus den Reviews der URL-Neuordnung Etappe 3 (24.10., Branch feature/url-etappe-3, Merge offen). Kein Fehlverhalten, nur Aufräumen; passt zu ADR 0033 Schritt 3 (Services je Fachbegriff).

Die Prüfung „Kennung ist eine Liga-Kategorie, sonst 404“ steht dreimal — CategoryRoundService.java:96 und TeamMatchCreationService.java:142 (requireLigaCategory), dazu ausgeschrieben in ScheduleLockService.java:76. Eine Stelle (z. B. ein Lookup wie TeamMatchLookup).
CupAdvanceRequest ist nur noch interner Wrapper (competition/api/CupAdvanceRequest.java:13): seit POST /api/team-matches kommt er auf keiner Leitung mehr an, liegt aber im api-Paket. In den Service ziehen oder durch List<CupAdvanceEntry> ersetzen.
SeasonPhaseService.java:39 lädt alle Liga-Kategorien samt teamCount/finalizedHalbserien nur für isEmpty() — seit Etappe 3 zwei Abfragen mehr je Aufruf. Ein exists im Repository reicht.
Überflüssiger Alias const meisterschaftCategories = categories; in portal/src/pages/SpielbetriebPage.tsx:121.
Importreihenfolge in CategoryRoundService (CompetitionType nach MatchWeek).
openapi.json: bei date von CreateTournamentRequest steht "type":"string" neben dem $ref (kosmetisch, der TS-Typ ist gleich).
CupTableau zeigt bei einem Serverfehler neben dem Toast auch „kein Cup“ — der Nutzer liest zwei Aussagen, von denen eine nicht stimmt. Besser: bei Fehler nur den Toast, keine Leer-Anzeige.

Zur Kenntnis, keine Änderung nötig: „Alle Spiele“ zeigt Spiele am selben Tag jetzt in Kategorie-Reihenfolge; vorher war die Reihenfolge innerhalb eines Tages zufällig (Datenbank).

Kleinigkeiten aus den Reviews der URL-Neuordnung Etappe 3 (24.10., Branch `feature/url-etappe-3`, Merge offen). Kein Fehlverhalten, nur Aufräumen; passt zu ADR 0033 Schritt 3 (Services je Fachbegriff). 1. **Die Prüfung „Kennung ist eine Liga-Kategorie, sonst `404`“ steht dreimal** — `CategoryRoundService.java:96` und `TeamMatchCreationService.java:142` (`requireLigaCategory`), dazu ausgeschrieben in `ScheduleLockService.java:76`. Eine Stelle (z. B. ein Lookup wie `TeamMatchLookup`). 2. **`CupAdvanceRequest` ist nur noch interner Wrapper** (`competition/api/CupAdvanceRequest.java:13`): seit `POST /api/team-matches` kommt er auf keiner Leitung mehr an, liegt aber im `api`-Paket. In den Service ziehen oder durch `List<CupAdvanceEntry>` ersetzen. 3. **`SeasonPhaseService.java:39` lädt alle Liga-Kategorien samt `teamCount`/`finalizedHalbserien` nur für `isEmpty()`** — seit Etappe 3 zwei Abfragen mehr je Aufruf. Ein `exists` im Repository reicht. 4. **Überflüssiger Alias** `const meisterschaftCategories = categories;` in `portal/src/pages/SpielbetriebPage.tsx:121`. 5. **Importreihenfolge** in `CategoryRoundService` (`CompetitionType` nach `MatchWeek`). 6. **`openapi.json`**: bei `date` von `CreateTournamentRequest` steht `"type":"string"` neben dem `$ref` (kosmetisch, der TS-Typ ist gleich). 7. **CupTableau zeigt bei einem Serverfehler** neben dem Toast auch „kein Cup“ — der Nutzer liest zwei Aussagen, von denen eine nicht stimmt. Besser: bei Fehler nur den Toast, keine Leer-Anzeige. Zur Kenntnis, keine Änderung nötig: „Alle Spiele“ zeigt Spiele am **selben Tag** jetzt in Kategorie-Reihenfolge; vorher war die Reihenfolge innerhalb eines Tages zufällig (Datenbank).