Commit efb32e7fead503c6320945f8ef1d07a22e896fae
1 parent
0d14fbc592
Exists in
master
refactor selección de cliente
Showing
1 changed file
with
25 additions
and
29 deletions
Show diff stats
src/js/controller.js
| ... | ... | @@ -354,12 +354,9 @@ angular.module('focaCrearCobranza').controller('cobranzaController', |
| 354 | 354 | size: 'lg' |
| 355 | 355 | } |
| 356 | 356 | ); |
| 357 | - modalInstance.result.then( | |
| 358 | - function (cliente) { | |
| 359 | - $scope.abrirModalDomicilios(cliente); | |
| 360 | - $scope.cliente = cliente; | |
| 361 | - }, function () { } | |
| 362 | - ); | |
| 357 | + modalInstance.result | |
| 358 | + .then($scope.abrirModalDomicilios) | |
| 359 | + .catch(console.log); | |
| 363 | 360 | }; |
| 364 | 361 | |
| 365 | 362 | $scope.seleccionarCliente = function () { |
| ... | ... | @@ -370,49 +367,48 @@ angular.module('focaCrearCobranza').controller('cobranzaController', |
| 370 | 367 | templateUrl: 'foca-busqueda-cliente-modal.html', |
| 371 | 368 | controller: 'focaBusquedaClienteModalController', |
| 372 | 369 | resolve: { |
| 373 | - vendedor: function () { return null; }, | |
| 374 | - cobrador: function () { return null; } | |
| 370 | + parametros: function () { | |
| 371 | + return { | |
| 372 | + vendedor: function () { return null; }, | |
| 373 | + cobrador: function () { return null; } | |
| 374 | + }; | |
| 375 | + } | |
| 375 | 376 | }, |
| 376 | 377 | size: 'lg' |
| 377 | 378 | } |
| 378 | 379 | ); |
| 379 | 380 | modalInstance.result.then( |
| 380 | 381 | function (cliente) { |
| 382 | + | |
| 383 | + if (angular.equals({}, cliente.cobrador) || !cliente.cobrador) { | |
| 384 | + focaModalService.alert('El cliente no tiene un cobrador asignado') | |
| 385 | + .then($scope.seleccionarCliente); | |
| 386 | + return; | |
| 387 | + } | |
| 388 | + | |
| 381 | 389 | //ELIMINO CLIENTE |
| 382 | 390 | $scope.$broadcast('removeCabecera', 'Cliente:'); |
| 383 | - $scope.cobranza.cliente = {}; | |
| 384 | - | |
| 385 | - var clienteMayus = { | |
| 386 | - COD: cliente.cod, | |
| 387 | - NOM: cliente.nom, | |
| 388 | - CUIT: cliente.cuit, | |
| 389 | - MAIL: cliente.mail, | |
| 390 | - DOM: cliente.DOM, | |
| 391 | - LOC: cliente.LOC, | |
| 392 | - PCI: cliente.PCI, | |
| 393 | - idCobrador: cliente.idCobrador, | |
| 394 | - cobrador: cliente.cobrador | |
| 395 | - }; | |
| 391 | + | |
| 392 | + $scope.cobranza.cliente = cliente; | |
| 393 | + | |
| 396 | 394 | $scope.$broadcast('addCabecera', { |
| 397 | 395 | label: 'Cobrador:', |
| 398 | - valor: $filter('rellenarDigitos')(clienteMayus.cobrador.NUM, 3) + | |
| 399 | - ' - ' + clienteMayus.cobrador.NOM | |
| 396 | + valor: $filter('rellenarDigitos')(cliente.cobrador.NUM, 3) + | |
| 397 | + ' - ' + cliente.cobrador.NOM | |
| 400 | 398 | }); |
| 401 | - $scope.cobranza.cobrador = clienteMayus.cobrador; | |
| 399 | + $scope.cobranza.cobrador = cliente.cobrador; | |
| 402 | 400 | |
| 403 | 401 | $scope.$broadcast('addCabecera', { |
| 404 | 402 | label: 'Cliente:', |
| 405 | - valor: $filter('rellenarDigitos')(clienteMayus.COD, 5) + ' - ' + | |
| 406 | - clienteMayus.NOM | |
| 403 | + valor: $filter('rellenarDigitos')(cliente.COD, 5) + ' - ' + | |
| 404 | + cliente.NOM | |
| 407 | 405 | }); |
| 408 | 406 | |
| 409 | 407 | $filter('filter')($scope.botonera, |
| 410 | 408 | { label: 'Cliente' })[0].checked = true; |
| 411 | 409 | |
| 412 | - $scope.cobranza.cliente = clienteMayus; | |
| 413 | 410 | $scope.cobranza.facturas = []; |
| 414 | - focaCrearCobranzaService | |
| 415 | - .getCobradorById($scope.cobranza.cliente.idCobrador); | |
| 411 | + | |
| 416 | 412 | }, function () { |
| 417 | 413 | } |
| 418 | 414 | ); |