Luis Suarez / foca-abm-vendedor-cobrador

Download as
Browse Code ยป

Commit 5ee870b97c41b527de9a54e2aa2c1a676c91e5d5

Authored by Eric Fernandez
2 parents 05907757a2 fc869a4032
Exists in master and in 2 other branches develop, lab

Merge branch 'develop' into 'master' 

Develop

See merge request !27
Showing 4 changed files   Show diff stats
spec/controllerSpec.js
Diff comments   View file @ 5ee870b
... ... @@ -23,6 +23,7 @@ describe('Controladores abm vendedor cobrador', function() {
23 23 $scope: {},
24 24 focaAbmVendedorCobradorService: {},
25 25 $location: {},
  26 + $localStorage: {},
26 27 $filter: {},
27 28 focaBotoneraLateralService: focaBotoneraLateralService,
28 29 focaModalService: {}
... ... @@ -41,6 +42,7 @@ describe('Controladores abm vendedor cobrador', function() {
41 42 $scope: scope,
42 43 focaAbmVendedorCobradorService: {},
43 44 $location: $location,
  45 + $localStorage: {},
44 46 $filter: {},
45 47 focaBotoneraLateralService: focaBotoneraLateralService,
46 48 focaModalService: {}
... ... @@ -66,6 +68,7 @@ describe('Controladores abm vendedor cobrador', function() {
66 68 $scope: scope,
67 69 focaAbmVendedorCobradorService: {},
68 70 $location: $location,
  71 + $localStorage: {},
69 72 $filter: {},
70 73 focaBotoneraLateralService: focaBotoneraLateralService,
71 74 focaModalService: focaModalService
... ... @@ -96,6 +99,7 @@ describe('Controladores abm vendedor cobrador', function() {
96 99 $scope: scope,
97 100 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
98 101 $location: $location,
  102 + $localStorage: {},
99 103 $filter: {},
100 104 focaBotoneraLateralService: focaBotoneraLateralService,
101 105 focaModalService: focaModalService
... ... @@ -126,6 +130,7 @@ describe('Controladores abm vendedor cobrador', function() {
126 130 $scope: scope,
127 131 focaAbmVendedorCobradorService: {},
128 132 $location: {},
  133 + $localStorage: {},
129 134 $filter: {},
130 135 focaBotoneraLateralService: focaBotoneraLateralService,
131 136 focaModalService: {}
... ... @@ -150,6 +155,7 @@ describe('Controladores abm vendedor cobrador', function() {
150 155 $scope: scope,
151 156 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
152 157 $location: {},
  158 + $localStorage: {},
153 159 $filter: {},
154 160 focaBotoneraLateralService: focaBotoneraLateralService,
155 161 focaModalService: {}
... ... @@ -175,6 +181,7 @@ describe('Controladores abm vendedor cobrador', function() {
175 181 $scope: scope,
176 182 focaAbmVendedorCobradorService: focaAbmVendedorCobradorService,
177 183 $location: {},
  184 + $localStorage: {},
178 185 $filter: {},
179 186 focaBotoneraLateralService: focaBotoneraLateralService,
180 187 focaModalService: {}
... ... @@ -210,7 +217,9 @@ describe('Controladores abm vendedor cobrador', function() {
210 217  
211 218 //act
212 219 var controlador = $controller('focaAbmVendedorCobradorController', {
213   - $scope: {},
  220 + $scope: {
  221 + $watch: function() {}
  222 + },
214 223 focaAbmVendedorCobradorService: {
215 224 getCodigoSiguiente: function() {
216 225 return {
... ... @@ -228,7 +237,9 @@ describe('Controladores abm vendedor cobrador', function() {
228 237 $timeout: timeout,
229 238 $uibModal: {},
230 239 $location: {},
231   - focaModalService: {}
  240 + $localStorage: {},
  241 + focaModalService: {
  242 + }
232 243 });
233 244  
234 245 //assert
... ... @@ -237,7 +248,9 @@ describe('Controladores abm vendedor cobrador', function() {
237 248  
238 249 it('la funciรณn $scope.guardar llama a guardaVendedorCobrador del servicio', function() {
239 250 //arrange
240   - var scope= {};
  251 + var scope = {
  252 + $watch: function() {},
  253 + };
241 254 var focaAbmVendedorCobradorService = {
242 255 getCodigoSiguiente: function() {
243 256 return {
... ... @@ -259,7 +272,11 @@ describe('Controladores abm vendedor cobrador', function() {
259 272 $timeout: timeout,
260 273 $uibModal: {},
261 274 $location: {},
262   - focaModalService: {}
  275 + $localStorage: {},
  276 + guardarVendedorCobrador: {},
  277 + focaModalService: {
  278 + alert: function() {}
  279 + }
263 280 });
264 281 var responseFake = {
265 282 then: function() {
... ... @@ -268,6 +285,19 @@ describe('Controladores abm vendedor cobrador', function() {
268 285 };
269 286 }
270 287 };
  288 + scope.vendedorCobrador = {
  289 + NOM: true,
  290 + domicilio: true,
  291 + codigoPostal: true,
  292 + dni: true,
  293 + rol: true,
  294 + provincia: {
  295 + NOMBRE: true
  296 + },
  297 + localidad: {
  298 + NOMBRE: true
  299 + },
  300 + };
271 301  
272 302 //act
273 303 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
... ... @@ -278,13 +308,15 @@ describe('Controladores abm vendedor cobrador', function() {
278 308 expect(focaAbmVendedorCobradorService.guardarVendedorCobrador).toHaveBeenCalled();
279 309 });
280 310  
281   - it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y edicion true' +
  311 + it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y edicion true ' +
282 312 'llama a $location.path', function(done)
283 313 {
284 314 inject(function($location) {
285 315  
286 316 //arrange
287   - var scope= {};
  317 + var scope= {
  318 + $watch: function(){},
  319 + };
288 320 var focaAbmVendedorCobradorService = {
289 321 getCodigoSiguiente: function() {
290 322 return {
... ... @@ -306,10 +338,27 @@ describe('Controladores abm vendedor cobrador', function() {
306 338 $timeout: timeout,
307 339 $uibModal: {},
308 340 $location: $location,
309   - focaModalService: {}
  341 + $localStorage: {},
  342 + focaModalService: {
  343 + alert: function() {}
  344 + }
310 345 });
311 346 var promiseFake = Promise.resolve(true, false);
312 347  
  348 + scope.vendedorCobrador = {
  349 + NOM: true,
  350 + domicilio: true,
  351 + codigoPostal: true,
  352 + dni: true,
  353 + rol: true,
  354 + provincia: {
  355 + NOMBRE: true
  356 + },
  357 + localidad: {
  358 + NOMBRE: true
  359 + },
  360 + };
  361 +
313 362 //act
314 363 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
315 364 .and.returnValue(promiseFake);
... ... @@ -324,11 +373,13 @@ describe('Controladores abm vendedor cobrador', function() {
324 373 });
325 374 });
326 375  
327   - it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y scope.edicion false' +
  376 + it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y scope.edicion false ' +
328 377 'llama a scope.cambiarContraseรฑa', function(done)
329 378 {
330 379 //arrange
331   - var scope= {};
  380 + var scope= {
  381 + $watch: function() {}
  382 + };
332 383 var focaAbmVendedorCobradorService = {
333 384 getCodigoSiguiente: function() {
334 385 return {
... ... @@ -350,10 +401,27 @@ describe('Controladores abm vendedor cobrador', function() {
350 401 $timeout: timeout,
351 402 $uibModal: {},
352 403 $location: {},
353   - focaModalService: {}
  404 + $localStorage: {},
  405 + focaModalService: {
  406 + alert: function() {}
  407 + }
354 408 });
355 409 var promiseFake = Promise.resolve(true, false);
356 410  
  411 + scope.vendedorCobrador = {
  412 + NOM: true,
  413 + domicilio: true,
  414 + codigoPostal: true,
  415 + dni: true,
  416 + rol: true,
  417 + provincia: {
  418 + NOMBRE: true
  419 + },
  420 + localidad: {
  421 + NOMBRE: true
  422 + },
  423 + };
  424 +
357 425 //act
358 426 spyOn(focaAbmVendedorCobradorService, 'guardarVendedorCobrador')
359 427 .and.returnValue(promiseFake);
... ... @@ -369,10 +437,12 @@ describe('Controladores abm vendedor cobrador', function() {
369 437 });
370 438  
371 439 it('la funciรณn $scope.guardar llama a guardaVendedorCobrador y llama foca modal alert' +
372   - 'cuando la promsa es rechazada y el error es 409', function(done)
  440 + 'cuando la promesa es rechazada y el error es 409', function(done)
373 441 {
374 442 //arrange
375   - var scope= {};
  443 + var scope= {
  444 + $watch: function() {}
  445 + };
376 446 var focaAbmVendedorCobradorService = {
377 447 getCodigoSiguiente: function() {
378 448 return {
... ... @@ -397,6 +467,7 @@ describe('Controladores abm vendedor cobrador', function() {
397 467 $timeout: timeout,
398 468 $uibModal: {},
399 469 $location: {},
  470 + $localStorage: {},
400 471 focaModalService: focaModalService
401 472 });
402 473 var promiseFake = Promise.reject({ status: 409 });
... ... @@ -418,7 +489,9 @@ describe('Controladores abm vendedor cobrador', function() {
418 489  
419 490 it('la funciรณn $scope.seleccionarProvincia llama a foca Modal', function() {
420 491 //arrange
421   - var scope= {};
  492 + var scope= {
  493 + $watch: function() {}
  494 + };
422 495 var focaAbmVendedorCobradorService = {
423 496 getCodigoSiguiente: function() {
424 497 return {
... ... @@ -442,6 +515,7 @@ describe('Controladores abm vendedor cobrador', function() {
442 515 $timeout: timeout,
443 516 $uibModal: {},
444 517 $location: {},
  518 + $localStorage: {},
445 519 focaModalService: focaModalService
446 520 });
447 521  
... ... @@ -455,7 +529,9 @@ describe('Controladores abm vendedor cobrador', function() {
455 529  
456 530 it('funciรณn $scope.seleccionarLocalidad llama foca alert si no hay provincia', function() {
457 531 //arrange
458   - var scope= {};
  532 + var scope = {
  533 + $watch: function() {}
  534 + };
459 535 var focaAbmVendedorCobradorService = {
460 536 getCodigoSiguiente: function() {
461 537 return {
... ... @@ -479,6 +555,7 @@ describe('Controladores abm vendedor cobrador', function() {
479 555 $timeout: timeout,
480 556 $uibModal: {},
481 557 $location: {},
  558 + $localStorage: {},
482 559 focaModalService: focaModalService
483 560 });
484 561  
... ... @@ -493,7 +570,9 @@ describe('Controladores abm vendedor cobrador', function() {
493 570  
494 571 it('funciรณn $scope.seleccionarLocalidad llama foca modal', function() {
495 572 //arrange
496   - var scope= {};
  573 + var scope= {
  574 + $watch: function() {}
  575 + };
497 576 var focaAbmVendedorCobradorService = {
498 577 getCodigoSiguiente: function() {
499 578 return {
... ... @@ -517,6 +596,7 @@ describe('Controladores abm vendedor cobrador', function() {
517 596 $timeout: timeout,
518 597 $uibModal: {},
519 598 $location: {},
  599 + $localStorage: {},
520 600 focaModalService: focaModalService
521 601 });
522 602  
... ... @@ -533,7 +613,9 @@ describe('Controladores abm vendedor cobrador', function() {
533 613  
534 614 inject(function($location) {
535 615 //arrange
536   - var scope = {};
  616 + var scope = {
  617 + $watch: function() {}
  618 + };
537 619 $controller('focaAbmVendedorCobradorController', {
538 620 $scope: scope,
539 621 focaAbmVendedorCobradorService: {
... ... @@ -553,6 +635,7 @@ describe('Controladores abm vendedor cobrador', function() {
553 635 $timeout: timeout,
554 636 $uibModal: {},
555 637 $location: $location,
  638 + $localStorage: {},
556 639 focaModalService: {}
557 640 });
558 641  
... ... @@ -570,7 +653,9 @@ describe('Controladores abm vendedor cobrador', function() {
570 653 it('la funcion scope.next suma uno a scope.focused', function() {
571 654  
572 655 //arrange
573   - var scope = {};
  656 + var scope = {
  657 + $watch: function() {}
  658 + };
574 659 $controller('focaAbmVendedorCobradorController', {
575 660 $scope: scope,
576 661 focaAbmVendedorCobradorService: {
... ... @@ -590,6 +675,7 @@ describe('Controladores abm vendedor cobrador', function() {
590 675 $timeout: timeout,
591 676 $uibModal: {},
592 677 $location: {},
  678 + $localStorage: {},
593 679 focaModalService: {}
594 680 });
595 681  
... ... @@ -604,7 +690,9 @@ describe('Controladores abm vendedor cobrador', function() {
604 690 it('la funcion scope.cambiarPassword abre modal', function() {
605 691  
606 692 //arrange
607   - var scope = {};
  693 + var scope = {
  694 + $watch: function() {}
  695 + };
608 696 var uibModal = {
609 697 open: function() { }
610 698 };
... ... @@ -627,6 +715,7 @@ describe('Controladores abm vendedor cobrador', function() {
627 715 $timeout: timeout,
628 716 $uibModal: uibModal,
629 717 $location: {},
  718 + $localStorage: {},
630 719 focaModalService: {}
631 720 });
632 721 var responseFake = {
... ... @@ -649,7 +738,9 @@ describe('Controladores abm vendedor cobrador', function() {
649 738 inject(function($location) {
650 739  
651 740 //arrange
652   - var scope = {};
  741 + var scope = {
  742 + $watch: function() {}
  743 + };
653 744 var uibModal = {
654 745 open: function() { }
655 746 };
... ... @@ -672,6 +763,7 @@ describe('Controladores abm vendedor cobrador', function() {
672 763 $timeout: timeout,
673 764 $uibModal: uibModal,
674 765 $location: $location,
  766 + $localStorage: {},
675 767 focaModalService: {}
676 768 });
677 769 var responseFake = {
src/js/controller.js
Diff comments   View file @ 5ee870b
1 1 angular.module('focaAbmVendedorCobrador')
2 2 .controller('focaAbmVendedoresCobradoresController', [
3   - '$scope', 'focaAbmVendedorCobradorService', '$location', '$filter',
4   - 'focaBotoneraLateralService', 'focaModalService',
  3 + '$scope', 'focaAbmVendedorCobradorService', '$location', '$filter',
  4 + 'focaBotoneraLateralService', 'focaModalService', '$localStorage',
5 5 function($scope, focaAbmVendedorCobradorService, $location, $filter,
6   - focaBotoneraLateralService, focaModalService) {
  6 + focaBotoneraLateralService, focaModalService, $localStorage) {
7 7 $scope.now = new Date();
8 8 $scope.vendedoresCobradores = [];
9 9 $scope.filters = '';
... ... @@ -17,6 +17,8 @@ angular.module('focaAbmVendedorCobrador')
17 17 focaBotoneraLateralService.showCancelar(false);
18 18 focaBotoneraLateralService.showGuardar(false);
19 19  
  20 +
  21 +
20 22 $scope.editar = function(id) {
21 23 $location.path('/vendedor-cobrador/' + id);
22 24 };
... ... @@ -26,7 +28,7 @@ angular.module('focaAbmVendedorCobrador')
26 28 focaModalService.confirm('ยฟEstรก seguro que desea borrar el ' + tipo +
27 29 ' ' + vendedorCobrador.NOM + '?')
28 30 .then(function(data) {
29   - if(data) {
  31 + if (data) {
30 32 focaAbmVendedorCobradorService
31 33 .eliminarVendedorCobrador(vendedorCobrador.id)
32 34 .then(function() {
... ... @@ -52,7 +54,7 @@ angular.module('focaAbmVendedorCobrador')
52 54 };
53 55  
54 56 $scope.search = function(key) {
55   - if(key !== 13) return;
  57 + if (key !== 13) return;
56 58  
57 59 focaAbmVendedorCobradorService
58 60 .getVendedoresCobradores($scope.filters)
... ... @@ -67,35 +69,40 @@ angular.module('focaAbmVendedorCobrador')
67 69 });
68 70 };
69 71  
  72 + if ($localStorage.vendedorCobrador) {
  73 + var vendedorCobrador = JSON.parse($localStorage.vendedorCobrador);
  74 + $location.path('/vendedor-cobrador/' + vendedorCobrador.NUM);
  75 + }
  76 +
70 77 function calcularPages(paginaActual) {
71 78 var paginas = [];
72 79 paginas.push(paginaActual);
73 80  
74   - if(paginaActual - 1 > 1) {
  81 + if (paginaActual - 1 > 1) {
75 82  
76 83 paginas.unshift(paginaActual - 1);
77   - if(paginaActual - 2 > 1) {
  84 + if (paginaActual - 2 > 1) {
78 85 paginas.unshift(paginaActual - 2);
79 86 }
80 87 }
81 88  
82   - if(paginaActual + 1 < $scope.lastPage) {
  89 + if (paginaActual + 1 < $scope.lastPage) {
83 90 paginas.push(paginaActual + 1);
84   - if(paginaActual + 2 < $scope.lastPage) {
  91 + if (paginaActual + 2 < $scope.lastPage) {
85 92 paginas.push(paginaActual + 2);
86 93 }
87 94 }
88 95  
89   - if(paginaActual !== 1) {
  96 + if (paginaActual !== 1) {
90 97 paginas.unshift(1);
91 98 }
92 99  
93   - if(paginaActual !== $scope.lastPage) {
  100 + if (paginaActual !== $scope.lastPage) {
94 101 paginas.push($scope.lastPage);
95 102 }
96   -
97 103 return paginas;
98 104 }
  105 +
99 106 // TODO: descomentar cuando se use
100 107 // function primera() {
101 108 // $scope.selectedClientes = 0;
src/js/controllerItem.js
Diff comments   View file @ 5ee870b
1 1 angular.module('focaAbmVendedorCobrador')
2 2 .controller('focaAbmVendedorCobradorController', [
3 3 '$scope', 'focaAbmVendedorCobradorService', '$routeParams', 'focaBotoneraLateralService',
4   - '$timeout', '$uibModal', '$location', 'focaModalService',
  4 + '$timeout', '$uibModal', '$location', 'focaModalService', '$localStorage', '$filter',
5 5 function($scope, focaAbmVendedorCobradorService, $routeParams, focaBotoneraLateralService,
6   - $timeout, $uibModal, $location, focaModalService) {
  6 + $timeout, $uibModal, $location, focaModalService, $localStorage, $filter) {
7 7  
8 8 $scope.guardado = false;
9 9 $scope.now = new Date();
... ... @@ -45,6 +45,9 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
45 45 focaBotoneraLateralService.showGuardar(true, $scope.guardar);
46 46 focaBotoneraLateralService.addCustomButton('Salir', $scope.cancelar);
47 47 });
  48 +
  49 + $timeout(function() {getLSVendedor();});
  50 +
48 51 focaAbmVendedorCobradorService
49 52 .getCodigoSiguiente()
50 53 .then(function(res) {
... ... @@ -78,12 +81,16 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
78 81 focaModalService.alert('Seleccione tipo');
79 82 return;
80 83 } else if (!$scope.vendedorCobrador.provincia.NOMBRE) {
81   - focaModalService.alert('Seleccione una Provincia');
  84 + focaModalService.alert('Seleccione una provincia');
82 85 return;
83 86 } else if (!$scope.vendedorCobrador.localidad.NOMBRE) {
84   - focaModalService.alert('Seleccione una Localidad');
  87 + focaModalService.alert('Seleccione una localidad');
  88 + return;
  89 + } else if (!isNaN($scope.vendedorCobrador.comisionCobrador) ||
  90 + !isNaN($scope.vendedorCobrador.comisionVendedor)) {
  91 + focaModalService.alert('Ingrese una comision vรกlida');
85 92 return;
86   - }
  93 + }
87 94  
88 95 if ($scope.guardado) {
89 96 $scope.cambiarPassword();
... ... @@ -125,6 +132,7 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
125 132 $scope.seleccionarProvincia = function(key) {
126 133 if (key === 13) {
127 134 var parametrosModal = {
  135 + searchText: $scope.vendedorCobrador.provincia.NOMBRE,
128 136 query: '/provincia',
129 137 columnas: [
130 138 {
... ... @@ -153,6 +161,7 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
153 161 });
154 162 }
155 163 };
  164 +
156 165 $scope.seleccionarLocalidad = function(key) {
157 166 if (!$scope.vendedorCobrador.provincia.ID) {
158 167 focaModalService.alert('Seleccione una provincia');
... ... @@ -160,6 +169,7 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
160 169 }
161 170 if (key === 13) {
162 171 var parametrosModal = {
  172 + searchText: $scope.vendedorCobrador.localidad.NOMBRE,
163 173 query: '/localidad/' + $scope.vendedorCobrador.provincia.ID,
164 174 columnas: [
165 175 {
... ... @@ -189,6 +199,29 @@ angular.module(&#39;focaAbmVendedorCobrador&#39;)
189 199 }
190 200 };
191 201  
  202 + $scope.$watch('vendedorCobrador', function(newValue) {
  203 + focaBotoneraLateralService.setPausarData({
  204 + label:'vendedorCobrador',
  205 + val: newValue
  206 + });
  207 + }, true);
  208 +
  209 + function getLSVendedor() {
  210 + var vendedorCobrador = JSON.parse($localStorage.vendedorCobrador || null);
  211 + if (vendedorCobrador) {
  212 + setearVendedor(vendedorCobrador);
  213 + delete $localStorage.vendedorCobrador;
  214 + }
  215 + }
  216 +
  217 + function setearVendedor(vendedorCobrador) {
  218 + $scope.vendedorCobrador = vendedorCobrador;
  219 + $scope.$broadcast('addCabecera', {
  220 + label: 'Vendedor / Cobrador:',
  221 + valor: $filter('rellenarDigitos')(vendedorCobrador.NUM, 2) + ' - ' + vendedorCobrador.NOM
  222 + });
  223 + }
  224 +
192 225 $scope.cancelar = function() {
193 226 if (!$scope.formVendedorCobrador.$pristine) {
194 227  
src/views/foca-abm-vendedor-cobrador-item.html
Diff comments   View file @ 5ee870b
... ... @@ -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>