From 96ac5d6b59a0843275fdad114ec45e0e3609032f Mon Sep 17 00:00:00 2001 From: noroot Date: Sat, 29 Aug 2026 21:07:00 +0200 Subject: [PATCH] refactor: unbenutzte View-Felder entfernen (ID auf drei Templates nie referenziert) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ein systematischer Audit nach demselben Muster wie beim Fragebogen- Antworten-Bug fand drei Felder, die vom Handler befüllt, aber im zugehörigen Template nie referenziert werden: antragDetailData.ID (kein Formular auf der Seite braucht sie), anforderungView.ID (die Beschreibung reicht als lesbare Anforderung, die interne Regel-ID ist kein Nutzerinhalt), betreiberAccountDetailData.AccountID (die Account-Detailseite ist rein lesend, kein Formular zeigt darauf). Anders als der Fragebogen-Antworten-Bug ist hier keine für Nutzer relevante Information verloren gegangen — die Felder waren schlicht totes Gewicht, deshalb entfernt statt eine Anzeige dafür zu erfinden. Co-Authored-By: Claude Sonnet 5 --- internal/web/antrag_handlers.go | 6 ++---- internal/web/betreiber_handlers.go | 3 +-- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/internal/web/antrag_handlers.go b/internal/web/antrag_handlers.go index 391d64e..ff52ca2 100644 --- a/internal/web/antrag_handlers.go +++ b/internal/web/antrag_handlers.go @@ -360,7 +360,6 @@ func (s *Server) handleAntragList(w http.ResponseWriter, r *http.Request) { } type anforderungView struct { - ID string Beschreibung string Herleitung string } @@ -387,7 +386,6 @@ type bewertungView struct { type antragDetailData struct { Title string Nav navData - ID string Titel string Beschreibung string Ergebnis string @@ -415,7 +413,7 @@ func (s *Server) handleAntragDetail(w http.ResponseWriter, r *http.Request) { } data := antragDetailData{ - Title: "Antrag", Nav: navFor(r), ID: antrag.ID, Titel: antrag.Titel, + Title: "Antrag", Nav: navFor(r), Titel: antrag.Titel, Beschreibung: antrag.Beschreibung, Ergebnis: antrag.Ergebnis, Haeufigkeit: antrag.Haeufigkeit, Status: antrag.Status, CreatedAt: antrag.CreatedAt.Format("02.01.2006 15:04"), Antworten: antwortenAnzeige(antworten), } @@ -442,7 +440,7 @@ func (s *Server) toBewertungView(ctx context.Context, b store.Bewertung) *bewert CreatedAt: b.CreatedAt.Format("02.01.2006 15:04"), } for _, a := range b.Anforderungen { - v.Anforderungen = append(v.Anforderungen, anforderungView{ID: a.ID, Beschreibung: a.Beschreibung, Herleitung: a.Herleitung}) + v.Anforderungen = append(v.Anforderungen, anforderungView{Beschreibung: a.Beschreibung, Herleitung: a.Herleitung}) } for _, id := range b.ZulaessigeWerkzeuge { v.ZulaessigeWerkzeuge = append(v.ZulaessigeWerkzeuge, s.werkzeugName(ctx, id)) diff --git a/internal/web/betreiber_handlers.go b/internal/web/betreiber_handlers.go index 8ee4715..c3e2b1d 100644 --- a/internal/web/betreiber_handlers.go +++ b/internal/web/betreiber_handlers.go @@ -80,7 +80,6 @@ type betreiberUserView struct { type betreiberAccountDetailData struct { Title string Nav navData - AccountID string Name string Users []betreiberUserView CreatedAt string @@ -101,7 +100,7 @@ func (s *Server) handleBetreiberAccountDetail(w http.ResponseWriter, r *http.Req } data := betreiberAccountDetailData{ - Title: "Account", Nav: navFor(r), AccountID: acc.ID, Name: acc.Name, + Title: "Account", Nav: navFor(r), Name: acc.Name, CreatedAt: acc.CreatedAt.Format("02.01.2006 15:04"), } for _, u := range users {