springboot3.5.6 (#205)

Signed-off-by: Abdelsalem <[email protected]>

Author: AbdelHedhili

pom.xml

@@ -42,8 +42,8 @@
     </developers>
 
     <properties>
+        <gridsuite-dependencies.version>45.0.0</gridsuite-dependencies.version>
         <jib.from.image>powsybl/java-dynawo:3.1.0</jib.from.image>
-        <gridsuite-dependencies.version>44.1.0</gridsuite-dependencies.version>
         <liquibase-hibernate-package>org.gridsuite.securityanalysis.server</liquibase-hibernate-package>
         <db-util.version>1.0.5</db-util.version>
         <mockwebserver3.version>5.0.0-alpha.14</mockwebserver3.version>

src/main/java/org/gridsuite/securityanalysis/server/repositories/specifications/PreContingencyLimitViolationSpecificationBuilder.java

@@ -25,7 +25,7 @@ public Specification<PreContingencyLimitViolationEntity> resultUuidEquals(UUID v
     }
 
     public Specification<PreContingencyLimitViolationEntity> buildSpecification(UUID resultUuid, List<ResourceFilterDTO> resourceFilters) {
-        Specification<PreContingencyLimitViolationEntity> specification = Specification.where(resultUuidEquals(resultUuid));
+        Specification<PreContingencyLimitViolationEntity> specification = resultUuidEquals(resultUuid);
 
         return SpecificationUtils.appendFiltersToSpecification(specification, resourceFilters);
     }

src/main/java/org/gridsuite/securityanalysis/server/service/SecurityAnalysisResultService.java

@@ -321,14 +321,30 @@ public Page<ContingencyEntity> findContingenciesPage(UUID resultUuid, UUID netwo
         // HHH000104: firstResult/maxResults specified with collection fetch; applying in memory!
         // cf. https://vladmihalcea.com/fix-hibernate-hhh000104-entity-fetch-pagination-warning-message/
         // We must separate in two requests, one with pagination the other one with Join Fetch
+        // Determine which properties to project based on sort fields
+        // When using DISTINCT, all ORDER BY columns must be in the SELECT list
+        List<String> projectionProperties = new ArrayList<>();
+        projectionProperties.add("uuid");
+
+        // Add sort properties to projection to satisfy DISTINCT + ORDER BY requirement
+        for (Sort.Order order : modifiedPageable.getSort()) {
+            String property = order.getProperty();
+            if (!property.equals("uuid") && !projectionProperties.contains(property)) {
+                projectionProperties.add(property);
+            }
+        }
 
         // First, we fetch contingencies UUIDs, with all the filters and pagination
-        Page<ContingencyRepository.EntityUuid> uuidPage = contingencyRepository.findBy(specification, q ->
-            q.project("uuid")
-                .as(ContingencyRepository.EntityUuid.class)
-                .sortBy(modifiedPageable.getSort())
-                .page(modifiedPageable)
-        );
+        Page<ContingencyRepository.EntityUuid> uuidPage = contingencyRepository.findBy(specification, q -> {
+            var query = q.as(ContingencyRepository.EntityUuid.class)
+                    .sortBy(modifiedPageable.getSort());
+            if (projectionProperties.size() == 1) {
+                query = query.project("uuid");
+            } else {
+                query = query.project(projectionProperties.toArray(new String[0]));
+            }
+            return query.page(modifiedPageable);
+        });
 
         if (!uuidPage.hasContent()) {
             // Since springboot 3.2, the return value of Page.empty() is not serializable. See https://github.com/spring-projects/spring-data-commons/issues/2987
@@ -365,12 +381,30 @@ public Page<SubjectLimitViolationEntity> findSubjectLimitViolationsPage(UUID res
         // HHH000104: firstResult/maxResults specified with collection fetch; applying in memory!
         // cf. https://vladmihalcea.com/fix-hibernate-hhh000104-entity-fetch-pagination-warning-message/
         // We must separate in two requests, one with pagination the other one with Join Fetch
-        Page<SubjectLimitViolationRepository.EntityId> uuidPage = subjectLimitViolationRepository.findBy(specification, q ->
-            q.project("id")
-                .as(SubjectLimitViolationRepository.EntityId.class)
-                .sortBy(modifiedPageable.getSort())
-                .page(modifiedPageable)
-        );
+
+        // Determine which properties to project based on sort fields
+        // When using DISTINCT, all ORDER BY columns must be in the SELECT list
+        List<String> projectionProperties = new ArrayList<>();
+        projectionProperties.add("id");
+
+        // Add sort properties to projection to satisfy DISTINCT + ORDER BY requirement
+        for (Sort.Order order : modifiedPageable.getSort()) {
+            String property = order.getProperty();
+            if (!property.equals("id") && !projectionProperties.contains(property)) {
+                projectionProperties.add(property);
+            }
+        }
+
+        Page<SubjectLimitViolationRepository.EntityId> uuidPage = subjectLimitViolationRepository.findBy(specification, q -> {
+            var query = q.as(SubjectLimitViolationRepository.EntityId.class)
+                .sortBy(modifiedPageable.getSort());
+            if (projectionProperties.size() == 1) {
+                query = query.project("id");
+            } else {
+                query = query.project(projectionProperties.toArray(new String[0]));
+            }
+            return query.page(modifiedPageable);
+        });
 
         if (!uuidPage.hasContent()) {
             // Since springboot 3.2, the return value of Page.empty() is not serializable. See https://github.com/spring-projects/spring-data-commons/issues/2987