Global: Add sorting to DMPs, Storage, Datasets and other entities by …

…ID to preserve order
tuwien-csd · Dec 2, 2024 · cee2442 · cee2442
1 parent 6fd4949
commit cee2442
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 12 deletions.
diff --git a/src/main/java/org/damap/base/rest/dmp/mapper/DmpDOMapper.java b/src/main/java/org/damap/base/rest/dmp/mapper/DmpDOMapper.java
@@ -1,9 +1,6 @@
 package org.damap.base.rest.dmp.mapper;
 
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Objects;
-import java.util.Optional;
+import java.util.*;
 import lombok.experimental.UtilityClass;
 import lombok.extern.jbosslog.JBossLog;
 import org.damap.base.domain.*;
@@ -74,7 +71,8 @@ public DmpDO mapEntityToDO(Dmp dmp, DmpDO dmpDO) {
     dmpDO.setDocumentation(dmp.getDocumentation());
 
     List<ContributorDO> contributorDOList = new ArrayList<>();
-    dmp.getContributorList()
+    dmp.getContributorList().stream()
+        .sorted(Comparator.comparing(contributor -> contributor.id))
         .forEach(
             contributor -> {
               ContributorDO contributorDO = new ContributorDO();
@@ -99,7 +97,8 @@ public DmpDO mapEntityToDO(Dmp dmp, DmpDO dmpDO) {
     dmpDO.setLegalRestrictionsDocuments(legalRestrictionsDocumentsDOList);
 
     List<DatasetDO> datasetDOList = new ArrayList<>();
-    dmp.getDatasetList()
+    dmp.getDatasetList().stream()
+        .sorted(Comparator.comparing(Dataset::getId))
         .forEach(
             dataset -> {
               DatasetDO datasetDO = new DatasetDO();
@@ -154,7 +153,8 @@ public DmpDO mapEntityToDO(Dmp dmp, DmpDO dmpDO) {
     dmpDO.setExternalStorage(externalStorageDOList);
 
     List<CostDO> costDOList = new ArrayList<>();
-    dmp.getCosts()
+    dmp.getCosts().stream()
+        .sorted(Comparator.comparing(cost -> cost.id))
         .forEach(
             cost -> {
               CostDO costDO = new CostDO();

diff --git a/src/main/java/org/damap/base/rest/dmp/service/DmpService.java b/src/main/java/org/damap/base/rest/dmp/service/DmpService.java
@@ -5,11 +5,7 @@
 import jakarta.transaction.Transactional;
 import jakarta.validation.Valid;
 import java.text.SimpleDateFormat;
-import java.util.ArrayList;
-import java.util.Date;
-import java.util.List;
-import java.util.Objects;
-import java.util.Optional;
+import java.util.*;
 import lombok.extern.jbosslog.JBossLog;
 import org.damap.base.domain.Access;
 import org.damap.base.domain.Contributor;
@@ -67,6 +63,9 @@ public List<DmpListItemDO> getAll() {
             dmpListItemDOList.add(
                 DmpListItemDOMapper.mapEntityToDO(
                     null, dmp, new DmpListItemDO(), versionService.getDmpVersions(dmp.id))));
+
+    dmpListItemDOList.sort(Comparator.comparing(DmpListItemDO::getId));
+
     return dmpListItemDOList;
   }
 
@@ -90,6 +89,9 @@ public List<DmpListItemDO> getDmpListByPersonId(String personId) {
                     access.getDmp(),
                     new DmpListItemDO(),
                     versionService.getDmpVersions(access.getDmp().id))));
+
+    dmpListItemDOS.sort(Comparator.comparing(DmpListItemDO::getId));
+
     return dmpListItemDOS;
   }
 

diff --git a/src/main/java/org/damap/base/rest/storage/InternalStorageService.java b/src/main/java/org/damap/base/rest/storage/InternalStorageService.java
@@ -9,6 +9,7 @@
 import jakarta.ws.rs.core.MultivaluedMap;
 import jakarta.ws.rs.core.Response;
 import java.util.ArrayList;
+import java.util.Comparator;
 import java.util.List;
 import java.util.Map;
 import lombok.extern.jbosslog.JBossLog;
@@ -179,6 +180,8 @@ public ResultList<InternalStorageDO> search(MultivaluedMap<String, String> query
               internalStorage, new InternalStorageDO(), translations));
     }
 
+    internalStorageDOList.sort(Comparator.comparingLong(InternalStorageDO::getId));
+
     Search search = Search.fromMap(queryParams);
 
     return ResultList.fromItemsAndSearch(internalStorageDOList, search);