From 7db17ec7cfa831138bd6b3b9babe50453a162b4e Mon Sep 17 00:00:00 2001
From: JeremiahUy <jeremiah.augie.uy@nav.no>
Date: Thu, 31 Oct 2024 16:11:04 +0100
Subject: [PATCH] UPDATE: id check

---
 .../java/no/nav/data/pvk/pvkdokument/PvkDokumentController.java | 1 +
 .../java/no/nav/data/pvk/pvkdokument/PvkDokumentService.java    | 1 -
 .../no/nav/data/pvk/pvkdokument/dto/PvkDokumentRequest.java     | 2 +-
 3 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentController.java b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentController.java
index e39cbba97..6bd369b81 100644
--- a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentController.java
+++ b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentController.java
@@ -80,6 +80,7 @@ public ResponseEntity<PvkDokumentResponse> getPvkDokumentByEtterlevelseDokumentI
     @PostMapping
     public ResponseEntity<PvkDokumentResponse> createPvkDokumente(@RequestBody PvkDokumentRequest request) {
         log.info("Create PvkDokument");
+
         var pvkDokument = pvkDokumentService.save(request.convertToPvkDokument(), request.isUpdate());
 
         return new ResponseEntity<>(PvkDokumentResponse.buildFrom(pvkDokument), HttpStatus.CREATED);
diff --git a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentService.java b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentService.java
index 757a5110c..8d4650088 100644
--- a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentService.java
+++ b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/PvkDokumentService.java
@@ -51,7 +51,6 @@ public Optional<PvkDokument> getByEtterlevelseDokumentasjon(String etterlevelseD
 
     @Transactional(propagation = Propagation.REQUIRED)
     public PvkDokument save(PvkDokument pvkDokument, boolean isUpdate) {
-
         if (!isUpdate) {
             var existingPvkDokument = getByEtterlevelseDokumentasjon(pvkDokument.getEtterlevelseDokumentId());
             if (existingPvkDokument.isPresent()) {
diff --git a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/dto/PvkDokumentRequest.java b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/dto/PvkDokumentRequest.java
index adf92a4f5..52c0a1f31 100644
--- a/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/dto/PvkDokumentRequest.java
+++ b/apps/backend/src/main/java/no/nav/data/pvk/pvkdokument/dto/PvkDokumentRequest.java
@@ -92,7 +92,7 @@ public PvkDokument convertToPvkDokument() {
                 .build();
 
         return PvkDokument.builder()
-                .id(id != null ? UUID.fromString(id) : null)
+                .id(id != null && !id.isEmpty() ? UUID.fromString(id) : null)
                 .etterlevelseDokumentId(etterlevelseDokumentId)
                 .status(status != null ? status : PvkDokumentStatus.AKTIV)
                 .pvkDokumentData(pkvDokumentData)