Cambio en validación de porcentajes de comision

Luigi
1 parent d01a61a08e
Showing 3 changed files with 9 additions and 6 deletions Show diff stats
src/js/controller.js
src/js/controllerItem.js
src/views/foca-abm-vendedor-cobrador-item.html
@@ -28,7 +28,7 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
                 focaModalService.confirm('¿Está seguro que desea borrar el ' + tipo +
                     ' ' + vendedorCobrador.NOM + '?')
                     .then(function(data) {
-                        if(data) {
+                        if (data) {
                             focaAbmVendedorCobradorService
                             .eliminarVendedorCobrador(vendedorCobrador.id)
                             .then(function() {
@@ -54,7 +54,7 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
             };
  
             $scope.search = function(key) {
-                if(key !== 13) return;
+                if (key !== 13) return;
  
                 focaAbmVendedorCobradorService
                     .getVendedoresCobradores($scope.filters)
@@ -81,13 +81,14 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
                     focaModalService.alert('Seleccione tipo');
                     return;
                 } else if (!$scope.vendedorCobrador.provincia.NOMBRE) {
-                    focaModalService.alert('Seleccione una Provincia');
+                    focaModalService.alert('Seleccione una provincia');
                     return;
                 } else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
-                    focaModalService.alert('Seleccione una Localidad');
+                    focaModalService.alert('Seleccione una localidad');
                     return;
-                } else if ($scope.vendedorCobrador.comisionCobrador < 0 || $scope.vendedorCobrador.comisionVendedor < 0){
-                    focaModalService.alert('Ingrese un numero superior a 0 en porcentaje de comision');
+                } else if (!isNaN($scope.vendedorCobrador.comisionCobrador) ||  
+                    !isNaN($scope.vendedorCobrador.comisionVendedor)) {
+                    focaModalService.alert('Ingrese una comision válida');
                     return;
                 }
  
@@ -181,6 +181,7 @@
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
+                            solo-positivo
                         />
                     </div>
                 </div>
@@ -199,6 +200,7 @@
                             ng-keypress="next($event.keyCode)"
                             autocomplete="off"
                             string-to-number
+                            solo-positivo
                         />
                     </div>
                 </div>
...	...	@@ -28,7 +28,7 @@ angular.module('focaAbmVendedorCobrador')
28	28	focaModalService.confirm('¿Está seguro que desea borrar el ' + tipo +
29	29	' ' + vendedorCobrador.NOM + '?')
30	30	.then(function(data) {
31		- if(data) {
	31	+ if (data) {
32	32	focaAbmVendedorCobradorService
33	33	.eliminarVendedorCobrador(vendedorCobrador.id)
34	34	.then(function() {
...	...	@@ -54,7 +54,7 @@ angular.module('focaAbmVendedorCobrador')
54	54	};
55	55
56	56	$scope.search = function(key) {
57		- if(key !== 13) return;
	57	+ if (key !== 13) return;
58	58
59	59	focaAbmVendedorCobradorService
60	60	.getVendedoresCobradores($scope.filters)
...	...	@@ -81,13 +81,14 @@ angular.module('focaAbmVendedorCobrador')
81	81	focaModalService.alert('Seleccione tipo');
82	82	return;
83	83	} else if (!$scope.vendedorCobrador.provincia.NOMBRE) {
84		- focaModalService.alert('Seleccione una Provincia');
	84	+ focaModalService.alert('Seleccione una provincia');
85	85	return;
86	86	} else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
87		- focaModalService.alert('Seleccione una Localidad');
	87	+ focaModalService.alert('Seleccione una localidad');
88	88	return;
89		- } else if ($scope.vendedorCobrador.comisionCobrador < 0 \|\| $scope.vendedorCobrador.comisionVendedor < 0){
90		- focaModalService.alert('Ingrese un numero superior a 0 en porcentaje de comision');
	89	+ } else if (!isNaN($scope.vendedorCobrador.comisionCobrador) \|\|
	90	+ !isNaN($scope.vendedorCobrador.comisionVendedor)) {
	91	+ focaModalService.alert('Ingrese una comision válida');
91	92	return;
92	93	}
93	94
...	...	@@ -181,6 +181,7 @@
181	181	ng-keypress="next($event.keyCode)"
182	182	autocomplete="off"
183	183	string-to-number
	184	+ solo-positivo
184	185	/>
185	186	</div>
186	187	</div>
...	...	@@ -199,6 +200,7 @@
199	200	ng-keypress="next($event.keyCode)"
200	201	autocomplete="off"
201	202	string-to-number
	203	+ solo-positivo
202	204	/>
203	205	</div>
204	206	</div>