From 8198ae90c88b5963d1e8fc265a602d34bffb5f25 Mon Sep 17 00:00:00 2001 From: efernandez Date: Thu, 3 Jan 2019 13:01:00 -0300 Subject: [PATCH] code review --- src/js/controllerHojaRuta.js | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/src/js/controllerHojaRuta.js b/src/js/controllerHojaRuta.js index 2ddeb62..e88f993 100644 --- a/src/js/controllerHojaRuta.js +++ b/src/js/controllerHojaRuta.js @@ -70,20 +70,17 @@ angular.module('focaLogisticaPedidoRuta') eliminarRemitos(remitosDel); } ); - function eliminarRemitos(remitosDel) { - var nuevosRemitos = $filter('filter')($scope.remitos, {checked: !true}); - focaLogisticaPedidoRutaService.desasociarRemitos(remitosDel, $scope.vehiculo.id, - nuevosRemitos ? true : false).then(function() { - alertaEliminados(); - $scope.remitos = nuevosRemitos; - }); - } - function alertaEliminados() { - focaModalService.alert('Remitos desasociados con éxito'); - } }; $scope.minimoUnoChecked = function() { var remitosChequeados = $filter('filter')($scope.remitos, {checked: true}); return !remitosChequeados.length; }; + function eliminarRemitos(remitosDel) { + var nuevosRemitos = $filter('filter')($scope.remitos, {checked: !true}); + focaLogisticaPedidoRutaService.desasociarRemitos(remitosDel, $scope.vehiculo.id, + nuevosRemitos ? true : false).then(function() { + focaModalService.alert('Remitos desasociados con éxito'); + $scope.remitos = nuevosRemitos; + }); + } }]); -- 1.9.1