put new unique functions inside the primary function
This commit is contained in:
committed by
Brian Talbot
parent
6399eda118
commit
279896e484
@@ -610,31 +610,6 @@ function addNewCourse(e) {
|
||||
}, checkForCancel);
|
||||
}
|
||||
|
||||
function setNewCourseFieldInErr(el, msg) {
|
||||
el.children('.tip-error').remove();
|
||||
if (msg !== null && msg !== '') {
|
||||
el.addClass('error');
|
||||
el.append('<span class="tip tip-error">' + msg + '</span>');
|
||||
} else {
|
||||
el.removeClass('error');
|
||||
}
|
||||
}
|
||||
|
||||
function setNewCourseErrMsgs(header_err_msg, display_name_errMsg, org_errMsg, number_errMsg, run_errMsg) {
|
||||
if (header_err_msg) {
|
||||
$('.wrap-error').addClass('is-shown');
|
||||
$('#course_creation_error').html('<p>' + header_err_msg + '</p>');
|
||||
} else {
|
||||
$('.wrap-error').removeClass('is-shown');
|
||||
$('#course_creation_error').html('');
|
||||
}
|
||||
|
||||
setNewCourseFieldInErr($('#field-course-name'), display_name_errMsg);
|
||||
setNewCourseFieldInErr($('#field-organization'), org_errMsg);
|
||||
setNewCourseFieldInErr($('#field-course-number'), number_errMsg);
|
||||
setNewCourseFieldInErr($('#field-course-run'), run_errMsg);
|
||||
}
|
||||
|
||||
function saveNewCourse(e) {
|
||||
e.preventDefault();
|
||||
|
||||
@@ -669,6 +644,31 @@ function saveNewCourse(e) {
|
||||
|
||||
var header_err_msg = (bInErr) ? gettext('Please correct the fields below.') : null;
|
||||
|
||||
var setNewCourseErrMsgs = function(header_err_msg, display_name_errMsg, org_errMsg, number_errMsg, run_errMsg) {
|
||||
if (header_err_msg) {
|
||||
$('.wrap-error').addClass('is-shown');
|
||||
$('#course_creation_error').html('<p>' + header_err_msg + '</p>');
|
||||
} else {
|
||||
$('.wrap-error').removeClass('is-shown');
|
||||
$('#course_creation_error').html('');
|
||||
}
|
||||
|
||||
var setNewCourseFieldInErr = function(el, msg) {
|
||||
el.children('.tip-error').remove();
|
||||
if (msg !== null && msg !== '') {
|
||||
el.addClass('error');
|
||||
el.append('<span class="tip tip-error">' + msg + '</span>');
|
||||
} else {
|
||||
el.removeClass('error');
|
||||
}
|
||||
};
|
||||
|
||||
setNewCourseFieldInErr($('#field-course-name'), display_name_errMsg);
|
||||
setNewCourseFieldInErr($('#field-organization'), org_errMsg);
|
||||
setNewCourseFieldInErr($('#field-course-number'), number_errMsg);
|
||||
setNewCourseFieldInErr($('#field-course-run'), run_errMsg);
|
||||
};
|
||||
|
||||
setNewCourseErrMsgs(header_err_msg, display_name_errMsg, org_errMsg, number_errMsg, run_errMsg);
|
||||
|
||||
if (bInErr)
|
||||
|
||||
Reference in New Issue
Block a user