chore(deps): bump openapi-generator-maven-plugin to v7.7.0

pom.xml

@@ -315,7 +315,7 @@
             <plugin>
                 <groupId>org.openapitools</groupId>
                 <artifactId>openapi-generator-maven-plugin</artifactId>
-                <version>6.6.0</version>
+                <version>7.7.0</version>
                 <executions>
                     <execution>
                         <id>data-api</id>
@@ -360,6 +360,19 @@
                     <generateModelTests>false</generateModelTests>
                     <generateSupportingFiles>false</generateSupportingFiles>
 
+                    <typeMappings>
+                        <typeMapping>string+date-time=Instant</typeMapping>
+                        <typeMapping>string+time=LocalTime</typeMapping>
+                    </typeMappings>
+                    <schemaMappings>
+                        <schemaMapping>Instant=java.time.Instant</schemaMapping>
+                        <schemaMapping>LocalTime=java.time.LocalTime</schemaMapping>
+                    </schemaMappings>
+                    <importMappings>
+                        <importMapping>Instant=java.time.Instant</importMapping>
+                        <importMapping>LocalTime=java.time.LocalTime</importMapping>
+                    </importMappings>
+
                     <configOptions>
                         <useSpringBoot3>true</useSpringBoot3>
                         <sourceFolder>src</sourceFolder>
@@ -368,6 +381,7 @@
                         <useTags>true</useTags>
                         <openApiNullable>false</openApiNullable>
                         <skipDefaultInterface>true</skipDefaultInterface>
+                        <requestMappingMode>api_interface</requestMappingMode>
                     </configOptions>
                 </configuration>
             </plugin>

src/main/java/io/redlink/more/data/controller/ExternalDataApiV1Controller.java

@@ -68,7 +68,7 @@ public ResponseEntity<Void> storeExternalBulk(String moreApiToken, EndpointDataB
             Interval interval = externalService.getIntervalForObservation(apiRoutingInfo.studyId(), apiRoutingInfo.observationId(), participantId);
 
             endpointDataBulkDTO.getDataPoints().stream()
-                .map(datapoint -> datapoint.getTimestamp().toInstant())
+                .map(ExternalDataDTO::getTimestamp)
                 .map(timestamp -> timestamp.isBefore(interval.getStart()) || timestamp.isAfter(interval.getEnd()))
                 .filter(v -> v)
                 .findFirst()

src/main/java/io/redlink/more/data/controller/transformer/BaseTransformers.java

@@ -4,8 +4,6 @@
 package io.redlink.more.data.controller.transformer;
 
 import java.time.Instant;
-import java.time.OffsetDateTime;
-import java.time.ZoneOffset;
 
 public final class BaseTransformers {
 
@@ -16,14 +14,4 @@ public static Long toVersionTag(Instant modified) {
         return modified.toEpochMilli();
     }
 
-    public static Instant toInstant(OffsetDateTime dateTime) {
-        if (dateTime == null)
-            return null;
-        return dateTime.toInstant();
-    }
-
-    public static OffsetDateTime toOffsetDateTime(Instant instant) {
-        if (instant == null) return null;
-        return instant.atOffset(ZoneOffset.UTC);
-    }
 }

src/main/java/io/redlink/more/data/controller/transformer/DataTransformer.java

@@ -26,13 +26,14 @@ public static List<DataPoint> createDataPoints(DataBulkDTO bulk) {
     }
 
     public static DataPoint createDataPoint(ObservationDataDTO dataPoint, Instant recordingTime) {
+        Instant dateTime = dataPoint.getTimestamp();
         return new DataPoint(
                 dataPoint.getDataId(),
                 dataPoint.getObservationId(),
                 dataPoint.getObservationType(),
                 dataPoint.getObservationType(),
                 recordingTime,
-                BaseTransformers.toInstant(dataPoint.getTimestamp()),
+                dateTime,
                 dataPoint.getDataValue());
     }
 
@@ -44,13 +45,14 @@ public static List<DataPoint> createDataPoints(EndpointDataBulkDTO bulk, ApiRout
     }
 
     public static DataPoint createDataPoint(ExternalDataDTO dataPoint, ApiRoutingInfo routingInfo, Instant recordingTime, Integer observationId) {
+        Instant dateTime = dataPoint.getTimestamp();
         return new DataPoint(
                 UUID.randomUUID().toString(),
                 observationId.toString(),
                 routingInfo.observationType(),
                 routingInfo.observationType(),
                 recordingTime,
-                BaseTransformers.toInstant(dataPoint.getTimestamp()),
+                dateTime,
                 dataPoint.getDataValue());
     }
 }

src/main/java/io/redlink/more/data/controller/transformer/ParticipantTransformer.java

@@ -18,7 +18,7 @@ public static ParticipantDTO toDTO(Participant participant) {
                 participant.alias(),
                 ParticipantStatusDTO.fromValue(participant.status()),
                 toGroupDto(participant),
-                BaseTransformers.toOffsetDateTime(participant.start())
+                participant.start()
         );
     }
 

src/main/java/io/redlink/more/data/controller/transformer/StudyTransformer.java

@@ -84,9 +84,11 @@ public static ObservationDTO toDTO(Observation observation, Instant start, Insta
     }
 
     public static ObservationScheduleDTO toObservationScheduleDTO(Range<Instant> schedule) {
+        Instant instant = schedule.getMaximum();
+        Instant instant1 = schedule.getMinimum();
         return new ObservationScheduleDTO()
-                .start(BaseTransformers.toOffsetDateTime(schedule.getMinimum()))
-                .end(BaseTransformers.toOffsetDateTime(schedule.getMaximum()))
+                .start(instant1)
+                .end(instant)
                 ;
     }
 

src/main/java/io/redlink/more/data/repository/NotificationRepository.java

@@ -11,16 +11,17 @@
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import io.redlink.more.data.api.app.v1.model.PushNotificationDTO;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Timestamp;
+import java.time.Instant;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
 import org.springframework.jdbc.core.RowMapper;
 import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate;
 import org.springframework.stereotype.Component;
 
-import java.sql.ResultSet;
-import java.sql.SQLException;
-import java.time.OffsetDateTime;
-import java.time.ZoneOffset;
-import java.util.*;
-
 @Component
 public class NotificationRepository {
 
@@ -68,10 +69,10 @@ private static RowMapper<PushNotificationDTO> getRowMapper() {
         };
     }
 
-    private static OffsetDateTime getTimestamp(ResultSet rs) {
+    private static Instant getTimestamp(ResultSet rs) {
         try {
             return Optional.ofNullable(rs.getTimestamp("timestamp"))
-                    .map(d -> d.toInstant().atOffset(ZoneOffset.UTC))
+                    .map(Timestamp::toInstant)
                     .orElse(null);
         } catch (SQLException e) {
             return null;