Skip to content

Commit cbb5abb

Browse files
authored
Fix enrollments for existing users (#3099)
1 parent 91cf80a commit cbb5abb

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

openedx/api.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1048,14 +1048,15 @@ def enroll_in_edx_course_runs(
10481048
username = user.edx_username
10491049

10501050
try:
1051-
created_user, _ = repair_faulty_edx_user(user)
1052-
if created_user is False:
1053-
msg = f"User {user.edx_username} does not exist in OpenEdX and could not be created"
1054-
raise OpenEdxUserMissingError(msg) # noqa: TRY301
1051+
repair_faulty_edx_user(user)
10551052
except Exception as exc:
10561053
msg = f"Failed to verify/create user {user.edx_username} in OpenEdX"
10571054
raise OpenEdxUserMissingError(msg) from exc
10581055

1056+
if not user.openedx_user_exists:
1057+
msg = f"User {user.edx_username} does not exist in OpenEdX and could not be created"
1058+
raise OpenEdxUserMissingError(msg)
1059+
10591060
results = []
10601061
for course_run in course_runs:
10611062
try:

0 commit comments

Comments
 (0)