Commit 472f33ef41ebeffefedb80a84c0616fa3a9eecfa
Exists in
master
and in
2 other branches
Merge branch 'master' into 'develop'
Master(efernandez) See merge request !29
Showing
1 changed file
Show diff stats
src/js/controllerItem.js
| ... | ... | @@ -86,9 +86,13 @@ angular.module('focaAbmVendedorCobrador') |
| 86 | 86 | } else if (!$scope.vendedorCobrador.localidad.NOMBRE) { |
| 87 | 87 | focaModalService.alert('Seleccione una localidad'); |
| 88 | 88 | return; |
| 89 | - } else if (!isNaN($scope.vendedorCobrador.comisionCobrador) || | |
| 90 | - !isNaN($scope.vendedorCobrador.comisionVendedor)) { | |
| 91 | - focaModalService.alert('Ingrese una comision válida'); | |
| 89 | + } else if (!parseFloat($scope.vendedorCobrador.comisionCobrador) && | |
| 90 | + ($scope.vendedorCobrador.rol === 2 || $scope.vendedorCobrador.rol === 3)) { | |
| 91 | + focaModalService.alert('Ingrese una comision cobrador válida'); | |
| 92 | + return; | |
| 93 | + } else if (!parseFloat($scope.vendedorCobrador.comisionVendedor) && | |
| 94 | + ($scope.vendedorCobrador.rol === 1 || $scope.vendedorCobrador.rol === 3)) { | |
| 95 | + focaModalService.alert('Ingrese una comision vendedor válida'); | |
| 92 | 96 | return; |
| 93 | 97 | } |
| 94 | 98 | |
| ... | ... | @@ -103,9 +107,9 @@ angular.module('focaAbmVendedorCobrador') |
| 103 | 107 | |
| 104 | 108 | delete vendedorCobrador.provincia; |
| 105 | 109 | delete vendedorCobrador.localidad; |
| 106 | - if (vendedorCobrador.rol == 1) { | |
| 110 | + if (vendedorCobrador.rol === 1) { | |
| 107 | 111 | delete vendedorCobrador.comisionCobrador; |
| 108 | - } else if (vendedorCobrador.rol == 2) { | |
| 112 | + } else if (vendedorCobrador.rol === 2) { | |
| 109 | 113 | delete vendedorCobrador.comisionVendedor; |
| 110 | 114 | } |
| 111 | 115 | vendedorCobrador.metodo = ($routeParams.id === '0') ? 'crear' : 'actualizar'; |
| ... | ... | @@ -218,7 +222,8 @@ angular.module('focaAbmVendedorCobrador') |
| 218 | 222 | $scope.vendedorCobrador = vendedorCobrador; |
| 219 | 223 | $scope.$broadcast('addCabecera', { |
| 220 | 224 | label: 'Vendedor / Cobrador:', |
| 221 | - valor: $filter('rellenarDigitos')(vendedorCobrador.NUM, 2) + ' - ' + vendedorCobrador.NOM | |
| 225 | + valor: $filter('rellenarDigitos')(vendedorCobrador.NUM, 2) + | |
| 226 | + ' - ' + vendedorCobrador.NOM | |
| 222 | 227 | }); |
| 223 | 228 | } |
| 224 | 229 |