diff --git a/src/js/controller.js b/src/js/controller.js index c9c6ea3..15cc3c3 100644 --- a/src/js/controller.js +++ b/src/js/controller.js @@ -60,29 +60,14 @@ angular.module('focaAbmChofer') }; $scope.agregarChofer = function (chofer) { - if (!chofer) { + if (!chofer.nombre) { focaModalService.alert('Ingrese nombre'); return; } else if (!chofer.idTipoDocumento) { focaModalService.alert('Ingrese tipo documento'); return; - } else if (!chofer.dni) { - focaModalService.alert('Ingrese DNI'); - return; - } else if (!chofer.telefono) { - focaModalService.alert('Ingrese teléfono'); - return; - } - - validaDni(chofer).then(function() { - chofer.idTransportista = focaAbmChoferService.transportistaSeleccionado.COD; - delete chofer.transportista; - }, function() { - focaModalService.alert('Dni existente'); - $scope.editando = true; - }); - $scope.crear = true; - chofer.editando = false; + } + validaDni(chofer); }; $scope.tipoDocumento = function (idTipoDocumento) { @@ -189,6 +174,14 @@ angular.module('focaAbmChofer') } function validaDni(chofer) { + if (!chofer.dni) { + focaModalService.alert('Ingrese DNI'); + return; + } else if (!chofer.telefono) { + focaModalService.alert('Ingrese teléfono'); + return; + } + return new Promise(function(resolve, reject) { focaAbmChoferService .getChoferPorDni(chofer.dni) @@ -199,7 +192,16 @@ angular.module('focaAbmChofer') } else { resolve(); } + }) + .then(function() { + chofer.idTransportista = focaAbmChoferService.transportistaSeleccionado.COD; + delete chofer.transportista; + }, function() { + focaModalService.alert('Dni existente'); + $scope.editando = true; }); + $scope.crear = true; + chofer.editando = false; }); }