Merge pull request #288 from MORE-Platform/#224-fix-data-cleanup

#224: fix data cleanup bug on study-transitions.
MORE-Platform · May 27, 2024 · f29335b · f29335b
2 parents 1b12217 + 4f75480
commit f29335b
Showing 1 changed file with 4 additions and 1 deletion.
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
@@ -142,7 +142,10 @@ public void setStatus(Long studyId, Study.Status newState, User user) {
                             )
                         );
                         participantService.alignParticipantsWithStudyState(s);
-                        elasticService.deleteIndex(s.getStudyId());
+                        if (s.getStudyState() == Study.Status.DRAFT) {
+                            log.info("Study {} transitioned back to {}, dropping collected observation data", study.getStudyId(), s.getStudyState());
+                            elasticService.deleteIndex(s.getStudyId());
+                        }
                     } catch (Exception e) {
                         log.warn("Could not set new state for study id {}; old state: {}; new state: {}", studyId, oldState.getValue(), s.getStudyState().getValue());
                         //ROLLBACK