Skip to content

Commit df59cc6

Browse files
fix errors in creating proc session on exam
1 parent 07ab43b commit df59cc6

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

webapp/shop/cred/views.py

+7-7
Original file line numberDiff line numberDiff line change
@@ -909,10 +909,6 @@ def cred_exam(trueability_api, proctor_api, **_):
909909
exam_date_time = None
910910
if not assessment_reservation:
911911
return flask.abort(403)
912-
# student_session = proctor_api.get_student_sessions(
913-
# {"ext_exam_id": assessment["id"]}
914-
# )
915-
# ext_exam_id = assessment["id"]
916912
else:
917913
student_session = proctor_api.get_student_sessions(
918914
{"ext_exam_id": assessment_reservation["uuid"]}
@@ -929,7 +925,7 @@ def cred_exam(trueability_api, proctor_api, **_):
929925
student_session_array = student_session.get("data", [{}])
930926
should_redirect = False
931927

932-
# session exists
928+
# if session exists
933929
if len(student_session_array) > 0:
934930
student_session = student_session_array[0]
935931
# create a new session if it does not exist
@@ -939,7 +935,7 @@ def cred_exam(trueability_api, proctor_api, **_):
939935
+ "credentials/exam?uuid="
940936
+ f"{assessment_reservation.get('uuid', '')}"
941937
)
942-
student_session = proctor_api.create_student_session(
938+
student_session_response = proctor_api.create_student_session(
943939
{
944940
"first_name": first_name,
945941
"last_name": last_name,
@@ -950,6 +946,7 @@ def cred_exam(trueability_api, proctor_api, **_):
950946
"exam_link": exam_link,
951947
}
952948
)
949+
student_session = student_session_response.get("data", None)
953950
should_redirect = True
954951

955952
if student_session is None or student_session.get("id", None) is None:
@@ -959,7 +956,10 @@ def cred_exam(trueability_api, proctor_api, **_):
959956
should_redirect
960957
or student_session.get("status", "not started") == "not started"
961958
):
962-
return flask.redirect(student_session["display_session_link"])
959+
if student_session.get("display_session_link"):
960+
return flask.redirect(student_session["display_session_link"])
961+
if exam_link:
962+
return flask.redirect(exam_link)
963963

964964
assessment_user = assessment["user"]["email"]
965965
sso_user = user_info(flask.session)["email"]

0 commit comments

Comments
 (0)