Merge pull request #1601 from navikt/dev

Small fixes

Author: dragutin-nav

src/main/java/no/nav/pto/veilarbportefolje/huskelapp/HuskelappRepository.java

@@ -110,6 +110,10 @@ public List<Huskelapp> hentAlleRaderPaHuskelapp(UUID huskelappId) {
         return dbReadOnly.queryForList(sql, huskelappId).stream().map(HuskelappRepository::huskelappMapper).toList();
     }
 
+    public void slettAlleHuskelappRaderPaaBruker(Fnr fnr) {
+        String sql = String.format("DELETE FROM %s WHERE %s=? ", TABLE_NAME, FNR);
+        db.update(sql, fnr.get());
+    }
 
     public void deaktivereAlleHuskelappRaderPaaBruker(Fnr fnr) {
         String sql = String.format("UPDATE %s SET %s = ? WHERE %s = ? AND %s = ? ", TABLE_NAME, STATUS, FNR, STATUS);

src/main/java/no/nav/pto/veilarbportefolje/huskelapp/HuskelappService.java

@@ -100,11 +100,11 @@ public void settHuskelappIkkeAktiv(UUID huskelappId, Fnr brukerFnr) {
         }
     }
 
-    public void deaktivereAlleHuskelapperPaaBruker(AktorId aktorId, Optional<Fnr> maybeFnr) {
+    public void sletteAlleHuskelapperPaaBruker(AktorId aktorId, Optional<Fnr> maybeFnr) {
         try {
             secureLog.info("Sletter alle huskelapper paa bruker med aktoerid: " + aktorId);
             if (maybeFnr.isPresent()) {
-                huskelappRepository.deaktivereAlleHuskelappRaderPaaBruker(maybeFnr.get());
+                huskelappRepository.slettAlleHuskelappRaderPaaBruker(maybeFnr.get());
                 opensearchIndexerV2.slettHuskelapp(aktorId);
             } else {
                 secureLog.warn("Kunne ikke slette huskelapper for bruker med AktørID {}. Årsak fødselsnummer-parameter var tom.", aktorId.get());
@@ -115,6 +115,21 @@ public void deaktivereAlleHuskelapperPaaBruker(AktorId aktorId, Optional<Fnr> ma
         }
     }
 
+    public void deaktivereAlleHuskelapperPaaBruker(AktorId aktorId, Optional<Fnr> maybeFnr) {
+        try {
+            secureLog.info("Deaktiverer alle huskelapper paa bruker med aktoerid: " + aktorId);
+            if (maybeFnr.isPresent()) {
+                huskelappRepository.deaktivereAlleHuskelappRaderPaaBruker(maybeFnr.get());
+                opensearchIndexerV2.slettHuskelapp(aktorId);
+            } else {
+                secureLog.warn("Kunne ikke deaktivere huskelapper for bruker med AktørID {}. Årsak fødselsnummer-parameter var tom.", aktorId.get());
+            }
+        } catch (Exception e) {
+            secureLog.error("Kunne ikke deaktivere huskelapper for aktoerId: " + aktorId.toString());
+            throw new RuntimeException("Kunne ikke deaktivere huskelapper", e);
+        }
+    }
+
     public boolean brukerHarHuskelappPaForrigeNavkontor(AktorId aktoerId) {
 
         Fnr fnrBruker = aktorClient.hentFnr(aktoerId);

src/main/java/no/nav/pto/veilarbportefolje/oppfolging/OppfolgingAvsluttetService.java

@@ -56,7 +56,7 @@ public void avsluttOppfolging(AktorId aktoerId) {
         arbeidssokerRegistreringService.slettRegistering(aktoerId);
         endringIArbeidssokerRegistreringService.slettEndringIRegistering(aktoerId);
         arbeidslisteService.slettArbeidsliste(aktoerId, maybeFnr);
-        huskelappService.deaktivereAlleHuskelapperPaaBruker(aktoerId, maybeFnr);
+        huskelappService.sletteAlleHuskelapperPaaBruker(aktoerId, maybeFnr);
         sisteEndringService.slettSisteEndringer(aktoerId);
         cvRepositoryV2.resetHarDeltCV(aktoerId);
         siste14aVedtakService.slettSiste14aVedtak(aktoerId.get());

src/test/java/no/nav/pto/veilarbportefolje/huskelapp/HuskelappRepositoryTest.java

@@ -165,7 +165,7 @@ public void sletterAlleHuskelappRader() {
         List<Huskelapp> alleHuskelappRader = repo.hentAlleRaderPaHuskelapp(huskelappFoer.get().huskelappId());
         assertThat(alleHuskelappRader.size()).isEqualTo(4);
 
-        repo.deaktivereAlleHuskelappRaderPaaBruker(huskelapp2.brukerFnr());
+        repo.slettAlleHuskelappRaderPaaBruker(huskelapp2.brukerFnr());
 
         List<Huskelapp> alleHuskelappRader2 = repo.hentAlleRaderPaHuskelapp(huskelappFoer.get().huskelappId());
         assertThat(alleHuskelappRader2.size()).isEqualTo(0);