From 60d06dd1f5c4f9dc8378bf089cd1ca10301c8968 Mon Sep 17 00:00:00 2001
From: Eric <fernandezeric1997@gmail.com>
Date: Tue, 26 Mar 2019 11:00:32 -0300
Subject: [PATCH] pre commit

---
 src/js/controllerItem.js                       | 17 +++++++++++------
 src/views/foca-abm-vendedor-cobrador-item.html |  2 --
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/js/controllerItem.js b/src/js/controllerItem.js
index ffa5802..832f316 100644
--- a/src/js/controllerItem.js
+++ b/src/js/controllerItem.js
@@ -86,9 +86,13 @@ angular.module('focaAbmVendedorCobrador')
                 } else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
                     focaModalService.alert('Seleccione una localidad');
                     return;
-                } else if (!isNaN($scope.vendedorCobrador.comisionCobrador) ||  
-                    !isNaN($scope.vendedorCobrador.comisionVendedor)) {
-                    focaModalService.alert('Ingrese una comision válida');
+                } else if (!parseFloat($scope.vendedorCobrador.comisionCobrador) &&  
+                    ($scope.vendedorCobrador.rol === 2 || $scope.vendedorCobrador.rol === 3)) {
+                    focaModalService.alert('Ingrese una comision cobrador válida');
+                    return;
+                } else if (!parseFloat($scope.vendedorCobrador.comisionVendedor) &&
+                    ($scope.vendedorCobrador.rol === 1 || $scope.vendedorCobrador.rol === 3)) {
+                    focaModalService.alert('Ingrese una comision vendedor válida');
                     return;
                 }
 
@@ -103,9 +107,9 @@ angular.module('focaAbmVendedorCobrador')
 
                     delete vendedorCobrador.provincia;
                     delete vendedorCobrador.localidad;
-                    if (vendedorCobrador.rol == 1) {
+                    if (vendedorCobrador.rol === 1) {
                         delete vendedorCobrador.comisionCobrador;
-                    } else if (vendedorCobrador.rol == 2) {
+                    } else if (vendedorCobrador.rol === 2) {
                         delete vendedorCobrador.comisionVendedor;
                     }
                     vendedorCobrador.metodo = ($routeParams.id === '0') ? 'crear' : 'actualizar';
@@ -218,7 +222,8 @@ angular.module('focaAbmVendedorCobrador')
                 $scope.vendedorCobrador = vendedorCobrador;
                 $scope.$broadcast('addCabecera', {
                     label: 'Vendedor / Cobrador:',
-                    valor: $filter('rellenarDigitos')(vendedorCobrador.NUM, 2) + ' - ' + vendedorCobrador.NOM
+                    valor: $filter('rellenarDigitos')(vendedorCobrador.NUM, 2) +
+                        ' - ' + vendedorCobrador.NOM
                 });
             }
             
diff --git a/src/views/foca-abm-vendedor-cobrador-item.html b/src/views/foca-abm-vendedor-cobrador-item.html
index dc486ae..552174b 100644
--- a/src/views/foca-abm-vendedor-cobrador-item.html
+++ b/src/views/foca-abm-vendedor-cobrador-item.html
@@ -181,7 +181,6 @@
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
-                            solo-positivo
                         />
                     </div>
                 </div>
@@ -200,7 +199,6 @@
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
-                            solo-positivo
                         />
                     </div>
                 </div>
-- 
1.9.1