Merge branch 'master' into 'develop'

Master See merge request !60

Merge branch 'master' into 'develop'
Master See merge request !60
Eric Fernandez
2 parents d28f1da104 f273342595
Showing 4 changed files Show diff stats
src/js/businessService.js
src/js/controller.js
src/js/service.js
src/views/remito.html
@@ -7,7 +7,8 @@ angular.module(&#39;focaCrearRemito&#39;)
                 for(var i = 0; i < articulosRemito.length; i++) {
                     delete articulosRemito[i].editCantidad;
                     delete articulosRemito[i].editPrecio;
-                    articulosRemito[i].idRemito = idRemito;
+                    articulosRemito[i].idRemito = articulosRemito[i].idRemito !== -1 ?
+                        idRemito : articulosRemito[i].idRemito;
                     articulosRemito[i].precio = articulosRemito[i].precio * cotizacion;
                     delete articulosRemito[i].idNotaPedido;
                     crearRemitoService.crearArticulosParaRemito(articulosRemito[i]);
@@ -159,7 +159,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                                 }
                             ];
  
-
                             function valorPrecioCondicion() {
                                 if (notaPedido.idPrecioCondicion > 0) {
                                     return notaPedido.precioCondicion.nombre;
@@ -196,7 +195,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
  
                             if (notaPedido.idPrecioCondicion > 0) {
                                 $scope.idLista = notaPedido.precioCondicion.idListaPrecio;
-                            }else {
+                            } else {
                                 $scope.idLista = -1;
                             }
  
@@ -239,19 +238,16 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                     focaModalService.alert('Ingrese Proveedor');
                     return;
                 } else if (!$scope.remito.cotizacion.moneda.id &&
-                    !$scope.remito.cotizacion.moneda.ID)
-                {
+                    !$scope.remito.cotizacion.moneda.ID){
                     focaModalService.alert('Ingrese Moneda');
                     return;
                 } else if (!$scope.remito.cotizacion.ID) {
                     focaModalService.alert('Ingrese Cotización');
                     return;
-                } else if (
-                    $scope.remito.flete === undefined || $scope.remito.flete === null)
-                {
+                } else if ($scope.remito.flete === undefined || $scope.remito.flete === null){
                     focaModalService.alert('Ingrese Flete');
                     return;
-                } else if ($scope.remito.articulosRemito.length === 0) {
+                } else if ($scope.articulosFiltro().length === 0) {
                     focaModalService.alert('Debe cargar al menos un articulo');
                     return;
                 }
@@ -313,6 +309,9 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         focaBotoneraLateralService.endGuardar(true);
                         $scope.saveLoading = false;
  
+                        remitoBusinessService.addArticulos($scope.remito.articulosRemito,
+                            $scope.remito.id, $scope.remito.cotizacion.VENDEDOR);
+
                         if(data.status === 500) {
                             focaModalService.alert(data.data);
                             return;
@@ -323,10 +322,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
  
                             $scope.remito.id = data.data.id;
                             $scope.remito.numeroRemito = data.data.numero;
-
-                            remitoBusinessService.addArticulos($scope.remito.articulosRemito,
-                                $scope.remito.id, $scope.remito.cotizacion.VENDEDOR);
-
                             var plazos = $scope.remito.remitoPlazo;
  
                             for(var j = 0; j < plazos.length; j++) {
@@ -383,6 +378,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         var newArt =
                         {
                             id: 0,
+                            idRemito: 0,
                             codigo: producto.codigo,
                             sector: producto.sector,
                             sectorCodigo: producto.sector + '-' + producto.codigo,
@@ -415,7 +411,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 if (!$scope.remito.cliente.COD || !$scope.remito.domicilio.id) {
                     focaModalService.alert('Primero seleccione un cliente y un domicilio');
                     return;
-                }else {
+                } else {
                     var modalInstance = $uibModal.open(
                         {
                             ariaLabelledBy: 'Búsqueda de Puntos de descarga',
@@ -556,7 +552,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         function(proveedor) {
                             $scope.remito.proveedor = proveedor;
                             $scope.remito.idProveedor = proveedor.COD;
-
                             $scope.$broadcast('addCabecera',{
                                 label: 'Proveedor:',
                                 valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' +
@@ -592,12 +587,10 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                             MOD: cliente.mod
                         };
  
-
                         var domicilioStamp =
                             domicilio.Calle + ' ' + domicilio.Numero + ', ' +
                             domicilio.Localidad + ', ' + domicilio.Provincia;
                         $scope.remito.domicilioStamp = domicilioStamp;
-
                         $scope.$broadcast('addCabecera',{
                             label: 'Cliente:',
                             valor: $filter('rellenarDigitos')(cliente.cod, 3) + ' - ' + cliente.nom
@@ -610,7 +603,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                         if (domicilio.verPuntos) {
                             delete $scope.remito.domicilio.verPuntos;
                             $scope.seleccionarPuntosDeDescarga();
-                        }else {
+                        } else {
                             crearRemitoService
                                 .getPuntosDescargaByClienDom(domicilio.id, cliente.cod)
                                 .then(function(res) {
@@ -626,7 +619,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
  
             $scope.getTotal = function() {
                 var total = 0;
-                var arrayTempArticulos = $scope.remito.articulosRemito;
+                var arrayTempArticulos = $scope.articulosFiltro();
                 for(var i = 0; i < arrayTempArticulos.length; i++) {
                     total += arrayTempArticulos[i].precio * arrayTempArticulos[i].cantidad;
                 }
@@ -671,10 +664,10 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                                 }
                                 cabecera = $filter('rellenarDigitos')(precioCondicion.id, 4) +
                                     ' - ' + precioCondicion.nombre + ' ' + plazosConcat.trim();
-                            }else { //Cuando se ingresan los plazos manualmente
+                            } else { //Cuando se ingresan los plazos manualmente
                                 $scope.remito.idPrecioCondicion = 0;
                                 //-1, el modal productos busca todos los productos
-                                $scope.idLista = -1; 
+                                $scope.idLista = -1;
                                 $scope.remito.remitoPlazo = precioCondicion;
                                 for(var j = 0; j < precioCondicion.length; j++) {
                                     plazosConcat += precioCondicion[j].dias + ' ';
@@ -686,7 +679,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                                 label: 'Precios y condiciones:',
                                 valor: cabecera
                             });
-
                             $scope.remito.precioCondicion = precioCondicion;
                         }, function() {
  
@@ -821,7 +813,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                             $scope.$broadcast('removeCabecera', 'Moneda:');
                             $scope.$broadcast('removeCabecera', 'Fecha cotizacion:');
                             $scope.$broadcast('removeCabecera', 'Cotizacion:');
-                        }else {
+                        } else {
                             $scope.$broadcast('addCabecera',{
                                 label: 'Moneda:',
                                 valor: moneda.DETALLE
@@ -855,8 +847,16 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 }
             };
  
-            $scope.quitarArticulo = function(key) {
-                $scope.remito.articulosRemito.splice(key, 1);
+            $scope.quitarArticulo = function(articulo) {
+                articulo.idRemito = -1;
+            };
+
+            $scope.articulosFiltro = function() {
+
+                var result = $scope.remito.articulosRemito.filter(function(articulo) {
+                    return articulo.idRemito >= 0;
+                });
+                return result;
             };
  
             $scope.editarArticulo = function(key, articulo) {
@@ -915,7 +915,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 while (relleno.length < longitud) {
                     relleno = '0' + relleno;
                 }
-
                 return relleno;
             }
  
@@ -952,7 +951,6 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 var boton = $scope.botonera.filter(function(botonObs) {
                     return botonObs.label === 'Observaciones';
                 });
-
                 boton[0].disable = !val;
             }
  
@@ -1091,7 +1089,7 @@ angular.module(&#39;focaCrearRemito&#39;) .controller(&#39;remitoController&#39;,
                 .then(function(res) {
                     if (res === false) {
                         abrirModalMail(id);
-                        focaModalService.alert('Descarga o envíe su remito ' + 
+                        focaModalService.alert('Descarga o envíe su remito ' +
                         'antes de cerrar esta ventana');
                     }
                 });
@@ -83,7 +83,7 @@ angular.module(&#39;focaCrearRemito&#39;)
                         image:  'cliente.png'
                     },
                     {
-                        label: 'Transportista',
+                        label: 'Proveedor',
                         image:  'proveedor.png'
                     },
                     {
@@ -11,7 +11,7 @@
             behavior="scroll"
             direction="left"
             ng-bind="remito.observaciones"
-            ></marquee>
+        ></marquee>
     <div class="col-lg-12">
         <div class="row mt-4">
             <div class="col-12 col-md-10 col-lg-10 border border-light rounded">
@@ -53,8 +53,8 @@
                         </thead>
                         <tbody class="tabla-articulo-body">
                             <tr
-                                ng-repeat="(key, articulo) in remito.articulosRemito"
-                                ng-show="show || key == (remito.articulosRemito.length - 1)"
+                                ng-repeat="(key, articulo) in articulosFiltro()"
+                                ng-show="show || key == (articulosFiltro().length - 1)"
                                 class="d-flex"
                             >
                                 <td ng-bind="key + 1"></td>
@@ -112,7 +112,7 @@
                                 <td class="text-center">
                                     <button
                                         class="btn btn-outline-light"
-                                        ng-click="quitarArticulo(key)"
+                                        ng-click="quitarArticulo(articulo)"
                                     >
                                         <i class="fa fa-trash"></i>
                                     </button>
@@ -130,7 +130,7 @@
                             <tr ng-show="!cargando" class="d-flex">
                                 <td 
                                     class="align-middle" 
-                                    ng-bind="remito.articulosRemito.length + 1"
+                                    ng-bind="articulosFiltro().length + 1"
                                 ></td>
                                 <td class="col">
                                     <input
@@ -195,7 +195,7 @@
                             <tr class="d-flex">
                                 <td colspan="4" class="no-border-top">
                                     <strong>Items:</strong>
-                                    <a ng-bind="remito.articulosRemito.length"></a>
+                                    <a ng-bind="articulosFiltro().length"></a>
                                 </td>
                                 <td class="text-right ml-auto table-celda-total no-border-top">
                                     <h3>Total:</h3>
...	...	@@ -7,7 +7,8 @@ angular.module('focaCrearRemito')
7	7	for(var i = 0; i < articulosRemito.length; i++) {
8	8	delete articulosRemito[i].editCantidad;
9	9	delete articulosRemito[i].editPrecio;
10		- articulosRemito[i].idRemito = idRemito;
	10	+ articulosRemito[i].idRemito = articulosRemito[i].idRemito !== -1 ?
	11	+ idRemito : articulosRemito[i].idRemito;
11	12	articulosRemito[i].precio = articulosRemito[i].precio * cotizacion;
12	13	delete articulosRemito[i].idNotaPedido;
13	14	crearRemitoService.crearArticulosParaRemito(articulosRemito[i]);
...	...	@@ -159,7 +159,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
159	159	}
160	160	];
161	161
162		-
163	162	function valorPrecioCondicion() {
164	163	if (notaPedido.idPrecioCondicion > 0) {
165	164	return notaPedido.precioCondicion.nombre;
...	...	@@ -196,7 +195,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
196	195
197	196	if (notaPedido.idPrecioCondicion > 0) {
198	197	$scope.idLista = notaPedido.precioCondicion.idListaPrecio;
199		- }else {
	198	+ } else {
200	199	$scope.idLista = -1;
201	200	}
202	201
...	...	@@ -239,19 +238,16 @@ angular.module('focaCrearRemito') .controller('remitoController',
239	238	focaModalService.alert('Ingrese Proveedor');
240	239	return;
241	240	} else if (!$scope.remito.cotizacion.moneda.id &&
242		- !$scope.remito.cotizacion.moneda.ID)
243		- {
	241	+ !$scope.remito.cotizacion.moneda.ID){
244	242	focaModalService.alert('Ingrese Moneda');
245	243	return;
246	244	} else if (!$scope.remito.cotizacion.ID) {
247	245	focaModalService.alert('Ingrese Cotización');
248	246	return;
249		- } else if (
250		- $scope.remito.flete === undefined \|\| $scope.remito.flete === null)
251		- {
	247	+ } else if ($scope.remito.flete === undefined \|\| $scope.remito.flete === null){
252	248	focaModalService.alert('Ingrese Flete');
253	249	return;
254		- } else if ($scope.remito.articulosRemito.length === 0) {
	250	+ } else if ($scope.articulosFiltro().length === 0) {
255	251	focaModalService.alert('Debe cargar al menos un articulo');
256	252	return;
257	253	}
...	...	@@ -313,6 +309,9 @@ angular.module('focaCrearRemito') .controller('remitoController',
313	309	focaBotoneraLateralService.endGuardar(true);
314	310	$scope.saveLoading = false;
315	311
	312	+ remitoBusinessService.addArticulos($scope.remito.articulosRemito,
	313	+ $scope.remito.id, $scope.remito.cotizacion.VENDEDOR);
	314	+
316	315	if(data.status === 500) {
317	316	focaModalService.alert(data.data);
318	317	return;
...	...	@@ -323,10 +322,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
323	322
324	323	$scope.remito.id = data.data.id;
325	324	$scope.remito.numeroRemito = data.data.numero;
326		-
327		- remitoBusinessService.addArticulos($scope.remito.articulosRemito,
328		- $scope.remito.id, $scope.remito.cotizacion.VENDEDOR);
329		-
330	325	var plazos = $scope.remito.remitoPlazo;
331	326
332	327	for(var j = 0; j < plazos.length; j++) {
...	...	@@ -383,6 +378,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
383	378	var newArt =
384	379	{
385	380	id: 0,
	381	+ idRemito: 0,
386	382	codigo: producto.codigo,
387	383	sector: producto.sector,
388	384	sectorCodigo: producto.sector + '-' + producto.codigo,
...	...	@@ -415,7 +411,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
415	411	if (!$scope.remito.cliente.COD \|\| !$scope.remito.domicilio.id) {
416	412	focaModalService.alert('Primero seleccione un cliente y un domicilio');
417	413	return;
418		- }else {
	414	+ } else {
419	415	var modalInstance = $uibModal.open(
420	416	{
421	417	ariaLabelledBy: 'Búsqueda de Puntos de descarga',
...	...	@@ -556,7 +552,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
556	552	function(proveedor) {
557	553	$scope.remito.proveedor = proveedor;
558	554	$scope.remito.idProveedor = proveedor.COD;
559		-
560	555	$scope.$broadcast('addCabecera',{
561	556	label: 'Proveedor:',
562	557	valor: $filter('rellenarDigitos')(proveedor.COD, 5) + ' - ' +
...	...	@@ -592,12 +587,10 @@ angular.module('focaCrearRemito') .controller('remitoController',
592	587	MOD: cliente.mod
593	588	};
594	589
595		-
596	590	var domicilioStamp =
597	591	domicilio.Calle + ' ' + domicilio.Numero + ', ' +
598	592	domicilio.Localidad + ', ' + domicilio.Provincia;
599	593	$scope.remito.domicilioStamp = domicilioStamp;
600		-
601	594	$scope.$broadcast('addCabecera',{
602	595	label: 'Cliente:',
603	596	valor: $filter('rellenarDigitos')(cliente.cod, 3) + ' - ' + cliente.nom
...	...	@@ -610,7 +603,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
610	603	if (domicilio.verPuntos) {
611	604	delete $scope.remito.domicilio.verPuntos;
612	605	$scope.seleccionarPuntosDeDescarga();
613		- }else {
	606	+ } else {
614	607	crearRemitoService
615	608	.getPuntosDescargaByClienDom(domicilio.id, cliente.cod)
616	609	.then(function(res) {
...	...	@@ -626,7 +619,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
626	619
627	620	$scope.getTotal = function() {
628	621	var total = 0;
629		- var arrayTempArticulos = $scope.remito.articulosRemito;
	622	+ var arrayTempArticulos = $scope.articulosFiltro();
630	623	for(var i = 0; i < arrayTempArticulos.length; i++) {
631	624	total += arrayTempArticulos[i].precio * arrayTempArticulos[i].cantidad;
632	625	}
...	...	@@ -671,10 +664,10 @@ angular.module('focaCrearRemito') .controller('remitoController',
671	664	}
672	665	cabecera = $filter('rellenarDigitos')(precioCondicion.id, 4) +
673	666	' - ' + precioCondicion.nombre + ' ' + plazosConcat.trim();
674		- }else { //Cuando se ingresan los plazos manualmente
	667	+ } else { //Cuando se ingresan los plazos manualmente
675	668	$scope.remito.idPrecioCondicion = 0;
676	669	//-1, el modal productos busca todos los productos
677		- $scope.idLista = -1;
	670	+ $scope.idLista = -1;
678	671	$scope.remito.remitoPlazo = precioCondicion;
679	672	for(var j = 0; j < precioCondicion.length; j++) {
680	673	plazosConcat += precioCondicion[j].dias + ' ';
...	...	@@ -686,7 +679,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
686	679	label: 'Precios y condiciones:',
687	680	valor: cabecera
688	681	});
689		-
690	682	$scope.remito.precioCondicion = precioCondicion;
691	683	}, function() {
692	684
...	...	@@ -821,7 +813,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
821	813	$scope.$broadcast('removeCabecera', 'Moneda:');
822	814	$scope.$broadcast('removeCabecera', 'Fecha cotizacion:');
823	815	$scope.$broadcast('removeCabecera', 'Cotizacion:');
824		- }else {
	816	+ } else {
825	817	$scope.$broadcast('addCabecera',{
826	818	label: 'Moneda:',
827	819	valor: moneda.DETALLE
...	...	@@ -855,8 +847,16 @@ angular.module('focaCrearRemito') .controller('remitoController',
855	847	}
856	848	};
857	849
858		- $scope.quitarArticulo = function(key) {
859		- $scope.remito.articulosRemito.splice(key, 1);
	850	+ $scope.quitarArticulo = function(articulo) {
	851	+ articulo.idRemito = -1;
	852	+ };
	853	+
	854	+ $scope.articulosFiltro = function() {
	855	+
	856	+ var result = $scope.remito.articulosRemito.filter(function(articulo) {
	857	+ return articulo.idRemito >= 0;
	858	+ });
	859	+ return result;
860	860	};
861	861
862	862	$scope.editarArticulo = function(key, articulo) {
...	...	@@ -915,7 +915,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
915	915	while (relleno.length < longitud) {
916	916	relleno = '0' + relleno;
917	917	}
918		-
919	918	return relleno;
920	919	}
921	920
...	...	@@ -952,7 +951,6 @@ angular.module('focaCrearRemito') .controller('remitoController',
952	951	var boton = $scope.botonera.filter(function(botonObs) {
953	952	return botonObs.label === 'Observaciones';
954	953	});
955		-
956	954	boton[0].disable = !val;
957	955	}
958	956
...	...	@@ -1091,7 +1089,7 @@ angular.module('focaCrearRemito') .controller('remitoController',
1091	1089	.then(function(res) {
1092	1090	if (res === false) {
1093	1091	abrirModalMail(id);
1094		- focaModalService.alert('Descarga o envíe su remito ' +
	1092	+ focaModalService.alert('Descarga o envíe su remito ' +
1095	1093	'antes de cerrar esta ventana');
1096	1094	}
1097	1095	});
...	...	@@ -83,7 +83,7 @@ angular.module('focaCrearRemito')
83	83	image: 'cliente.png'
84	84	},
85	85	{
86		- label: 'Transportista',
	86	+ label: 'Proveedor',
87	87	image: 'proveedor.png'
88	88	},
89	89	{
...	...	@@ -11,7 +11,7 @@
11	11	behavior="scroll"
12	12	direction="left"
13	13	ng-bind="remito.observaciones"
14		- ></marquee>
	14	+ ></marquee>
15	15	<div class="col-lg-12">
16	16	<div class="row mt-4">
17	17	<div class="col-12 col-md-10 col-lg-10 border border-light rounded">
...	...	@@ -53,8 +53,8 @@
53	53	</thead>
54	54	<tbody class="tabla-articulo-body">
55	55	<tr
56		- ng-repeat="(key, articulo) in remito.articulosRemito"
57		- ng-show="show \|\| key == (remito.articulosRemito.length - 1)"
	56	+ ng-repeat="(key, articulo) in articulosFiltro()"
	57	+ ng-show="show \|\| key == (articulosFiltro().length - 1)"
58	58	class="d-flex"
59	59	>
60	60	<td ng-bind="key + 1"></td>
...	...	@@ -112,7 +112,7 @@
112	112	<td class="text-center">
113	113	<button
114	114	class="btn btn-outline-light"
115		- ng-click="quitarArticulo(key)"
	115	+ ng-click="quitarArticulo(articulo)"
116	116	>
117	117	<i class="fa fa-trash"></i>
118	118	</button>
...	...	@@ -130,7 +130,7 @@
130	130	<tr ng-show="!cargando" class="d-flex">
131	131	<td
132	132	class="align-middle"
133		- ng-bind="remito.articulosRemito.length + 1"
	133	+ ng-bind="articulosFiltro().length + 1"
134	134	></td>
135	135	<td class="col">
136	136	<input
...	...	@@ -195,7 +195,7 @@
195	195	<tr class="d-flex">
196	196	<td colspan="4" class="no-border-top">
197	197	<strong>Items:</strong>
198		- <a ng-bind="remito.articulosRemito.length"></a>
	198	+ <a ng-bind="articulosFiltro().length"></a>
199	199	</td>
200	200	<td class="text-right ml-auto table-celda-total no-border-top">
201	201	<h3>Total:</h3>