bollwerkadmin/bollwerk
1
0
Fork 0
Code Issues 12 Pull requests Projects Releases Packages Wiki Activity Actions

Refactoring: Settings Type-Safety und Validierung #82

New issue
Closed
opened 2026-05-16 23:42:54 +00:00 by jreinemann-euris · 0 comments
jreinemann-euris commented 2026-05-16 23:42:54 +00:00 (Migrated from github.com)
Copy link

Kontext

Die App speichert Settings als Key-Value-Strings in der Room-Datenbank (SettingsEntity(key, value)). Es gibt keine Type-Safety, keine Validierung und keine zentrale Definition der gültigen Keys.

Problem

  • Ein Tippfehler in einem Key (z.B. "sever_url" statt "server_url") führt zu stillem Fehlverhalten
  • Kein Compile-Time-Check auf gültige Settings-Keys
  • Werte werden als Strings gespeichert, Typkonvertierung passiert ad-hoc beim Lesen
  • Kein Default-Wert-Management – fehlende Keys müssen überall individuell behandelt werden

Akzeptanzkriterien

  • Zentrale Settings-Key-Definition (enum oder sealed class)
  • Type-safe Getter/Setter im SettingsRepository (z.B. getInt(key), getString(key))
  • Default-Werte zentral definiert
  • Bestehende Settings-Zugriffe auf die neue API migrieren
  • Compile-Time-Fehler bei ungültigen Keys
  • Alle Tests grün
## Kontext Die App speichert Settings als Key-Value-Strings in der Room-Datenbank (`SettingsEntity(key, value)`). Es gibt keine Type-Safety, keine Validierung und keine zentrale Definition der gültigen Keys. ## Problem - Ein Tippfehler in einem Key (z.B. `"sever_url"` statt `"server_url"`) führt zu stillem Fehlverhalten - Kein Compile-Time-Check auf gültige Settings-Keys - Werte werden als Strings gespeichert, Typkonvertierung passiert ad-hoc beim Lesen - Kein Default-Wert-Management – fehlende Keys müssen überall individuell behandelt werden ## Akzeptanzkriterien - [ ] Zentrale Settings-Key-Definition (enum oder sealed class) - [ ] Type-safe Getter/Setter im SettingsRepository (z.B. `getInt(key)`, `getString(key)`) - [ ] Default-Werte zentral definiert - [ ] Bestehende Settings-Zugriffe auf die neue API migrieren - [ ] Compile-Time-Fehler bei ungültigen Keys - [ ] Alle Tests grün
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
No results found.
Labels
Clear labels   
block-planning

   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
feature

   
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
infrastructure

   
invalid

This doesn't seem right

  
planning

   
priority:high

   
priority:low

   
question

Further information is requested

  
refactoring

   
status:backlog

Im Backlog

  
status:done

Abgeschlossen

  
status:in-progress

Aktuell in Bearbeitung

  
status:todo

Naechstes Todo-Ticket

  
tech-decision

   
test

   
wontfix

This will not be worked on

No labels
block-planning
bug
documentation
duplicate
enhancement
feature
good first issue
help wanted
infrastructure
invalid
planning
priority:high
priority:low
question
refactoring
status:backlog
status:done
status:in-progress
status:todo
tech-decision
test
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: bollwerkadmin/bollwerk#82
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?