Commit b1067ac76ba10382c9cc33553fb1d06c637a3b15
Exists in
master
Merge branch 'master' into 'develop'
Master See merge request !54
Showing
1 changed file
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 | ]); |