Improve empty value handeling

FINTLabs · Nov 5, 2024 · 42eac23 · 42eac23
1 parent 88a3213
commit 42eac23
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/main/java/no/fintlabs/model/personopplysning/PersonopplysningMappingService.java b/src/main/java/no/fintlabs/model/personopplysning/PersonopplysningMappingService.java
@@ -8,6 +8,7 @@
 import no.fint.model.resource.personvern.kodeverk.PersonopplysningResource;
 import no.fintlabs.model.personopplysning.model.PersonopplysningEntity;
 import org.springframework.stereotype.Service;
+import org.springframework.util.StringUtils;
 
 import java.util.Date;
 import java.util.List;
@@ -28,11 +29,14 @@ public static PersonopplysningResource toResource(PersonopplysningEntity persono
         personopplysningResource.setGyldighetsperiode(mapToPeriode(personopplysningEntity.getStartGyldighetsdato(), personopplysningEntity.getEndGyldighetsdato(), personopplysningEntity.getBeskrivelseGyldighetsPeriode()));
         personopplysningResource.setSystemId(identifikator);
 
-        try {
-            String links = personopplysningEntity.getLinks();
-            personopplysningResource.setLinks(objectMapper.readValue(links, new TypeReference<Map<String, List<Link>>>() {}));
-        } catch (Exception e) {
-            throw new RuntimeException("Could not parse links" + e.getMessage());
+        if (StringUtils.hasText(personopplysningEntity.getLinks())) {
+            try {
+                String links = personopplysningEntity.getLinks();
+                personopplysningResource.setLinks(objectMapper.readValue(links, new TypeReference<Map<String, List<Link>>>() {
+                }));
+            } catch (Exception e) {
+                throw new RuntimeException("Could not parse links: " + e.getMessage());
+            }
         }
 
         return personopplysningResource;
@@ -54,7 +58,7 @@ public static PersonopplysningEntity toEntity(PersonopplysningResource personopp
             personopplysningEntity.setPassiv(personopplysningResource.getPassiv());
         personopplysningEntity.setIdentifikatorVerdi(personopplysningResource.getSystemId().getIdentifikatorverdi());
 
-        if(personopplysningResource.getLinks() != null || !personopplysningResource.getLinks().isEmpty()) {
+        if (personopplysningResource.getLinks() != null && !personopplysningResource.getLinks().isEmpty()) {
             try {
                 Map<String, List<Link>> links = personopplysningResource.getLinks();
                 personopplysningEntity.setLinks(objectMapper.writeValueAsString(links));