Commit d8424eb0c14a1fffe6ae3950bc091e2b496b0b09
1 parent
7eea5ec2fa
Exists in
master
Error en el boton cancelar del modal detalles
Showing
1 changed file
with
25 additions
and
15 deletions
Show diff stats
src/js/controllerDetalles.js
| ... | ... | @@ -88,22 +88,12 @@ angular.module('focaCrearHojaRuta') |
| 88 | 88 | |
| 89 | 89 | $scope.guardar = function () { |
| 90 | 90 | |
| 91 | - var cisternasFilter = $scope.hojasRutas.vehiculo.cisternas.filter(function (cisterna) { | |
| 92 | - return parseInt(cisterna.disponible) > 0 || cisterna.articuloSeleccionado; | |
| 93 | - }); | |
| 94 | - | |
| 95 | - | |
| 96 | - var cisternasIncompletas = cisternasFilter.filter(function(cisterna) { | |
| 97 | - return (cisterna.articuloSeleccionado && !cisterna.disponible) || | |
| 98 | - (!cisterna.articuloSeleccionado && cisterna.disponible); | |
| 99 | - }); | |
| 100 | - | |
| 101 | - if (cisternasIncompletas.length || !cisternasFilter.length) { | |
| 102 | - return focaModalService.alert('Ingrese todos los campos para completar el remito'); | |
| 103 | - } | |
| 91 | + var cisternasFilter = validarArticulos(); | |
| 104 | 92 | |
| 105 | 93 | var articulos = []; |
| 106 | - | |
| 94 | + if (cisternasFilter === undefined) { | |
| 95 | + return; | |
| 96 | + } | |
| 107 | 97 | cisternasFilter.forEach(function (cisterna) { |
| 108 | 98 | |
| 109 | 99 | var fechaReparto = $scope.hojasRutas.fechaReparto; |
| ... | ... | @@ -154,7 +144,27 @@ angular.module('focaCrearHojaRuta') |
| 154 | 144 | }; |
| 155 | 145 | |
| 156 | 146 | $scope.cancel = function () { |
| 157 | - $uibModalInstance.dismiss(null); | |
| 147 | + var validarCisternas = validarArticulos(); | |
| 148 | + if (validarCisternas !== undefined) { | |
| 149 | + $uibModalInstance.dismiss(null); | |
| 150 | + } | |
| 158 | 151 | }; |
| 152 | + | |
| 153 | + function validarArticulos () { | |
| 154 | + var cisternasFilter = $scope.hojasRutas.vehiculo.cisternas.filter(function (cisterna) { | |
| 155 | + return parseInt(cisterna.disponible) > 0 || cisterna.articuloSeleccionado; | |
| 156 | + }); | |
| 157 | + | |
| 158 | + var cisternasIncompletas = cisternasFilter.filter(function(cisterna) { | |
| 159 | + return (cisterna.articuloSeleccionado && !cisterna.disponible) || | |
| 160 | + (!cisterna.articuloSeleccionado && cisterna.disponible); | |
| 161 | + }); | |
| 162 | + | |
| 163 | + if (cisternasIncompletas.length || !cisternasFilter.length) { | |
| 164 | + focaModalService.alert('Ingrese todos los campos para completar el remito'); | |
| 165 | + cisternasFilter = undefined; | |
| 166 | + } | |
| 167 | + return cisternasFilter; | |
| 168 | + } | |
| 159 | 169 | } |
| 160 | 170 | ]); |