search

it-at-m / Wahllokalsystem

system that supports elections in the city of munich
https://opensource.muenchen.de/
MIT License
5 stars 0 forks source link

Auslagern von Authority Strings #420

Closed vjohnslhm closed 1 month ago

vjohnslhm commented 2 months ago

Dies Auslagerung an sich gefällt mir und wäre zu überlegen auch auf die anderen Services zu übernehmen.

_Originally posted by @MrSebastian in https://github.com/it-at-m/Wahllokalsystem/pull/412#discussion_r1760907764_

Beispiel

String AUTHORITY_WRITE_EREIGNIS = "hasAuthority('VorfaelleUndVorkommnisse_WRITE_Ereignisse')";
@PreAuthorize(AUTHORITY_WRITE_EREIGNIS)

statt

@PreAuthorize("hasAuthority('VorfaelleUndVorkommnisse_WRITE_Ereignisse')")

Beschreibung

Das Auslagern der AuthorityStrings hätte folgende Vor- und Nachteile die mir einfallen:

+ -
Wiederverwendbarkeit mehr Codezeilen
bessere Lesbarkeit
Reduzierung von Errors durch Tippfehler
MrSebastian commented 1 month ago

Entscheidung

Nic12345678 commented 1 month ago

Hey team! Please add your planning poker estimate with Zenhub @dragonfly28 @GerhardPx @MrSebastian @vjohnslhm