code review

Eric Fernandez
1 parent 7530f73686
Showing 1 changed file with 8 additions and 11 deletions Show diff stats
src/js/controllerHojaRuta.js
@@ -70,20 +70,17 @@ angular.module(&#39;focaLogisticaPedidoRuta&#39;)
                     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;
+                });
+        }
     }]);
...	...	@@ -70,20 +70,17 @@ angular.module('focaLogisticaPedidoRuta')
70	70	eliminarRemitos(remitosDel);
71	71	}
72	72	);
73		- function eliminarRemitos(remitosDel) {
74		- var nuevosRemitos = $filter('filter')($scope.remitos, {checked: !true});
75		- focaLogisticaPedidoRutaService.desasociarRemitos(remitosDel, $scope.vehiculo.id,
76		- nuevosRemitos ? true : false).then(function() {
77		- alertaEliminados();
78		- $scope.remitos = nuevosRemitos;
79		- });
80		- }
81		- function alertaEliminados() {
82		- focaModalService.alert('Remitos desasociados con éxito');
83		- }
84	73	};
85	74	$scope.minimoUnoChecked = function() {
86	75	var remitosChequeados = $filter('filter')($scope.remitos, {checked: true});
87	76	return !remitosChequeados.length;
88	77	};
	78	+ function eliminarRemitos(remitosDel) {
	79	+ var nuevosRemitos = $filter('filter')($scope.remitos, {checked: !true});
	80	+ focaLogisticaPedidoRutaService.desasociarRemitos(remitosDel, $scope.vehiculo.id,
	81	+ nuevosRemitos ? true : false).then(function() {
	82	+ focaModalService.alert('Remitos desasociados con éxito');
	83	+ $scope.remitos = nuevosRemitos;
	84	+ });
	85	+ }
89	86	}]);