From a71f89fd7d1b1ba5b916724c05eb7e1082caf547 Mon Sep 17 00:00:00 2001
From: Luis Suarez <lsuarezh23@gmail.com>
Date: Wed, 13 May 2020 09:02:55 -0400
Subject: [PATCH] validar crear vendedor/cobrador

---
 src/js/controllerItem.js                       | 28 +++++++++++++-------------
 src/views/foca-abm-vendedor-cobrador-item.html | 14 ++++++++-----
 2 files changed, 23 insertions(+), 19 deletions(-)

diff --git a/src/js/controllerItem.js b/src/js/controllerItem.js
index 037b7a9..857ee0e 100644
--- a/src/js/controllerItem.js
+++ b/src/js/controllerItem.js
@@ -75,12 +75,12 @@ angular.module('focaAbmVendedorCobrador')
                     return;
                 } else if (!$scope.vendedorCobrador.codigoPostal || 
                             $scope.vendedorCobrador.codigoPostal.length != 8) {
-                    focaModalService.alert('Ingrese un código postal válido');
+                    focaModalService.alert('El campo Código Postal debe tener 8 dígitos');
                     return;
                 } else if (!$scope.vendedorCobrador.dni ||
                            $scope.vendedorCobrador.dni.length < 7 ||
-                           $scope.vendedorCobrador.dni.length > 12) {
-                    focaModalService.alert('Ingrese un DNI válido');
+                           $scope.vendedorCobrador.dni.length > 8) {
+                    focaModalService.alert('El campo DNI debe tener de 7 a 8 digitos');
                     return;
                 } else if (!$scope.vendedorCobrador.rol) {
                     focaModalService.alert('Seleccione tipo');
@@ -94,7 +94,7 @@ angular.module('focaAbmVendedorCobrador')
                 } else if (!$scope.vendedorCobrador.telefono ||
                            $scope.vendedorCobrador.telefono.length < 7 ||
                            $scope.vendedorCobrador.telefono.length > 10) {
-                    focaModalService.alert('Ingrese un teléfono válido');
+                    focaModalService.alert('El campo Teléfono debe tener de 7 a 10 digitos');
                     return;
                 } else if (($scope.vendedorCobrador.rol === 2 && isNaN(cobradorComision)) || 
                           (($scope.vendedorCobrador.rol === 3) && isNaN(cobradorComision))) {
@@ -295,17 +295,17 @@ angular.module('focaAbmVendedorCobrador')
                 });
 
             };
-
-            $scope.validarDatos = function() {
-                if (!/^([a-zA-ZÀ-ÿ ])*$/.test($scope.vendedorCobrador.NOM)) {
+            $scope.validarDatoNombre = function (vendedorCobrador) {
+                if (!/^([a-zA-ZÀ-ÿ ])*$/.test(vendedorCobrador.NOM)) {
             		focaModalService.alert('el campo Nombre no debe incluir números o símbolos');
-                    $scope.vendedorCobrador.NOM = '';
-                } else if (vendedorCobrador.comisionCobrador > 100) {
-                    focaModalService.alert('La Comisión debe ser menor al 100%');
-                    vendedorCobrador.comisionCobrador='';
-                } else if (vendedorCobrador.comisionVendedor > 100) {
-                    focaModalService.alert('La Comisión debe ser menor al 100%');
-                    vendedorCobrador.comisionVendedor='';
+                    vendedorCobrador.NOM = '';
+                }
+            }
+            $scope.validarDatoComision = function (vendedorCobrador) {
+                if ((vendedorCobrador.comisionCobrador && vendedorCobrador.comisionCobrador > 100) ||
+                    (vendedorCobrador.comisionVendedor && vendedorCobrador.comisionVendedor > 100)) {
+                        focaModalService.alert('La Comisión debe ser menor al 100%');
+                        vendedorCobrador.comisionCobrador='';
                 }
             }
         }
diff --git a/src/views/foca-abm-vendedor-cobrador-item.html b/src/views/foca-abm-vendedor-cobrador-item.html
index b6b1135..eddb714 100644
--- a/src/views/foca-abm-vendedor-cobrador-item.html
+++ b/src/views/foca-abm-vendedor-cobrador-item.html
@@ -17,7 +17,7 @@
                             ng-model="vendedorCobrador.NUM"
                             solo-positivos
                             ng-disabled="edicion"
-                            disabled/>
+                            disabled="disabled"/>
                     </div>
                 </div>
                 <div class="form-group d-flex mb-2 col-md-6">
@@ -40,12 +40,11 @@
                             type="text"
                             teclado-virtual
                             ng-model="vendedorCobrador.NOM"
-                            ng-keyup="validarDatos()"
+                            ng-keyup="validarDatoNombre(vendedorCobrador)"
                             foca-focus="focused == 1"
                             ng-focus="focused = 1"
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
-                            disabled
                             uppercase-only
                             
                         />
@@ -78,6 +77,7 @@
                             solo-positivos
                             ng-model="vendedorCobrador.codigoPostal"
                             ng-required="true"
+                            limite-numeros-max="8"
                             maxlength="8"
                             foca-focus="focused == 3"
                             ng-focus="focused = 3"
@@ -171,8 +171,8 @@
                             foca-tipo-input
                             teclado-virtual
                             solo-positivos
-                            limite-numeros-max="12"
-                            maxlength="12"
+                            limite-numeros-max="8"
+                            maxlength="8"
                             minlength="7"
                             ng-model="vendedorCobrador.dni"
                             ng-required="true"
@@ -195,8 +195,10 @@
                             ng-required="true"
                             max="100"
                             maxlength="3"
+                            limite-numeros-max="3"
                             foca-focus="focused == 10"
                             ng-focus="focused = 10"
+                            ng-keyup="validarDatoComision(vendedorCobrador)"
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
@@ -215,9 +217,11 @@
                             ng-model="vendedorCobrador.comisionCobrador"
                             max="100"
                             maxlength="3"
+                            limite-numeros-max="3"
                             ng-required="true"
                             foca-focus="focused == 10"
                             ng-focus="focused = 10"
+                            ng-keyup="validarDatoComision(vendedorCobrador)"
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
-- 
1.9.1