Toter Code: DELETE /api/saisons/{id} und deleteSaison - keine Oberfläche ruft sie auf #134

New issue

Open

opened 2026-10-23 18:26:11 +00:00 by claude · 0 comments

claude commented

2026-10-23 18:26:11 +00:00

Collaborator

Beim Messen, welche Endpunkte E2E aufruft (URL-Neuordnung, 23.10.), fiel DELETE /api/saisons/{id} auf: Keine Oberfläche ruft ihn auf, und das war auch nie so.

Befund

Backend: SaisonResource.delete (backend/src/main/java/ch/sffs/tt/cms/resource/SaisonResource.java:116-135) löscht eine CMS-Saison samt Events, Anhängen und Fotos. Aufgerufen wird er nur aus SaisonResourceTest.
Frontend: deleteSaison in content/src/lib/events-api.ts:26-28. Importiert wird es nirgends, aufgerufen nur in content/src/test/events-api.test.ts.
Die Veranstaltungsseite (EventManager.tsx) hat keinen Knopf „Saison löschen“, nur „Event wirklich löschen?“.
Git-Historie: deleteSaison( steht in drei Commits, keiner davon in einer Seite (05af31e8 Anlage 28.04., 4bb157a5 Umbenennung, 268c0652 Unit-Test 10.07.).
Auch Skripte, Skills, Runbooks und E2E rufen den Endpunkt nicht auf.

Vorschlag
Endpunkt und Frontend-Funktion samt ihren Tests entfernen (CLAUDE.md: „Kein Produktivcode nur für Tests“, wie #45). In docs/2026-10-20-service-verantwortlichkeiten.md:658 steht „Admin · Veranstaltungen · Saison löschen“ als Anwendungsfall. Die Zeile ist aus dem Endpunkt abgeleitet, nicht aus der Oberfläche, und fällt mit weg.

Bis dahin zieht der Endpunkt in Etappe 1 der URL-Neuordnung ohne E2E-Fall nach DELETE /api/event-seasons/{event_season_id} um. Olli, 23.10.

Beim Messen, welche Endpunkte E2E aufruft (URL-Neuordnung, 23.10.), fiel `DELETE /api/saisons/{id}` auf: Keine Oberfläche ruft ihn auf, und das war auch nie so. **Befund** - Backend: `SaisonResource.delete` (`backend/src/main/java/ch/sffs/tt/cms/resource/SaisonResource.java:116-135`) löscht eine CMS-Saison samt Events, Anhängen und Fotos. Aufgerufen wird er nur aus `SaisonResourceTest`. - Frontend: `deleteSaison` in `content/src/lib/events-api.ts:26-28`. Importiert wird es nirgends, aufgerufen nur in `content/src/test/events-api.test.ts`. - Die Veranstaltungsseite (`EventManager.tsx`) hat keinen Knopf „Saison löschen“, nur „Event wirklich löschen?“. - Git-Historie: `deleteSaison(` steht in drei Commits, keiner davon in einer Seite (05af31e8 Anlage 28.04., 4bb157a5 Umbenennung, 268c0652 Unit-Test 10.07.). - Auch Skripte, Skills, Runbooks und E2E rufen den Endpunkt nicht auf. **Vorschlag** Endpunkt und Frontend-Funktion samt ihren Tests entfernen (CLAUDE.md: „Kein Produktivcode nur für Tests“, wie #45). In `docs/2026-10-20-service-verantwortlichkeiten.md:658` steht „Admin · Veranstaltungen · Saison löschen“ als Anwendungsfall. Die Zeile ist aus dem Endpunkt abgeleitet, nicht aus der Oberfläche, und fällt mit weg. Bis dahin zieht der Endpunkt in Etappe 1 der URL-Neuordnung ohne E2E-Fall nach `DELETE /api/event-seasons/{event_season_id}` um. Olli, 23.10.