Small fix for change of enhet for arbeidsliste (#1642)

* Small changes for huskelapp & fargekategori
navikt · Jul 3, 2024 · f1c1de1 · f1c1de1
1 parent 4e935f9
commit f1c1de1
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 10 deletions.
diff --git a/src/main/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslisteService.java b/src/main/java/no/nav/pto/veilarbportefolje/arbeidsliste/ArbeidslisteService.java
@@ -156,8 +156,7 @@ public void oppdaterEnhetPaaArbeidsliste(Fnr fnr, EnhetId enhetId, VeilederId ve
                     Timestamp.from(now()),
                     enhetId.get()
             );
-            arbeidslisteRepositoryV2.updateArbeidslisteUtenFargekategori(arbeidslisteDTO)
-                    .onSuccess(opensearchIndexerV2::updateArbeidsliste);
+            arbeidslisteRepositoryV2.updateArbeidslisteUtenFargekategori(arbeidslisteDTO);
         }
     }
 

diff --git a/src/main/java/no/nav/pto/veilarbportefolje/fargekategori/FargekategoriController.java b/src/main/java/no/nav/pto/veilarbportefolje/fargekategori/FargekategoriController.java
@@ -37,10 +37,10 @@ public class FargekategoriController {
     @PostMapping("/hent-fargekategori")
     public ResponseEntity<FargekategoriEntity> hentFargekategoriForBruker(@RequestBody HentFargekategoriRequest request) {
         validerRequest(request.fnr);
-        NavKontor navKontorForBruker = brukerServiceV2.hentNavKontor(request.fnr).orElseThrow(() -> new ResponseStatusException(HttpStatus.FORBIDDEN, "Bruker har ikke tilordnet enhet"));
 
         authService.innloggetVeilederHarTilgangTilOppfolging();
         authService.innloggetVeilederHarTilgangTilBruker(request.fnr.get());
+        NavKontor navKontorForBruker = brukerServiceV2.hentNavKontor(request.fnr).orElseThrow(() -> new ResponseStatusException(HttpStatus.FORBIDDEN, "Bruker har ikke tilordnet enhet"));
         authService.innloggetVeilederHarTilgangTilEnhet(navKontorForBruker.getValue());
 
         try {

diff --git a/src/main/java/no/nav/pto/veilarbportefolje/huskelapp/controller/HuskelappController.java b/src/main/java/no/nav/pto/veilarbportefolje/huskelapp/controller/HuskelappController.java
@@ -145,18 +145,16 @@ public record HentErBrukerUfordelt(Fnr fnr) {
     }
 
     private void validerOppfolgingOgBrukerOgEnhet(String fnr) {
-        Optional<NavKontor> navKontor = brukerServiceV2.hentNavKontor(Fnr.of(fnr));
-        if (navKontor.isEmpty()) {
+        Validation<String, Fnr> validateFnr = ValideringsRegler.validerFnr(fnr);
+        if (validateFnr.isInvalid()) {
             throw new ResponseStatusException(HttpStatus.BAD_REQUEST);
         }
 
         authService.innloggetVeilederHarTilgangTilOppfolging();
-        Validation<String, Fnr> validateFnr = ValideringsRegler.validerFnr(fnr);
         authService.innloggetVeilederHarTilgangTilBruker(fnr);
-        authService.innloggetVeilederHarTilgangTilEnhet(navKontor.get().getValue());
-        if (validateFnr.isInvalid()) {
-            throw new ResponseStatusException(HttpStatus.BAD_REQUEST);
-        }
+
+        NavKontor navKontor = brukerServiceV2.hentNavKontor(Fnr.of(fnr)).orElseThrow(() -> new ResponseStatusException(HttpStatus.BAD_REQUEST));
+        authService.innloggetVeilederHarTilgangTilEnhet(navKontor.getValue());
     }
 
     private HuskelappResponse mapToHuskelappResponse(Huskelapp huskelapp) {