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

Merge pull request #29182 from edx/bseverino/hide-proctored-after-due

Browse Source 
MST-980: Hide proctored exams after due by default
This commit is contained in:
Bianca Severino
2021-11-01 14:39:23 -04:00
committed by GitHub
parent 7ae8ed37a3 6f4811d7e7
commit 1bb5685897
2 changed files with 28 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
cms/static/js/views/modals/course_outline_modals.js
View File

@@ -521,6 +521,8 @@ define(['jquery', 'backbone', 'underscore', 'gettext', 'js/views/baseview',
'change input.onboarding_exam': 'setSpecialExamWithoutRules',
'focusout .field-time-limit input': 'timeLimitFocusout'
},
startingExamType: '',
notTimedExam: function(event) {
event.preventDefault();
this.$('.exam-options').hide();
@@ -575,6 +577,7 @@ define(['jquery', 'backbone', 'underscore', 'gettext', 'js/views/baseview',
this.$('.field-exam-review-rules').hide();
if (!isTimeLimited) {
this.startingExamType = 'no_special_exam';
this.$('input.no_special_exam').prop('checked', true);
return;
}
@@ -583,10 +586,13 @@ define(['jquery', 'backbone', 'underscore', 'gettext', 'js/views/baseview',
if (this.options.enable_proctored_exams && isProctoredExam) {
if (isOnboardingExam) {
this.startingExamType = 'onboarding_exam';
this.$('input.onboarding_exam').prop('checked', true);
} else if (isPracticeExam) {
this.startingExamType = 'practice_exam';
this.$('input.practice_exam').prop('checked', true);
} else {
this.startingExamType = 'proctored_exam';
this.$('input.proctored_exam').prop('checked', true);
this.$('.field-exam-review-rules').show();
}
@@ -594,6 +600,7 @@ define(['jquery', 'backbone', 'underscore', 'gettext', 'js/views/baseview',
// Since we have an early exit at the top of the method
// if the subsection is not time limited, then
// here we rightfully assume that it just a timed exam
this.startingExamType = 'timed_exam';
this.$('input.timed_exam').prop('checked', true);
}
},
@@ -631,20 +638,27 @@ define(['jquery', 'backbone', 'underscore', 'gettext', 'js/views/baseview',
var timeLimit = this.getExamTimeLimit();
var examReviewRules = this.$('.field-exam-review-rules textarea').val();
var metadata = {
is_practice_exam: isPracticeExamChecked,
is_time_limited: !isNoSpecialExamChecked,
exam_review_rules: examReviewRules,
// We have to use the legacy field name
// as the Ajax handler directly populates
// the xBlocks fields. We will have to
// update this call site when we migrate
// seq_module.py to use 'is_proctored_exam'
is_proctored_enabled: isProctoredExamChecked || isPracticeExamChecked || isOnboardingExamChecked,
default_time_limit_minutes: this.convertTimeLimitToMinutes(timeLimit),
is_onboarding_exam: isOnboardingExamChecked
};
// If setting as a proctored exam, set to hide after due by default
if (isProctoredExamChecked && this.startingExamType !== 'proctored_exam') {
metadata.hide_after_due = true;
}
return {
metadata: {
is_practice_exam: isPracticeExamChecked,
is_time_limited: !isNoSpecialExamChecked,
exam_review_rules: examReviewRules,
// We have to use the legacy field name
// as the Ajax handler directly populates
// the xBlocks fields. We will have to
// update this call site when we migrate
// seq_module.py to use 'is_proctored_exam'
is_proctored_enabled: isProctoredExamChecked || isPracticeExamChecked || isOnboardingExamChecked,
default_time_limit_minutes: this.convertTimeLimitToMinutes(timeLimit),
is_onboarding_exam: isOnboardingExamChecked
}
metadata: metadata
};
}
});

2
cms/templates/js/timed-examination-preference-editor.underscore
View File

@@ -38,7 +38,7 @@
/>
<%- gettext('Proctored') %>
</label>
<p class='field-message' id='proctored-exam-description'> <%- gettext('Proctored exams are timed and they record video of each learner taking the exam. The videos are then reviewed to ensure that learners follow all examination rules.') %> </p>
<p class='field-message' id='proctored-exam-description'> <%- gettext('Proctored exams are timed and they record video of each learner taking the exam. The videos are then reviewed to ensure that learners follow all examination rules. Please note that setting this exam as proctored will change the visibility settings to "Hide content after due date."') %> </p>
<% var supports_onboarding = xblockInfo.get('supports_onboarding'); %>
<% if (supports_onboarding) { %>