Не уверен, что это все, но эта часть:
if (JobHander.MaxInstances == 0) {
jConfirm('Continue?', 'Current Maximum Instances', function (ans) {
if (!ans)
return;
});
}
, вероятно, не делает то, что вы хотите. Он выходит из функции function(ans) { ... }
, в то время как вы, вероятно, захотите выйти из всего обработчика, т.е. $("#UpdateJobHandler").click(function () { ... }
. Если это так, вам нужно будет сделать то же самое, что вы делаете ниже - т.е. поместить все это в function(ans) { ... }
после возврата. Вероятно, лучше всего разделить на более мелкие функции.
РЕДАКТИРОВАТЬ: Что-то вроде этого:
function afterContinue() {
var json = $.toJSON(JobHander);
$.ajax({
// ... all other lines here ...
});
}
if (JobHander.MaxInstances == 0) {
jConfirm('Continue?', 'Current Maximum Instances', function (ans) {
if (ans) {
afterContinue();
}
});
}
Вы можете сделать то же самое для всех функций success
.
Другой пример, вы можете переписать проверку Instances
следующим образом:
function afterInstances() {
var JobHandlerNew = getJobHandler();
JobHandlerNew.FinalUpdate = "Yes";
// ... and everything under else branch ...
}
if (alertM == "Instances") {
jConfirm(message, 'Instances Confirmation?', function (answer) {
if (answer) {
afterInstances();
}
});
}
Важно - переименуйте методы (afterContinue
, afterInstances
, ...), чтобы они получили какое-то имя, которое будет полезно кому-то, читающему это в будущем.