Refktorert Kommuner2024

The logic for filtering "kommune" codes based on the year 2024 has been shifted to the KodeverkConsumer class from the Kodeverk controller class. A utility function filterKommuner2024 has also been updated to work with KodeverkBetydningerResponse type instead of KodeverkDTO and KodeverkAdjustedDTO. This change improves the structure and organization of the code.

Author: krharum

apps/kodeverk-service/src/main/java/no/nav/testnav/kodeverkservice/consumer/KodeverkConsumer.java

@@ -3,13 +3,17 @@
 import no.nav.testnav.kodeverkservice.config.Consumers;
 import no.nav.testnav.kodeverkservice.consumer.command.KodeverkGetCommand;
 import no.nav.testnav.kodeverkservice.dto.KodeverkBetydningerResponse;
+import no.nav.testnav.kodeverkservice.utility.KommunerUtility;
 import no.nav.testnav.libs.reactivesecurity.exchange.TokenExchange;
 import no.nav.testnav.libs.securitycore.domain.ServerProperties;
 import org.springframework.stereotype.Service;
 import org.springframework.web.reactive.function.client.ExchangeStrategies;
 import org.springframework.web.reactive.function.client.WebClient;
 import reactor.core.publisher.Mono;
 
+import static no.nav.testnav.kodeverkservice.utility.KommunerUtility.KOMMUNER;
+import static no.nav.testnav.kodeverkservice.utility.KommunerUtility.KOMMUNER2024;
+
 @Service
 public class KodeverkConsumer {
 
@@ -39,6 +43,10 @@ public KodeverkConsumer(
     public Mono<KodeverkBetydningerResponse> getKodeverk(String kodeverk) {
 
         return tokenService.exchange(serverProperties)
-                .flatMap(token -> new KodeverkGetCommand(webClient, kodeverk, token.getTokenValue()).call());
+                .flatMap(token -> new KodeverkGetCommand(webClient,
+                        !KOMMUNER2024.equals(kodeverk) ? kodeverk : KOMMUNER,
+                        token.getTokenValue()).call())
+                .map(response -> !KOMMUNER2024.equals(kodeverk) ? response :
+                        KommunerUtility.filterKommuner2024(response));
     }
 }

apps/kodeverk-service/src/main/java/no/nav/testnav/kodeverkservice/provider/KodeverkController.java

@@ -3,7 +3,6 @@
 import io.swagger.v3.oas.annotations.Operation;
 import lombok.RequiredArgsConstructor;
 import no.nav.testnav.kodeverkservice.service.KodeverkService;
-import no.nav.testnav.kodeverkservice.utility.KommunerUtility;
 import no.nav.testnav.libs.dto.kodeverkservice.v1.KodeverkAdjustedDTO;
 import no.nav.testnav.libs.dto.kodeverkservice.v1.KodeverkDTO;
 import org.springframework.cache.annotation.Cacheable;
@@ -17,8 +16,6 @@
 
 import static no.nav.testnav.kodeverkservice.config.CacheConfig.CACHE_KODEVERK;
 import static no.nav.testnav.kodeverkservice.config.CacheConfig.CACHE_KODEVERK_2;
-import static no.nav.testnav.kodeverkservice.utility.KommunerUtility.KOMMUNER;
-import static no.nav.testnav.kodeverkservice.utility.KommunerUtility.KOMMUNER2024;
 
 @RestController
 @RequestMapping("/api/v1/kodeverk")
@@ -33,24 +30,14 @@ public class KodeverkController {
     @Operation(description = "Hent kodeverk, returnerer map")
     public Mono<KodeverkDTO> fetchKodeverk(@RequestParam String kodeverk) {
 
-        return !KOMMUNER2024.equals(kodeverk) ?
-
-                kodeverkService.getKodeverkMap(kodeverk) :
-
-                kodeverkService.getKodeverkMap(KOMMUNER)
-                        .map(KommunerUtility::filterKommuner2024);
+        return kodeverkService.getKodeverkMap(kodeverk);
     }
 
     @Cacheable(value = CACHE_KODEVERK_2, unless = "#result.koder?.size() == 0")
     @GetMapping("/{kodeverkNavn}")
     @Operation(description = "Hent kodeverk etter kodeverkNavn")
     public Mono<KodeverkAdjustedDTO> getKodeverkByName(@PathVariable("kodeverkNavn") String kodeverkNavn) {
 
-        return !KOMMUNER2024.equals(kodeverkNavn) ?
-
-                kodeverkService.getKodeverkByName(kodeverkNavn) :
-
-                kodeverkService.getKodeverkByName(KOMMUNER)
-                        .map(KommunerUtility::filterKommuner2024);
+        return kodeverkService.getKodeverkByName(kodeverkNavn);
     }
 }

apps/kodeverk-service/src/main/java/no/nav/testnav/kodeverkservice/utility/KommunerUtility.java

@@ -2,8 +2,7 @@
 
 import lombok.Getter;
 import lombok.experimental.UtilityClass;
-import no.nav.testnav.libs.dto.kodeverkservice.v1.KodeverkAdjustedDTO;
-import no.nav.testnav.libs.dto.kodeverkservice.v1.KodeverkDTO;
+import no.nav.testnav.kodeverkservice.dto.KodeverkBetydningerResponse;
 
 import java.util.HashSet;
 import java.util.Map;
@@ -126,26 +125,12 @@ public class KommunerUtility {
         gamleKommunenummer.add("3019"); // Vestby
         gamleKommunenummer.add("3825"); // Vinje
     }
+    public static KodeverkBetydningerResponse filterKommuner2024(KodeverkBetydningerResponse response) {
 
-    public static KodeverkAdjustedDTO filterKommuner2024(KodeverkAdjustedDTO response) {
-
-        return KodeverkAdjustedDTO.builder()
-                .name(KOMMUNER2024)
-                .koder(
-                        response.getKoder().stream()
-                                .filter(kode -> !gamleKommunenummer.contains(kode.getValue()))
-                                .toList())
-                .build();
-    }
-
-    public static KodeverkDTO filterKommuner2024(KodeverkDTO response) {
-
-        return KodeverkDTO.builder()
-                .kodeverknavn(KOMMUNER2024)
-                .kodeverk(
-                        response.getKodeverk().entrySet().stream()
-                                .filter(entry ->!gamleKommunenummer.contains(entry.getKey()))
-                                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue)))
+        return KodeverkBetydningerResponse.builder()
+                .betydninger(response.getBetydninger().entrySet().stream()
+                        .filter(entry -> !gamleKommunenummer.contains(entry.getKey()))
+                        .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue)))
                 .build();
     }
 }