fix a copy-paste issue on a dead element

gridsuite · Feb 22, 2024 · 6fe2244 · 6fe2244
1 parent 3b9d35a
commit 6fe2244
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/src/main/java/org/gridsuite/explore/server/services/ExploreService.java b/src/main/java/org/gridsuite/explore/server/services/ExploreService.java
@@ -266,9 +266,12 @@ public void createNetworkModifications(List<ElementAttributes> modificationAttri
         // create all corresponding directory elements
         modificationAttributesList.forEach(m -> {
             final UUID newId = newModificationsUuids.get(m.getElementUuid());
-            ElementAttributes elementAttributes = new ElementAttributes(newId, m.getElementName(), MODIFICATION,
-                    null, userId, 0L, m.getDescription());
-            directoryService.createElementWithNewName(elementAttributes, parentDirectoryUuid, userId, true);
+            if (newId != null) {
+                // an Id may be null if a duplication could not succeed (ex: we provide a bad uuid)
+                ElementAttributes elementAttributes = new ElementAttributes(newId, m.getElementName(), MODIFICATION,
+                        null, userId, 0L, m.getDescription());
+                directoryService.createElementWithNewName(elementAttributes, parentDirectoryUuid, userId, true);
+            }
         });
     }
 }
diff --git a/src/test/java/org/gridsuite/explore/server/ExploreTest.java b/src/test/java/org/gridsuite/explore/server/ExploreTest.java
@@ -662,7 +662,11 @@ public void testGetMetadata() throws Exception {
     @Test
     @SneakyThrows
     public void testcreateNetworkModifications() {
-        final String body = mapper.writeValueAsString(List.of(new ElementAttributes(MODIFICATION_UUID, "one modif", "", null, USER1, 0L, "a description")));
+        final String body = mapper.writeValueAsString(List.of(
+                new ElementAttributes(MODIFICATION_UUID, "one modif", "", null, USER1, 0L, "a description"),
+                new ElementAttributes(UUID.randomUUID(), "2nd modif", "", null, USER1, 0L, "a description")
+                )
+        );
         mockMvc.perform(post("/v1/explore/modifications?parentDirectoryUuid={parentDirectoryUuid}", PARENT_DIRECTORY_UUID)
                 .header("userId", USER1)
                 .contentType(MediaType.APPLICATION_JSON)