Alte Fachmethoden der Status-Endpunkte sind noch öffentlich #140

New issue

Open

opened 2026-10-24 11:30:23 +00:00 by claude · 0 comments

claude commented

2026-10-24 11:30:23 +00:00

Collaborator

Seit der URL-Neuordnung Etappe 2 laufen Zustandswechsel über je eine Methode setStatus/set…Status, die Rolle, „schon erreicht“ und Mails regelt. Die alten Fachmethoden dahinter sind teils noch public @Transactional: TeamMatchResultService.submit, SeasonHalbserieService.finalizeWeeks, HalbseriePublicationService.publish/unpublish, ClubRegistrationService.complete/reopen, ScheduleLockService.unlock. Kein Endpunkt führt an setStatus vorbei dorthin, ein Risiko besteht heute also nicht – aber ein künftiger Aufrufer umginge Rollenprüfung und Idempotenz.

Erwartet: privat; die Service-Tests, die sie heute direkt aufrufen (z.B. TeamMatchResultServiceTest, SeasonHalbserieServiceTest), gehen den Weg über setStatus. (confirmResult, requestCorrection, unfinalizeWeeks sind bereits privat – sie hatten keine Testaufrufer.)

Gefunden im Gesamt-Review der URL-Neuordnung Etappe 2 (24.10.2026).

Seit der URL-Neuordnung Etappe 2 laufen Zustandswechsel über je eine Methode `setStatus`/`set…Status`, die Rolle, „schon erreicht“ und Mails regelt. Die alten Fachmethoden dahinter sind teils noch `public @Transactional`: `TeamMatchResultService.submit`, `SeasonHalbserieService.finalizeWeeks`, `HalbseriePublicationService.publish/unpublish`, `ClubRegistrationService.complete/reopen`, `ScheduleLockService.unlock`. Kein Endpunkt führt an `setStatus` vorbei dorthin, ein Risiko besteht heute also nicht – aber ein künftiger Aufrufer umginge Rollenprüfung und Idempotenz. **Erwartet:** privat; die Service-Tests, die sie heute direkt aufrufen (z.B. `TeamMatchResultServiceTest`, `SeasonHalbserieServiceTest`), gehen den Weg über `setStatus`. (`confirmResult`, `requestCorrection`, `unfinalizeWeeks` sind bereits privat – sie hatten keine Testaufrufer.) Gefunden im Gesamt-Review der URL-Neuordnung Etappe 2 (24.10.2026).