diff --git a/cms/djangoapps/models/settings/course_metadata.py b/cms/djangoapps/models/settings/course_metadata.py index 0dbf338218..988feed9f6 100644 --- a/cms/djangoapps/models/settings/course_metadata.py +++ b/cms/djangoapps/models/settings/course_metadata.py @@ -41,6 +41,7 @@ class CourseMetadata: 'enrollment_start', 'enrollment_end', 'certificate_available_date', + 'certificates_display_behavior', 'tabs', 'graceperiod', 'show_timezone', diff --git a/cms/static/js/factories/settings.js b/cms/static/js/factories/settings.js index 6ed11c4916..4e399215ca 100644 --- a/cms/static/js/factories/settings.js +++ b/cms/static/js/factories/settings.js @@ -13,6 +13,12 @@ define([ $('label').removeClass('is-focused'); }); + // Toggle collapsibles when trigger is clicked + $(".collapsible .collapsible-trigger").click(function() { + const contentId = this.id.replace("-trigger", "-content") + $(`#${contentId}`).toggleClass("collapsed") + }) + model = new CourseDetailsModel(); model.urlRoot = detailsUrl; model.showCertificateAvailableDate = showCertificateAvailableDate; diff --git a/cms/static/js/models/settings/course_details.js b/cms/static/js/models/settings/course_details.js index c0d8c506f4..22c3f462a3 100644 --- a/cms/static/js/models/settings/course_details.js +++ b/cms/static/js/models/settings/course_details.js @@ -1,7 +1,7 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js/utils/date_utils', 'edx-ui-toolkit/js/utils/string-utils' ], - function(Backbone, _, gettext, ValidationHelpers, DateUtils, StringUtils) { + function (Backbone, _, gettext, ValidationHelpers, DateUtils, StringUtils) { 'use strict'; var CourseDetails = Backbone.Model.extend({ defaults: { @@ -11,6 +11,7 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js language: '', start_date: null, // maps to 'start' end_date: null, // maps to 'end' + certificates_display_behavior: "", certificate_available_date: null, enrollment_start: null, enrollment_end: null, @@ -38,10 +39,16 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js self_paced: null }, - validate: function(newattrs) { - // Returns either nothing (no return call) so that validate works or an object of {field: errorstring} pairs - // A bit funny in that the video key validation is asynchronous; so, it won't stop the validation. + validate: function (newattrs) { + // Returns either nothing (no return call) so that validate works or an object of {field: errorstring} pairs + // A bit funny in that the video key validation is asynchronous; so, it won't stop the validation. var errors = {}; + const CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS = { + END: "end", + END_WITH_DATE: "end_with_date", + EARLY_NO_INFO: "early_no_info" + }; + newattrs = DateUtils.convertDateStringsToObjects( newattrs, ['start_date', 'end_date', 'certificate_available_date', 'enrollment_start', 'enrollment_end'] @@ -55,15 +62,15 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js errors.end_date = gettext('The course end date must be later than the course start date.'); } if (newattrs.start_date && newattrs.enrollment_start && - newattrs.start_date < newattrs.enrollment_start) { + newattrs.start_date < newattrs.enrollment_start) { errors.enrollment_start = gettext( - 'The course start date must be later than the enrollment start date.' + 'The course start date must be later than the enrollment start date.' ); } if (newattrs.enrollment_start && newattrs.enrollment_end && - newattrs.enrollment_start >= newattrs.enrollment_end) { + newattrs.enrollment_start >= newattrs.enrollment_end) { errors.enrollment_end = gettext( - 'The enrollment start date cannot be after the enrollment end date.' + 'The enrollment start date cannot be after the enrollment end date.' ); } if (newattrs.end_date && newattrs.enrollment_end && newattrs.end_date < newattrs.enrollment_end) { @@ -75,11 +82,43 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js 'The certificate available date must be later than the course end date.' ); } + + + if ( + newattrs.certificates_display_behavior + && !(Object.values(CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS).includes(newattrs.certificates_display_behavior)) + ) { + + errors.certificates_display_behavior = StringUtils.interpolate( + gettext( + "The certificate display behavior must be one of: {behavior_options}" + ), + { + behavior_options: Object.values(CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS).join(', ') + } + ); + } + + // Throw error if there's a value for certificate_available_date + if( + (newattrs.certificate_available_date && newattrs.certificates_display_behavior != CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS.END_WITH_DATE) + || (!newattrs.certificate_available_date && newattrs.certificates_display_behavior == CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS.END_WITH_DATE) + ){ + errors.certificates_display_behavior = StringUtils.interpolate( + gettext( + "The certificates display behavior must be {valid_option} if certificate available date is set." + ), + { + valid_option: CERTIFICATES_DISPLAY_BEHAVIOR_OPTIONS.END_WITH_DATE + } + ); + } + if (newattrs.intro_video && newattrs.intro_video !== this.get('intro_video')) { if (this._videokey_illegal_chars.exec(newattrs.intro_video)) { errors.intro_video = gettext('Key should only contain letters, numbers, _, or -'); } - // TODO check if key points to a real video using google's youtube api + // TODO check if key points to a real video using google's youtube api } if (_.has(newattrs, 'entrance_exam_minimum_score_pct')) { var range = { @@ -88,37 +127,37 @@ define(['backbone', 'underscore', 'gettext', 'js/models/validation_helpers', 'js }; if (!ValidationHelpers.validateIntegerRange(newattrs.entrance_exam_minimum_score_pct, range)) { errors.entrance_exam_minimum_score_pct = StringUtils.interpolate(gettext( - 'Please enter an integer between %(min)s and %(max)s.' + 'Please enter an integer between %(min)s and %(max)s.' ), range, true); } } if (!_.isEmpty(errors)) return errors; - // NOTE don't return empty errors as that will be interpreted as an error state + // NOTE don't return empty errors as that will be interpreted as an error state }, _videokey_illegal_chars: /[^a-zA-Z0-9_-]/g, - set_videosource: function(newsource) { - // newsource either is