diff --git a/src/js/controllerDetalles.js b/src/js/controllerDetalles.js index 71163b9..1d77b28 100644 --- a/src/js/controllerDetalles.js +++ b/src/js/controllerDetalles.js @@ -88,22 +88,12 @@ angular.module('focaCrearHojaRuta') $scope.guardar = function () { - var cisternasFilter = $scope.hojasRutas.vehiculo.cisternas.filter(function (cisterna) { - return parseInt(cisterna.disponible) > 0 || cisterna.articuloSeleccionado; - }); - - - var cisternasIncompletas = cisternasFilter.filter(function(cisterna) { - return (cisterna.articuloSeleccionado && !cisterna.disponible) || - (!cisterna.articuloSeleccionado && cisterna.disponible); - }); - - if (cisternasIncompletas.length || !cisternasFilter.length) { - return focaModalService.alert('Ingrese todos los campos para completar el remito'); - } + var cisternasFilter = validarArticulos(); var articulos = []; - + if (cisternasFilter === undefined) { + return; + } cisternasFilter.forEach(function (cisterna) { var fechaReparto = $scope.hojasRutas.fechaReparto; @@ -154,7 +144,27 @@ angular.module('focaCrearHojaRuta') }; $scope.cancel = function () { - $uibModalInstance.dismiss(null); + var validarCisternas = validarArticulos(); + if (validarCisternas !== undefined) { + $uibModalInstance.dismiss(null); + } }; + + function validarArticulos () { + var cisternasFilter = $scope.hojasRutas.vehiculo.cisternas.filter(function (cisterna) { + return parseInt(cisterna.disponible) > 0 || cisterna.articuloSeleccionado; + }); + + var cisternasIncompletas = cisternasFilter.filter(function(cisterna) { + return (cisterna.articuloSeleccionado && !cisterna.disponible) || + (!cisterna.articuloSeleccionado && cisterna.disponible); + }); + + if (cisternasIncompletas.length || !cisternasFilter.length) { + focaModalService.alert('Ingrese todos los campos para completar el remito'); + cisternasFilter = undefined; + } + return cisternasFilter; + } } ]);