Skjermer kode 6 bedre #2753
Skjermer kode 6 bedre #2753
Conversation
| @@ -99,7 +99,7 @@ class RefreshK9v3Tjeneste( | |||
| @WithSpan | |||
| fun behandlingerTilOppfriskningForKøer(tx: TransactionalSession, køId: List<Long>, antallPrKø: Int) : Set<UUID>{ | |||
| return DetaljerMetrikker.time("RefreshK9V3", "refreshForKøer", køId.size.toString()) { | |||
| val køer = køId.map { oppgaveKoRepository.hent(it) } | |||
| val køer = køId.map { oppgaveKoRepository.hentIkkeTaHensyn(it) } | |||
There was a problem hiding this comment.
Nettopp-tilbake-på-jobb-alibi: Husker ikke hva man lar være å ta hensyn til her, men husker at jeg syntes at dette var et forferdelig metodenavn. Hva tar vi ikke hensyn til?
There was a problem hiding this comment.
hentIkkeTaHensyn(køId: Long) tar ikke hensyn til skjerming/kode 6 som hent(køId: Long, skjermet: Boolean) gjør. Så de gir mening om man ser begge metoder, men er ikke så bra alene. Forslag til bedre navnekonvensjon for dette?
There was a problem hiding this comment.
vet ikke om jeg skal stipulere en konvensjon, men hentInkluderSkjermet() gir mening i mitt hue?
There was a problem hiding this comment.
Det kan fungere.
Ser ut som at i LOS betyr "skjermet" noe annet enn i Nav for øvrig. Her er skjermet det samme som kode 6. Andre snakker om skjerming som enten kode 6, kode 7 eller egen ansatt. Atter andre bruker skjermet kun om egne ansatte.
Vi kunne nok døpt om til kode6 alle steder vi bruker skjermet.
There was a problem hiding this comment.
Oi ja det virker som en viktig distinksjon å holde styr på
There was a problem hiding this comment.
Døper om til f. eks. hentInkluderKode6 i denne omgang, og så får vi se om vi endrer skjermet til noe annet på sikt.
Viser ikke skjermede køer til vanlige saksbehandlere, og omvendt.
https://jira.adeo.no/browse/TSFF-830