diff --git a/src/js/controller.js b/src/js/controller.js index f4e4b1f..bd1963d 100644 --- a/src/js/controller.js +++ b/src/js/controller.js @@ -310,6 +310,9 @@ angular.module('focaCrearRemito') .controller('remitoController', focaBotoneraLateralService.endGuardar(true); $scope.saveLoading = false; + $scope.remito.id = data.data.id; + $scope.remito.numeroRemito = data.data.numero; + remitoBusinessService.addArticulos($scope.remito.articulosRemito, $scope.remito.id, $scope.remito.cotizacion.VENDEDOR); @@ -320,9 +323,6 @@ angular.module('focaCrearRemito') .controller('remitoController', // TODO: updatear plazos if ($scope.remito.id === 0) { - - $scope.remito.id = data.data.id; - $scope.remito.numeroRemito = data.data.numero; var plazos = $scope.remito.remitoPlazo; for(var j = 0; j < plazos.length; j++) { @@ -345,7 +345,7 @@ angular.module('focaCrearRemito') .controller('remitoController', init(); }, function(error) { - focaModalService.alert('Hubo un error al crear el remito'); + focaModalService.alert(error.data || 'Hubo un error al crear el remito'); focaBotoneraLateralService.endGuardar(); $scope.saveLoading = false; console.info(error); @@ -524,7 +524,7 @@ angular.module('focaCrearRemito') .controller('remitoController', }, ocultarVendedor); }; - $scope.seleccionarProveedor = function(callback) { + $scope.seleccionarProveedor = function() { if (varlidarRemitoFacturado()) { var parametrosModal = { titulo: 'Búsqueda de Proveedor', @@ -552,14 +552,7 @@ angular.module('focaCrearRemito') .controller('remitoController', }; focaModalService.modal(parametrosModal).then( function(proveedor) { - $scope.remito.proveedor = proveedor; - $scope.remito.idProveedor = proveedor.COD; - $scope.$broadcast('addCabecera',{ - label: 'Proveedor:', - valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' + - proveedor.NOM - }); - callback(); + $scope.seleccionarFlete(proveedor); }, function() { } ); } @@ -689,64 +682,71 @@ angular.module('focaCrearRemito') .controller('remitoController', } }; - $scope.seleccionarTransportista = function() { - $scope.seleccionarProveedor(function() { - if (varlidarRemitoFacturado()) { - var modalInstance = $uibModal.open( - { - ariaLabelledBy: 'Busqueda de Flete', - templateUrl: 'modal-flete.html', - controller: 'focaModalFleteController', - size: 'lg', - resolve: { - parametrosFlete: - function() { - return { - flete: $scope.remito.flete ? '1' : - ($scope.remito.fob ? 'FOB' : - ($scope.remito.flete === undefined ? - null : '0')), - bomba: $scope.remito.bomba ? '1' : - ($scope.remito.bomba === undefined ? - null : '0'), - kilometros: $scope.remito.kilometros - }; - } - } + $scope.seleccionarFlete = function(proveedor) { + if (varlidarRemitoFacturado()) { + var modalInstance = $uibModal.open( + { + ariaLabelledBy: 'Busqueda de Flete', + templateUrl: 'modal-flete.html', + controller: 'focaModalFleteController', + size: 'lg', + resolve: { + parametrosFlete: + function() { + return { + flete: $scope.remito.flete ? '1' : + ($scope.remito.fob ? 'FOB' : + ($scope.remito.flete === undefined ? + null : '0')), + bomba: $scope.remito.bomba ? '1' : + ($scope.remito.bomba === undefined ? + null : '0'), + kilometros: $scope.remito.kilometros + }; + } } - ); - modalInstance.result.then( - function(datos) { - $scope.remito.flete = datos.flete; - $scope.remito.fob = datos.FOB; - $scope.remito.bomba = datos.bomba; - $scope.remito.kilometros = datos.kilometros; - + } + ); + modalInstance.result.then( + function(datos) { + + $scope.remito.proveedor = proveedor; + $scope.remito.idProveedor = proveedor.COD; + $scope.$broadcast('addCabecera',{ + label: 'Proveedor:', + valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' + + proveedor.NOM + }); + + $scope.remito.flete = datos.flete; + $scope.remito.fob = datos.FOB; + $scope.remito.bomba = datos.bomba; + $scope.remito.kilometros = datos.kilometros; + + $scope.$broadcast('addCabecera',{ + label: 'Flete:', + valor: datos.flete ? 'Si' : ($scope.remito.fob ? 'FOB' : 'No') + }); + if (datos.flete) { $scope.$broadcast('addCabecera',{ - label: 'Flete:', - valor: datos.flete ? 'Si' : ($scope.remito.fob ? 'FOB' : 'No') + label: 'Bomba:', + valor: datos.bomba ? 'Si' : 'No' }); - if (datos.flete) { - $scope.$broadcast('addCabecera',{ - label: 'Bomba:', - valor: datos.bomba ? 'Si' : 'No' - }); - $scope.$broadcast('addCabecera',{ - label: 'Kilometros:', - valor: datos.kilometros - }); - } else { - $scope.$broadcast('removeCabecera', 'Bomba:'); - $scope.$broadcast('removeCabecera', 'Kilometros:'); - $scope.remito.bomba = false; - $scope.remito.kilometros = null; - } - }, function() { - $scope.seleccionarTransportista(); + $scope.$broadcast('addCabecera',{ + label: 'Kilometros:', + valor: datos.kilometros + }); + } else { + $scope.$broadcast('removeCabecera', 'Bomba:'); + $scope.$broadcast('removeCabecera', 'Kilometros:'); + $scope.remito.bomba = false; + $scope.remito.kilometros = null; } - ); - } - }); + }, function() { + $scope.seleccionarTransportista(); + } + ); + } }; $scope.seleccionarMoneda = function() {