Skip to content

Commit e0f1ce4

Browse files
fix errors in creating proc session on exam
1 parent 5249b4f commit e0f1ce4

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
@@ -915,10 +915,6 @@ def cred_exam(trueability_api, proctor_api, **_):
915915
exam_date_time = None
916916
if not assessment_reservation:
917917
return flask.abort(403)
918-
# student_session = proctor_api.get_student_sessions(
919-
# {"ext_exam_id": assessment["id"]}
920-
# )
921-
# ext_exam_id = assessment["id"]
922918
else:
923919
student_session = proctor_api.get_student_sessions(
924920
{"ext_exam_id": assessment_reservation["uuid"]}
@@ -935,7 +931,7 @@ def cred_exam(trueability_api, proctor_api, **_):
935931
student_session_array = student_session.get("data", [{}])
936932
should_redirect = False
937933

938-
# session exists
934+
# if session exists
939935
if len(student_session_array) > 0:
940936
student_session = student_session_array[0]
941937
# create a new session if it does not exist
@@ -945,7 +941,7 @@ def cred_exam(trueability_api, proctor_api, **_):
945941
+ "credentials/exam?uuid="
946942
+ f"{assessment_reservation.get('uuid', '')}"
947943
)
948-
student_session = proctor_api.create_student_session(
944+
student_session_response = proctor_api.create_student_session(
949945
{
950946
"first_name": first_name,
951947
"last_name": last_name,
@@ -956,6 +952,7 @@ def cred_exam(trueability_api, proctor_api, **_):
956952
"exam_link": exam_link,
957953
}
958954
)
955+
student_session = student_session_response.get("data", None)
959956
should_redirect = True
960957

961958
if student_session is None or student_session.get("id", None) is None:
@@ -965,7 +962,10 @@ def cred_exam(trueability_api, proctor_api, **_):
965962
should_redirect
966963
or student_session.get("status", "not started") == "not started"
967964
):
968-
return flask.redirect(student_session["display_session_link"])
965+
if student_session.get("display_session_link"):
966+
return flask.redirect(student_session["display_session_link"])
967+
if exam_link:
968+
return flask.redirect(exam_link)
969969

970970
assessment_user = assessment["user"]["email"]
971971
sso_user = user_info(flask.session)["email"]

0 commit comments

Comments
 (0)