projekte/deklarix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: unbenutzte View-Felder entfernen (ID auf drei Templates nie referenziert)

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
noroot
2026-08-29 21:07:00 +02:00
parent 29961f5657
commit 96ac5d6b59
2 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
internal/web/antrag_handlers.go
View File

@@ -360,7 +360,6 @@ func (s *Server) handleAntragList(w http.ResponseWriter, r *http.Request) {
} }
type anforderungView struct { type anforderungView struct {
ID string
Beschreibung string Beschreibung string
Herleitung string Herleitung string
} }
@@ -387,7 +386,6 @@ type bewertungView struct {
type antragDetailData struct { type antragDetailData struct {
Title string Title string
Nav navData Nav navData
ID string
Titel string Titel string
Beschreibung string Beschreibung string
Ergebnis string Ergebnis string
@@ -415,7 +413,7 @@ func (s *Server) handleAntragDetail(w http.ResponseWriter, r *http.Request) {
} }
data := antragDetailData{ 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, Beschreibung: antrag.Beschreibung, Ergebnis: antrag.Ergebnis, Haeufigkeit: antrag.Haeufigkeit,
Status: antrag.Status, CreatedAt: antrag.CreatedAt.Format("02.01.2006 15:04"), Antworten: antwortenAnzeige(antworten), 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"), CreatedAt: b.CreatedAt.Format("02.01.2006 15:04"),
} }
for _, a := range b.Anforderungen { 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 { for _, id := range b.ZulaessigeWerkzeuge {
v.ZulaessigeWerkzeuge = append(v.ZulaessigeWerkzeuge, s.werkzeugName(ctx, id)) v.ZulaessigeWerkzeuge = append(v.ZulaessigeWerkzeuge, s.werkzeugName(ctx, id))

3
internal/web/betreiber_handlers.go
View File

@@ -80,7 +80,6 @@ type betreiberUserView struct {
type betreiberAccountDetailData struct { type betreiberAccountDetailData struct {
Title string Title string
Nav navData Nav navData
AccountID string
Name string Name string
Users []betreiberUserView Users []betreiberUserView
CreatedAt string CreatedAt string
@@ -101,7 +100,7 @@ func (s *Server) handleBetreiberAccountDetail(w http.ResponseWriter, r *http.Req
} }
data := betreiberAccountDetailData{ 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"), CreatedAt: acc.CreatedAt.Format("02.01.2006 15:04"),
} }
for _, u := range users { for _, u := range users {