Aufräumen nach URL-Neuordnung Etappe 4b #147

New issue

Open

opened 2026-10-25 07:30:31 +00:00 by claude · 0 comments

claude commented

2026-10-25 07:30:31 +00:00

Collaborator

Kleinigkeiten aus den Reviews der URL-Neuordnung Etappe 4b (25.10., Branch feature/url-etappe-4b, Merge offen; Gesamt-Review logs/url-neuordnung/final-review-etappe4b.md, M2). Kein Fehlverhalten, nur Aufräumen.

Der interne Spielbericht entscheidet über eine Null-Prüfung, der öffentliche über den Status. TeamMatchReportService.java:61 lädt Spiele und Entscheid, wenn row.score() != null; öffentlich (:87) steht dieselbe Frage als match.resultStatus == ResultStatus.BESTAETIGT. Intern hängt die Regel daran, dass TeamMatchScoreService.submittedScores Begegnungen ohne eingereichtes Resultat weglässt (TeamMatchScoreService.java:29–41). Ändert sich das (z. B. für KORREKTUR_ANGEFRAGT), ändert sich der Bericht still mit; score beantwortet dann zwei Fragen (CLAUDE.md „Eine Bedeutung je Feld“). Fix: TeamMatchScoreService.hasSubmittedResult (:44, heute private) paketweit sichtbar machen und in :61 nutzen. Kein Test ändert sich.
matches doppelt belegt in homepage/src/pages/spielbetrieb/meisterschaft.astro: Z. 25 ist die ganze Liste der Saison, das Feld Panel.matches (Z. 19, 29) die Begegnungen einer Kategorie. Die Liste z. B. seasonMatches nennen.

Zur Kenntnis, für eine spätere Etappe: Wenn cupCategoryId mit seinem Aufrufer kommt, gehört TeamMatchListService.inLigaCategory (package-static, von zwei Services benutzt) in einen kleinen Filter-Typ.

Kleinigkeiten aus den Reviews der URL-Neuordnung Etappe 4b (25.10., Branch `feature/url-etappe-4b`, Merge offen; Gesamt-Review `logs/url-neuordnung/final-review-etappe4b.md`, M2). Kein Fehlverhalten, nur Aufräumen. 1. **Der interne Spielbericht entscheidet über eine Null-Prüfung, der öffentliche über den Status.** `TeamMatchReportService.java:61` lädt Spiele und Entscheid, wenn `row.score() != null`; öffentlich (`:87`) steht dieselbe Frage als `match.resultStatus == ResultStatus.BESTAETIGT`. Intern hängt die Regel daran, dass `TeamMatchScoreService.submittedScores` Begegnungen ohne eingereichtes Resultat weglässt (`TeamMatchScoreService.java:29–41`). Ändert sich das (z. B. für `KORREKTUR_ANGEFRAGT`), ändert sich der Bericht still mit; `score` beantwortet dann zwei Fragen (CLAUDE.md „Eine Bedeutung je Feld“). Fix: `TeamMatchScoreService.hasSubmittedResult` (`:44`, heute `private`) paketweit sichtbar machen und in `:61` nutzen. Kein Test ändert sich. 2. **`matches` doppelt belegt in `homepage/src/pages/spielbetrieb/meisterschaft.astro`:** Z. 25 ist die ganze Liste der Saison, das Feld `Panel.matches` (Z. 19, 29) die Begegnungen einer Kategorie. Die Liste z. B. `seasonMatches` nennen. Zur Kenntnis, für eine spätere Etappe: Wenn `cupCategoryId` mit seinem Aufrufer kommt, gehört `TeamMatchListService.inLigaCategory` (package-static, von zwei Services benutzt) in einen kleinen Filter-Typ.