alerta jConfirm - plugin de jQuery
-
09-10-2019 - |
Pregunta
Am jConfirm la confirmación del usuario.
Mi primera parada jConfirm duerma por la acción del usuario y simplemente pasa al siguiente.
Mi código:
$(function () {
$("#UpdateJobHandler").click(function () {
var JobHander = getJobHandler();
if (JobHander.MaxInstances == 0) {
jConfirm('Continue?', 'Current Maximum Instances', function (ans) {
if (!ans)
return;
});
}
var json = $.toJSON(JobHander);
$.ajax({
url: '../Metadata/JobHandlerUpdate',
type: 'POST',
dataType: 'json',
data: json,
contentType: 'application/json; charset=utf-8',
success: function (data) {
var message = data.Message;
var alertM = data.MessageType;
if (alertM == 'Error') {
$("#resultMessage").html(message);
}
if (alertM == 'Success') {
$("#resultMessage").empty();
alert(alertM + '-' + message);
action = "JobHandler";
controller = "MetaData";
loc = "../" + controller + "/" + action;
window.location = loc;
}
if (alertM == "Instances") {
jConfirm(message, 'Instances Confirmation?', function (answer) {
if (!answer)
return;
else {
var JobHandlerNew = getJobHandler();
JobHandlerNew.FinalUpdate = "Yes";
var json = $.toJSON(JobHandlerNew);
$.ajax({
url: '../Metadata/JobHandlerUpdate',
type: 'POST',
dataType: 'json',
data: json,
contentType: 'application/json; charset=utf-8',
success: function (data) {
var message = data.Message;
$("#resultMessage").empty();
alert(alertM + '-' + message);
action = "JobHandler";
controller = "MetaData";
loc = "../" + controller + "/" + action;
window.location = loc;
}
});
}
});
}
}
});
});
});
¿Qué me falta?
Solución
No estoy seguro si esto es todo, pero esta parte:
if (JobHander.MaxInstances == 0) {
jConfirm('Continue?', 'Current Maximum Instances', function (ans) {
if (!ans)
return;
});
}
probablemente no hace lo que quiere. Se sale de la función function(ans) { ... }
, mientras que es probable que desee salir todo el manejador, es decir $("#UpdateJobHandler").click(function () { ... }
. Si es así, se tendría que hacer similar a lo que se hace a continuación - es decir, poner todo en function(ans) { ... }
, después de la vuelta. Probablemente la mejor manera de separar en funciones más pequeñas.
EDIT: Algo a lo largo de estas líneas:
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();
}
});
}
Se puede hacer algo similar para todas las funciones success
.
Otro ejemplo, se puede volver a escribir el cheque Instances
como esto:
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();
}
});
}
Importante -. Cambiar el nombre de los métodos (afterContinue
, afterInstances
, ...) que tiene un nombre que significa algo útil para alguien que lee esto en el futuro