Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #13357 from edx/aj/ECOM-5405-update-bulk-enroll-mgm

Browse Source 
Update each enrollment in atomic transaction not all
This commit is contained in:
Awais Jibran
2016-09-05 20:22:50 +05:00
committed by GitHub
parent 79085fff73 896b3cb170
commit 0bc4f4e026
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
common/djangoapps/student/management/commands/bulk_change_enrollment.py
View File

@@ -91,9 +91,10 @@ class Command(BaseCommand):
if not commit:
logger.info('Dry run, changes have not been saved. Run again with "commit" argument to save changes')
raise Exception('The --commit flag was not given; forcing rollback.')
with transaction.atomic():
# call `change_mode` which will change the mode and also emit tracking event
for enrollment in course_enrollments:
# call `change_mode` which will change the mode and also emit tracking event
for enrollment in course_enrollments:
with transaction.atomic():
enrollment.change_mode(mode=to_mode)
logger.info('Finished moving users from %s to %s in course %s.', from_mode, to_mode, course_key_str)