Flytt logikk så ingen sletting kjører før validering av nye mottakere…

… er OK
navikt · Feb 13, 2025 · 1825a41 · 1825a41
1 parent 32432c7
commit 1825a41
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt b/apps/etterlatte-brev-api/src/main/kotlin/no/nav/etterlatte/brev/BrevService.kt
@@ -311,17 +311,18 @@ class BrevService(
         val brev = sjekkOmBrevKanEndres(brevId)
         logger.info("Tilbakestiller mottakere for brev=$brevId")
         val personerISakOgSak = brevdataFacade.hentPersonerISakforBrev(brev.sakId, brev.behandlingId, bruker)
-        brev.mottakere.forEach { mottaker ->
-            db.slettMottaker(brev.id, mottaker.id, bruker)
-        }
-        val mottakere = adresseService.hentMottakere(personerISakOgSak.sak.sakType, personerISakOgSak.personerISak, bruker)
-        if (mottakere.isEmpty()) {
+        val nyeMottakere = adresseService.hentMottakere(personerISakOgSak.sak.sakType, personerISakOgSak.personerISak, bruker)
+        if (nyeMottakere.isEmpty()) {
             throw KanIkkeTilbakestilleUtenNyeMottakere()
         }
-        if (!mottakere.any { it.type == MottakerType.HOVED }) {
+        if (!nyeMottakere.any { it.type == MottakerType.HOVED }) {
             throw KanIkkeSletteHovedmottaker()
         }
-        mottakere.forEach { mottaker ->
+        // bare slett hvis testene går gjennom
+        brev.mottakere.forEach { mottaker ->
+            db.slettMottaker(brev.id, mottaker.id, bruker)
+        }
+        nyeMottakere.forEach { mottaker ->
             db.opprettMottaker(brev.id, mottaker, bruker)
         }