Merge branch 'master' into 'develop'

Master(efernandez) See merge request !63

Merge branch 'master' into 'develop'
Master(efernandez) See merge request !63
Eric Fernandez
2 parents ed8024d650 49f31e0f40
Showing 1 changed file Show diff stats
src/js/controller.js
@@ -33,6 +33,8 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                     );
                     $scope.inicial.cotizacion = $scope.remito.cotizacion;
                     $scope.cotizacionPorDefecto = angular.copy($scope.remito.cotizacion);
+
+                    $timeout(function() { getLSRemito(); });
                 });
  
                 //SETEO BOTONERA LATERAL
@@ -44,7 +46,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 });
  
                 init();
-                $timeout(function() {getLSRemito();});
             }
  
             function init() {
@@ -310,6 +311,9 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         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 +324,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
  
                         // 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 +346,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         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 +525,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 }, ocultarVendedor);
             };
  
-            $scope.seleccionarProveedor = function(callback) {
+            $scope.seleccionarProveedor = function() {
                 if (varlidarRemitoFacturado()) {
                     var parametrosModal = {
                         titulo: 'Búsqueda de Proveedor',
@@ -552,14 +553,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                     };
                     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 +683,71 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 }
             };
  
-            $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() {
...	...	@@ -33,6 +33,8 @@ angular.module('focaCrearRemito') .controller('remitoController',
33	33	);
34	34	$scope.inicial.cotizacion = $scope.remito.cotizacion;
35	35	$scope.cotizacionPorDefecto = angular.copy($scope.remito.cotizacion);
	36	+
	37	+ $timeout(function() { getLSRemito(); });
36	38	});
37	39
38	40	//SETEO BOTONERA LATERAL
...	...	@@ -44,7 +46,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
44	46	});
45	47
46	48	init();
47		- $timeout(function() {getLSRemito();});
48	49	}
49	50
50	51	function init() {
...	...	@@ -310,6 +311,9 @@ angular.module('focaCrearRemito') .controller('remitoController',
310	311	focaBotoneraLateralService.endGuardar(true);
311	312	$scope.saveLoading = false;
312	313
	314	+ $scope.remito.id = data.data.id;
	315	+ $scope.remito.numeroRemito = data.data.numero;
	316	+
313	317	remitoBusinessService.addArticulos($scope.remito.articulosRemito,
314	318	$scope.remito.id, $scope.remito.cotizacion.VENDEDOR);
315	319
...	...	@@ -320,9 +324,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
320	324
321	325	// TODO: updatear plazos
322	326	if ($scope.remito.id === 0) {
323		-
324		- $scope.remito.id = data.data.id;
325		- $scope.remito.numeroRemito = data.data.numero;
326	327	var plazos = $scope.remito.remitoPlazo;
327	328
328	329	for(var j = 0; j < plazos.length; j++) {
...	...	@@ -345,7 +346,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
345	346	init();
346	347
347	348	}, function(error) {
348		- focaModalService.alert('Hubo un error al crear el remito');
	349	+ focaModalService.alert(error.data \|\| 'Hubo un error al crear el remito');
349	350	focaBotoneraLateralService.endGuardar();
350	351	$scope.saveLoading = false;
351	352	console.info(error);
...	...	@@ -524,7 +525,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
524	525	}, ocultarVendedor);
525	526	};
526	527
527		- $scope.seleccionarProveedor = function(callback) {
	528	+ $scope.seleccionarProveedor = function() {
528	529	if (varlidarRemitoFacturado()) {
529	530	var parametrosModal = {
530	531	titulo: 'Búsqueda de Proveedor',
...	...	@@ -552,14 +553,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
552	553	};
553	554	focaModalService.modal(parametrosModal).then(
554	555	function(proveedor) {
555		- $scope.remito.proveedor = proveedor;
556		- $scope.remito.idProveedor = proveedor.COD;
557		- $scope.$broadcast('addCabecera',{
558		- label: 'Proveedor:',
559		- valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' +
560		- proveedor.NOM
561		- });
562		- callback();
	556	+ $scope.seleccionarFlete(proveedor);
563	557	}, function() { }
564	558	);
565	559	}
...	...	@@ -689,64 +683,71 @@ angular.module('focaCrearRemito') .controller('remitoController',
689	683	}
690	684	};
691	685
692		- $scope.seleccionarTransportista = function() {
693		- $scope.seleccionarProveedor(function() {
694		- if (varlidarRemitoFacturado()) {
695		- var modalInstance = $uibModal.open(
696		- {
697		- ariaLabelledBy: 'Busqueda de Flete',
698		- templateUrl: 'modal-flete.html',
699		- controller: 'focaModalFleteController',
700		- size: 'lg',
701		- resolve: {
702		- parametrosFlete:
703		- function() {
704		- return {
705		- flete: $scope.remito.flete ? '1' :
706		- ($scope.remito.fob ? 'FOB' :
707		- ($scope.remito.flete === undefined ?
708		- null : '0')),
709		- bomba: $scope.remito.bomba ? '1' :
710		- ($scope.remito.bomba === undefined ?
711		- null : '0'),
712		- kilometros: $scope.remito.kilometros
713		- };
714		- }
715		- }
	686	+ $scope.seleccionarFlete = function(proveedor) {
	687	+ if (varlidarRemitoFacturado()) {
	688	+ var modalInstance = $uibModal.open(
	689	+ {
	690	+ ariaLabelledBy: 'Busqueda de Flete',
	691	+ templateUrl: 'modal-flete.html',
	692	+ controller: 'focaModalFleteController',
	693	+ size: 'lg',
	694	+ resolve: {
	695	+ parametrosFlete:
	696	+ function() {
	697	+ return {
	698	+ flete: $scope.remito.flete ? '1' :
	699	+ ($scope.remito.fob ? 'FOB' :
	700	+ ($scope.remito.flete === undefined ?
	701	+ null : '0')),
	702	+ bomba: $scope.remito.bomba ? '1' :
	703	+ ($scope.remito.bomba === undefined ?
	704	+ null : '0'),
	705	+ kilometros: $scope.remito.kilometros
	706	+ };
	707	+ }
716	708	}
717		- );
718		- modalInstance.result.then(
719		- function(datos) {
720		- $scope.remito.flete = datos.flete;
721		- $scope.remito.fob = datos.FOB;
722		- $scope.remito.bomba = datos.bomba;
723		- $scope.remito.kilometros = datos.kilometros;
724		-
	709	+ }
	710	+ );
	711	+ modalInstance.result.then(
	712	+ function(datos) {
	713	+
	714	+ $scope.remito.proveedor = proveedor;
	715	+ $scope.remito.idProveedor = proveedor.COD;
	716	+ $scope.$broadcast('addCabecera',{
	717	+ label: 'Proveedor:',
	718	+ valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' +
	719	+ proveedor.NOM
	720	+ });
	721	+
	722	+ $scope.remito.flete = datos.flete;
	723	+ $scope.remito.fob = datos.FOB;
	724	+ $scope.remito.bomba = datos.bomba;
	725	+ $scope.remito.kilometros = datos.kilometros;
	726	+
	727	+ $scope.$broadcast('addCabecera',{
	728	+ label: 'Flete:',
	729	+ valor: datos.flete ? 'Si' : ($scope.remito.fob ? 'FOB' : 'No')
	730	+ });
	731	+ if (datos.flete) {
725	732	$scope.$broadcast('addCabecera',{
726		- label: 'Flete:',
727		- valor: datos.flete ? 'Si' : ($scope.remito.fob ? 'FOB' : 'No')
	733	+ label: 'Bomba:',
	734	+ valor: datos.bomba ? 'Si' : 'No'
728	735	});
729		- if (datos.flete) {
730		- $scope.$broadcast('addCabecera',{
731		- label: 'Bomba:',
732		- valor: datos.bomba ? 'Si' : 'No'
733		- });
734		- $scope.$broadcast('addCabecera',{
735		- label: 'Kilometros:',
736		- valor: datos.kilometros
737		- });
738		- } else {
739		- $scope.$broadcast('removeCabecera', 'Bomba:');
740		- $scope.$broadcast('removeCabecera', 'Kilometros:');
741		- $scope.remito.bomba = false;
742		- $scope.remito.kilometros = null;
743		- }
744		- }, function() {
745		- $scope.seleccionarTransportista();
	736	+ $scope.$broadcast('addCabecera',{
	737	+ label: 'Kilometros:',
	738	+ valor: datos.kilometros
	739	+ });
	740	+ } else {
	741	+ $scope.$broadcast('removeCabecera', 'Bomba:');
	742	+ $scope.$broadcast('removeCabecera', 'Kilometros:');
	743	+ $scope.remito.bomba = false;
	744	+ $scope.remito.kilometros = null;
746	745	}
747		- );
748		- }
749		- });
	746	+ }, function() {
	747	+ $scope.seleccionarTransportista();
	748	+ }
	749	+ );
	750	+ }
750	751	};
751	752
752	753	$scope.seleccionarMoneda = function() {