Commit 6a06288d8c841d9ca32d2b3fb88647c8658c0534

Authored by Eric Fernandez
1 parent 9ba5c967b7
Exists in master

code review

... ... @@ -19,7 +19,7 @@ function drop(ev) {
19 19 }
20 20 function dropEnd() {
21 21 console.log('drop');
22   - var elementoDom = document.getElementById('test');
  22 + var elementoDom = document.getElementById('test');
23 23 var scope = angular.element(elementoDom).scope();
24 24 scope.noArrastra();
25   -}
26 25 \ No newline at end of file
  26 +}
src/js/controller.js
... ... @@ -119,7 +119,7 @@ angular.module('focaLogisticaPedidoRuta') .controller('focaLogisticaPedidoRutaCo
119 119 modalInstance.result.then(
120 120 function(vehiculo) {
121 121 var existe = $filter('filter')($scope.vehiculos, {id: vehiculo.id});
122   - if(existe.length > 0){
  122 + if(existe.length) {
123 123 focaModalService.alert('El vehiculo que intenta cargar ya ha sido cargado');
124 124 return;
125 125 }
src/js/controllerDetalleVehiculo.js
... ... @@ -34,7 +34,7 @@ angular.module('focaLogisticaPedidoRuta')
34 34 };
35 35  
36 36 $scope.cargarACisternas = function(vehiculo) {
37   - for (var i = 0; i < vehiculo.cisternas.length; i++) {
  37 + for(var i = 0; i < vehiculo.cisternas.length; i++) {
38 38 var cisterna = vehiculo.cisternas[i];
39 39 var aCargar = parseFloat($scope.aCargar[i]);
40 40 if(!aCargar) {
... ... @@ -66,7 +66,7 @@ angular.module(&#39;focaLogisticaPedidoRuta&#39;)
66 66 return filter.cargado !== true;
67 67 }
68 68 );
69   - if(articuloSiguiente.length > 0){
  69 + if(articuloSiguiente.length > 0) {
70 70 $scope.cambioArticulo(articuloSiguiente[0]);
71 71 }
72 72 $scope.aCargar = [];