Lo que sucede es que las dos llamadas Ajax
se ejecutan a la vez, por lo tanto no importa lo que devuelva la primera llamada, la segunda siempre se ejecutará. Para que la segunda se ejecute solo cuando ocurra una de las condiciones deberías ejecutar la segunda dentro de dicha condición.
Voy a reescribir tu código siguiendo algunas buenas prácticas que te ayudarán a que el mismo esté más optimizado y más ordenado:
function obtenerSancionado () {
var cedulaInput = $("#ccedula");
var cedula = { rcedula: cedulaInput.val() };
var limpiarCedula = function () {
cedulaInput.val("").focus();
};
var metodos = {
"metodo0" : function () {
alert("Este funcionario se encuentra disponible para la sanción");
metodos.obtener();
},
"metodo1" : function () {
alert("Este funcionario NO se encuentra disponible por estar sancionado");
limpiarCedula();
},
"metodo2" : function () {
alert("Este n\u00famero de C\u00e9dula no está registrado en la base de datos");
limpiarCedula();
},
"metodo3" : function () {
alert("Este n\u00famero de C\u00e9dula no está registrado en la base de datos");
limpiarCedula();
},
"metodo4" : function () {
// ejemplo modificado con el json devuelto
$("#papellido").val(respuesta.papellidox);
$("#sapellido").val(respuesta.sapellidox);
$("#pnombre").val(respuesta.pnombrex);
$("#snombre").val(respuesta.snombrex);
$("#grados").val(respuesta.gradof);
$("#valor_grados").val(respuesta.gradoc);
$("#fascenso").val(respuesta.fult_ascenso);
$("#cargo").val(respuesta.cargo);
$("#ccedula").prop("disabled", true);
$("#restablecer1").prop("hidden", false);
},
"obtener" : function () {
$.post("./funciones_php_obtenerDatosBasicos.php", cedula , function (respuesta) {
if (respuesta.pnombrex == 99999) {
metodos["metodo3"]();
} else {
metodos["metodo4"]();
}
}, "json" );
}
};
//---Primera llamada Ajax
$.post("./funciones_php_verificarSancion.php", cedula , function (respuesta) {
metodos["metodo" + respuesta.sancionado]();
}, "json" );
}
Te explico los cambios en tu código:
- Si vas a utilizar el elemento
$("#ccedula")
muchas veces, lo mejor es cachearlo en una variable. - Si vas a repetir las mismas líneas varias veces en tu código, lo mejor es situar estas líneas en una función y llamar a esta cada vez que lo necesites (esto lo he aplicado con la función
limpiarCedula
que lo que hace es limpiar el valor del input y darle foco al mismo). - En vez de crear varios
if
yelse if
, sitúa todos tus métodos dentro de un objeto y llama a estos métodos cuando sea necesario. - Uno de los métodos es la segunda llamada
Ajax
(le he situado como nombre al método,obtener
). Cuando se llame al métodometodo0
se llama al métodoobtener
, si no se llama almetodo0
la segunda llamadaAjax
nunca se ejecutará.