sartography/cr-connect-workflow
1
0
Fork 0
mirror of https://github.com/sartography/cr-connect-workflow.git synced 2025-02-22 20:58:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added a comment on why we moved the session.add(study) line

Browse Source
This commit is contained in:
mike cullerton 2022-03-17 15:18:49 -04:00
parent db6f594ba9
commit 15750beb10
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
crc/services/study_service.py
View File

@ -411,6 +411,7 @@ class StudyService(object):
new_status = StudyStatus.in_progress
new_progress_status = ProgressStatus.in_progress
# we use add_study below to determine whether we add the study to the session
add_study = True
db_studies.append(db_study)
@ -426,6 +427,8 @@ class StudyService(object):
StudyService.add_study_update_event(db_study,
status=new_status,
event_type=StudyEventType.automatic)
# we moved session.add here so that it comes after we update the study
# we only add if it doesnt already exist in the DB
if add_study:
session.add(db_study)