Flytter dobbelføring av reservasjoneri saksbehandlerdokument til ette…

…r lagring av reservasjon, slik at den ikke blir hengende igjen hvis v1 reservasjon mangler. (#1951)
navikt · Mar 7, 2024 · 64f06ca · 64f06ca
1 parent 5f6cef1
commit 64f06ca
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/src/main/kotlin/no/nav/k9/los/tjenester/saksbehandler/oppgave/OppgaveTjeneste.kt b/src/main/kotlin/no/nav/k9/los/tjenester/saksbehandler/oppgave/OppgaveTjeneste.kt
@@ -594,12 +594,8 @@ class OppgaveTjeneste constructor(
     suspend fun endreReservasjonPåOppgave(resEndring: ReservasjonEndringDto): Reservasjon {
         val identTilInnloggetBruker = azureGraphService.hentIdentTilInnloggetBruker()
         val oppgavUUID = UUID.fromString(resEndring.oppgaveNøkkel.oppgaveEksternId)
-        if (resEndring.brukerIdent != null) {
-            val reservasjon = reservasjonRepository.hent(oppgavUUID)
-            saksbehandlerRepository.fjernReservasjon(reservasjon.reservertAv, reservasjon.oppgave)
-            saksbehandlerRepository.leggTilReservasjon(resEndring.brukerIdent, reservasjon.oppgave)
-        }
-        return reservasjonRepository.lagre(oppgavUUID, true) {
+
+        val oppdatertReservasjon = reservasjonRepository.lagre(oppgavUUID, true) {
             if (it == null) {
                 throw IllegalArgumentException("Kan ikke oppdatere reservasjon som ikke finnes.")
             }
@@ -624,17 +620,20 @@ class OppgaveTjeneste constructor(
             }
             it
         }
+        if (resEndring.brukerIdent != null) {
+            val reservasjon = reservasjonRepository.hent(oppgavUUID)
+            saksbehandlerRepository.fjernReservasjon(reservasjon.reservertAv, reservasjon.oppgave)
+            saksbehandlerRepository.leggTilReservasjon(resEndring.brukerIdent, reservasjon.oppgave)
+        }
+        return oppdatertReservasjon
     }
 
     suspend fun flyttReservasjon(uuid: UUID, ident: String, begrunnelse: String): Reservasjon {
         if (ident == "") {
             return reservasjonRepository.hent(uuid)
         }
         val hentIdentTilInnloggetBruker = azureGraphService.hentIdentTilInnloggetBruker()
-        val reservasjon = reservasjonRepository.hent(uuid)
-        saksbehandlerRepository.fjernReservasjon(reservasjon.reservertAv, reservasjon.oppgave)
-        saksbehandlerRepository.leggTilReservasjon(ident, reservasjon.oppgave)
-        return reservasjonRepository.lagre(uuid, true) {
+        val oppdatertReservasjon = reservasjonRepository.lagre(uuid, true) {
             if (it!!.reservertTil == null) {
                 it.reservertTil = LocalDateTime.now().plusHours(24).forskyvReservasjonsDato()
             } else {
@@ -646,6 +645,11 @@ class OppgaveTjeneste constructor(
             it.begrunnelse = begrunnelse
             it
         }
+        val reservasjon = reservasjonRepository.hent(uuid)
+        saksbehandlerRepository.fjernReservasjon(reservasjon.reservertAv, reservasjon.oppgave)
+        saksbehandlerRepository.leggTilReservasjon(ident, reservasjon.oppgave)
+
+        return oppdatertReservasjon
     }
 
     suspend fun hentSisteBehandledeOppgaver(): List<BehandletOppgave> {