X-Git-Url: https://git.llucax.com/z.facultad/75.06/emufs.git/blobdiff_plain/e81cf1f1af923bee85d60e896bbbe0491bca4264..0e6e058c514e07878085512095d5be4f5b50e8d9:/emufs_gui/articulos.c

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index defdf7e..4d78133 100644
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -70,6 +70,7 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 	xmlDocPtr document;
 	xmlNode *node, *inicio;
 	int error = 0, i;
+	char *prop;
 	EMUFS_REG_SIZE size;
 	t_LstArticulos *tmp;
 	lst_articulos = NULL;
@@ -81,8 +82,11 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 	tmp->primero = NULL;
 
 	if (filename != NULL) {
+		PERR("Voy a crear desde un XML");
 		document = xmlReadFile(filename, "ISO-8859-1",0);
 		if (document == NULL) {
+			free(tmp);
+			lst_articulos = NULL;
 			return NULL;
 		}
 
@@ -106,13 +110,15 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 					t_Articulo art;
 					void *save;
 					memset(&art, '*', sizeof(t_Articulo));
-					art.numero = atoi(xmlGetProp(node, "NroArtículo"));
-					strcpy(art.desc, xmlGetProp(node, "Descripción"));
-					strcpy(art.presentacion, xmlGetProp(node, "Presentación"));
-					strcpy(art.existencia, xmlGetProp(node, "Existencia"));
+					prop = xmlGetProp(node, "NroArtículo");
+					art.numero = atoi(prop);
+					xmlFree(prop);
+					strcpy(art.desc, prop = xmlGetProp(node, "Descripción")); xmlFree(prop);
+					strcpy(art.presentacion, prop = xmlGetProp(node, "Presentación")); xmlFree(prop);
+					strcpy(art.existencia, prop = xmlGetProp(node, "Existencia")); xmlFree(prop);
 					/*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/
-					strcpy(art.pvu, xmlGetProp(node, "PVU"));
-					strcpy(art.emin, xmlGetProp(node, "Emín"));
+					strcpy(art.pvu, prop = xmlGetProp(node, "PVU")); xmlFree(prop);
+					strcpy(art.emin, prop = xmlGetProp(node, "Emín")); xmlFree(prop);
 					/* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
 					save = procesar_guardar_articulo(&art, &size, lst_articulos);
 					if (save != NULL) {
@@ -127,7 +133,14 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
 		xmlFreeDoc(document);
 		xmlCleanupParser();
 	} else {
+		PERR("Voy a recuperar desde un archivo");
 		tmp->fp = emufs_abrir("articulos");
+		if (tmp->fp == NULL) {
+			PERR("No se pudo cargar archivo de articulos.");
+			free(tmp);
+			lst_articulos = NULL;
+			return NULL;
+		}
 		/* Ahora trato de recuperar la info */
 		indices = emufs_idx_get(tmp->fp, &indices_cant);
 		for(i=0; i<indices_cant; i++) {
@@ -151,9 +164,14 @@ int art_liberar(t_LstArticulos *l)
 {
 	if (l == NULL) l = lst_articulos;
 	if (l == NULL) return 1;
+	t_Reg_Articulo *del;
 
 	emufs_destruir(l->fp);
-	/* TODO : Liberar lista */
+	while (l->primero) {
+		del = l->primero;
+		l->primero = l->primero->sig;
+		free(del);
+	}
 	free(l);
 
 	lst_articulos = NULL;
@@ -408,7 +426,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
 		case T2:
 			ini = (char *)src;
 			/* Copio el primer campo, esto es facil :-) */
-			memset(dst, '*', sizeof(t_Articulo));
+			memset(dst, 0, sizeof(t_Articulo));
 			memcpy(&dst->numero, ini, sizeof(unsigned int));
 			ini+=sizeof(unsigned int);
 			/* Ahora empieza el juego */
@@ -463,6 +481,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
 			(*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5];
 			tmp = (char *)malloc((*size));
 			if (tmp == NULL) return NULL;
+			memset(tmp, 0, *size);
 			memcpy(tmp, &src->numero, i[0]);
 			memcpy(tmp+i[0], src->desc, i[1]);
 			memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);
@@ -475,6 +494,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
 			 * registro se vea bien 
 			 */
 			tmp = (char *)malloc(sizeof(t_Articulo));
+			memset(tmp, '*', sizeof(t_Articulo));
 			memcpy(tmp, from, sizeof(t_Articulo));
 			(*size) = sizeof(t_Articulo);
 	}
@@ -491,15 +511,17 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 	int error;
 	char *save;
 
+	PERR("==== EMPIEZO ====\n");
 	old = lst_articulos->fp;
 
 	/* Si el tipo es el mismo, no tengo que hacer nada! */
 	if (old->tipo == tipo) return;
 
 	/* Creo el nuevo file */
+	PERR("Creo el archivo\n");
 	nuevo = emufs_crear("emufs_tmp", tipo, tam_bloque, sizeof(t_Articulo));
 	if (nuevo == NULL) {
-		fprintf(stderr, "ARCHIVO NUEVO NO CREADO\n");
+		PERR("ARCHIVO NUEVO NO CREADO");
 		return;
 	}
 
@@ -509,14 +531,15 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 	lst_nueva->fp = nuevo;
 
 	/* Leo los indices del archivo viejo */
+	PERR("Obtengo Indices\n");
 	indices = emufs_idx_get(old, &indices_total);
 	if (indices == NULL) {
 		art_liberar(lst_nueva);
 		return;
 	}
 
+	PERR("Proceso datos\n");
 	for(i=0; i<indices_total; i++) {
-		fprintf(stderr, "A procesar %lu de %lu\n",i, indices_total);
 		error = 0;
 		save = old->leer_registro(old, indices[i], &size, &error);
 		if (procesar_leer_articulo(&art, save, size, lst_articulos) == 1) {
@@ -532,18 +555,26 @@ void art_reformatear(int tipo, int tam_bloque, int tam_reg)
 		}
 	}
 
-	ver_archivo_FS(old);
-	ver_archivo_FS(nuevo);
+	free(indices);
 
+	PERR("Libero lo viejo\n");
 	art_liberar(lst_articulos);
+
+	PERR("Ahora tengo lo nuevo\n");
 	lst_articulos = lst_nueva;
 
-	fprintf(stderr, "Listo. Renombre y me voy\n");
+	/* El nuevo tiene como nombre emufs_tmp, lo cambio a mano! */
+	free(lst_articulos->fp->nombre);
+	lst_articulos->fp->nombre = (char *)malloc(sizeof(char)*(strlen("articulos")+1));
+	strcpy(lst_articulos->fp->nombre, "articulos");
+	
 	/* Muevo los archivos! */
 	/* TODO : Poner en otro lugar mas generico! */
+	PERR("Renombre!!\n");
 	rename("emufs_tmp.dat", "articulos.dat");
 	rename("emufs_tmp.idx", "articulos.idx");
 	rename("emufs_tmp.fsc", "articulos.fsc");
 	rename("emufs_tmp.did", "articulos.did");
+	PERR("==== TERMINE ====\n");
 }