Samle logikk for utsatt oppgave i service #374
Conversation
| private val logger = this.logger() | ||
|
|
||
| fun opprettOppgaveIGosys( |
There was a problem hiding this comment.
Denne er bare flyttet på, men noen småforbedringer som ikke endrer funksjonalitet.
| if (oppgave.timeout == null) { | ||
| metrikk.tellUtsattOppgave_UtenDato() |
There was a problem hiding this comment.
Sletter denne, da den aldri til oppstå. timeout er ikke nullable. Fjerner også tellUtsattOppgave_UtenDato-metrikken, da den kun ble brukt her.
There was a problem hiding this comment.
Timeout ER jo nullable:
class OppgaveOppdatering(
val id: UUID,
val handling: Handling,
val timeout: LocalDateTime?,
val oppdateringstype: OppdateringstypeDTO
)
| if (oppgave.tilstand == Tilstand.Utsatt && oppdatering.handling == no.nav.syfo.utsattoppgave.Handling.Utsett) { | ||
| if (oppgave.timeout == null) { | ||
| metrikk.tellUtsattOppgave_UtenDato() | ||
| return when { |
There was a problem hiding this comment.
Her bytter jeg bare if-ene med en when, og bruker piping der det føles naturlig.
| oppgave | ||
| .apply { | ||
| oppdatert = LocalDateTime.now() | ||
| } | ||
| .copy( | ||
| tilstand = Tilstand.Forkastet, | ||
| speil = gjelderSpeil, | ||
| ) | ||
| .lagre() |
There was a problem hiding this comment.
Jeg forstår ikke helt hvorfor oppgave blir muterer oppdatert-feltet, for så å bruke den ikke-muterende copy-funksjonen 🤷
|
SonarCloud Quality Gate failed. |
| if (oppgave.timeout == null) { | ||
| metrikk.tellUtsattOppgave_UtenDato() |
There was a problem hiding this comment.
Timeout ER jo nullable:
class OppgaveOppdatering(
val id: UUID,
val handling: Handling,
val timeout: LocalDateTime?,
val oppdateringstype: OppdateringstypeDTO
)
No description provided.