More #302: fix bug where study duration by end date was calculated in…

…correctly
MORE-Platform · Jun 4, 2024 · 02f8fca · 02f8fca
1 parent 9d26cd8
commit 02f8fca
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/studymanager/src/main/java/io/redlink/more/studymanager/service/StudyService.java b/studymanager/src/main/java/io/redlink/more/studymanager/service/StudyService.java
@@ -23,6 +23,8 @@
 import io.redlink.more.studymanager.repository.StudyGroupRepository;
 import io.redlink.more.studymanager.repository.StudyRepository;
 import io.redlink.more.studymanager.repository.UserRepository;
+
+import java.time.temporal.ChronoUnit;
 import java.util.EnumSet;
 import java.util.List;
 import java.util.Map;
@@ -205,11 +207,11 @@ public Optional<Duration> getStudyDuration(Long studyId) {
                 .flatMap(study ->
                         Optional.ofNullable(study.getDuration())
                                 .or(() -> Optional.of(new Duration()
-                                        .setValue(
-                                                java.time.Period.between(
+                                        .setValue(Long.valueOf(
+                                                 ChronoUnit.DAYS.between(
                                                         Objects.requireNonNullElse(study.getStartDate(), study.getPlannedStartDate()),
                                                         Objects.requireNonNullElse(study.getEndDate(), study.getPlannedEndDate())
-                                                ).getDays() + 1)
+                                                ) + 1).intValue())
                                         .setUnit(Duration.Unit.DAY)
                                 ))
                 );