Commit 60d06dd1f5c4f9dc8378bf089cd1ca10301c8968
1 parent
fc869a4032
Exists in
master
and in
2 other branches
pre commit
Showing
2 changed files
with
11 additions
and
8 deletions
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 |
src/views/foca-abm-vendedor-cobrador-item.html
| ... | ... | @@ -181,7 +181,6 @@ |
| 181 | 181 | ng-keypress="next($event.keyCode)" |
| 182 | 182 | autocomplete="off" |
| 183 | 183 | string-to-number |
| 184 | - solo-positivo | |
| 185 | 184 | /> |
| 186 | 185 | </div> |
| 187 | 186 | </div> |
| ... | ... | @@ -200,7 +199,6 @@ |
| 200 | 199 | ng-keypress="next($event.keyCode)" |
| 201 | 200 | autocomplete="off" |
| 202 | 201 | string-to-number |
| 203 | - solo-positivo | |
| 204 | 202 | /> |
| 205 | 203 | </div> |
| 206 | 204 | </div> |