Commit 8d474c82e0b5023b132f23e6260366ad1df3ecdd
Exists in
master
and in
2 other branches
Merge branch 'develop' into 'master'
Develop See merge request !30
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 |