Un poco mas de limpieza usando los codigos de error.

[z.facultad/75.06/emufs.git] / emufs_gui / facturas.c

diff --git a/emufs_gui/facturas.c b/emufs_gui/facturas.c
index f940890f9ffb842eb0cc62a7c8ea3361a22a7d79..00b5276371093b8394cb20887f044b4a12253d8c 100644 (file)
--- a/emufs_gui/facturas.c
+++ b/emufs_gui/facturas.c
@@ -1,5 +1,6 @@
 
 #include "facturas.h"
 
 #include "facturas.h"

+#include "idx.h"

 
 static t_LstFacturas *lst_facturas;
 
 
 static t_LstFacturas *lst_facturas;
 


@@ -20,7 +21,12 @@ t_LstFacturas *fact_get_lst()
 /* Hack! ... Si no existe propiedad retorna "" */
 char *xml_get_prop(xmlNode *node, char *nombre)
 {
 /* Hack! ... Si no existe propiedad retorna "" */
 char *xml_get_prop(xmlNode *node, char *nombre)
 {

-       if (xmlGetProp(node, nombre) == NULL) return "";
+       char *s;
+       if (xmlGetProp(node, nombre) == NULL) {
+               s = malloc(1);
+               s[0] = '\0';
+               return s;
+       }

        return xmlGetProp(node, nombre);
 }
 
        return xmlGetProp(node, nombre);
 }
 


@@ -75,25 +81,47 @@ t_Item *leer_items(xmlNode *node, int *cant, int size)
 {
        t_Item *tmp;
        int count;
 {
        t_Item *tmp;
        int count;

+       char *prop;

        if (size == -1) {
        if (size == -1) {

-               *cant = 0;
-               return NULL;
+               tmp = NULL;
+               count = 0;
+               node = node->children;
+               while (node) {
+                       if (node->type == XML_ELEMENT_NODE) {
+                               if (strcmp(node->name, "ITEMVENTA") == 0) {
+                                       count++;
+                                       tmp = realloc(tmp, sizeof(t_Item)*count);
+                                       memset(&tmp[count-1], '*', sizeof(t_Item));
+                                       prop = xml_get_prop(node, "NroArtículo");
+                                       tmp[count-1].numero = atoi(prop);
+                                       xmlFree(prop);
+                                       strcpy(tmp[count-1].cv, prop = xml_get_prop(node, "CV")); xmlFree(prop);
+                                       strcpy(tmp[count-1].pvu, prop = xml_get_prop(node, "PVU")); xmlFree(prop);
+                               }
+                       }
+                       node = node->next;
+               }
+               *cant = count;

        } else {
                (*cant) = size;
                tmp = (t_Item *)malloc(sizeof(t_Item)*size);
        } else {
                (*cant) = size;
                tmp = (t_Item *)malloc(sizeof(t_Item)*size);

-               memset(tmp, '$', sizeof(t_Item)*size);
+               memset(tmp, 0, sizeof(t_Item)*size);

 
                count = 0;
                node = node->children;
                while (node) {
                        if (node->type == XML_ELEMENT_NODE) {
                                if (strcmp(node->name, "ITEMVENTA") == 0) {
 
                count = 0;
                node = node->children;
                while (node) {
                        if (node->type == XML_ELEMENT_NODE) {
                                if (strcmp(node->name, "ITEMVENTA") == 0) {

-                                       tmp[count].numero = atoi(xml_get_prop(node, "NroArtículo"));
-                                       strcpy(tmp[count].cv, xml_get_prop(node, "CV"));
-                                       strcpy(tmp[count].pvu, xml_get_prop(node, "PVU"));
+                                       memset(&tmp[count], '*', sizeof(t_Item));
+                                       prop = xml_get_prop(node, "NroArtículo");
+                                       tmp[count].numero = atoi(prop);
+                                       xmlFree(prop);
+                                       strcpy(tmp[count].cv, prop = xml_get_prop(node, "CV")); xmlFree(prop);
+                                       strcpy(tmp[count].pvu, prop = xml_get_prop(node, "PVU")); xmlFree(prop);

                                        count++;
                                }
                        }
                                        count++;
                                }
                        }

+                       if (count == 10) break; /* No me entran mas items! */

                        node = node->next;
                }
        }
                        node = node->next;
                }
        }


@@ -129,10 +157,11 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
 {
        xmlDocPtr document;
        xmlNode *node, *inicio;
 {
        xmlDocPtr document;
        xmlNode *node, *inicio;

-       int error = 0, cant_items;
+       int error = 0, cant_items, i;
+       char *prop;

        EMUFS_REG_SIZE size;
        t_LstFacturas *tmp;
        EMUFS_REG_SIZE size;
        t_LstFacturas *tmp;

-       EMUFS_REG_ID id; /*, *indices, indices_cant;*/
+       EMUFS_REG_ID id, *indices, indices_cant;

        
        lst_facturas = NULL;
 
        
        lst_facturas = NULL;
 


@@ -142,6 +171,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
        tmp->primero = NULL;
 
        if (filename != NULL) {
        tmp->primero = NULL;
 
        if (filename != NULL) {

+               PERR("Voy a cargar de un XML");

                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                        free(tmp);
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                        free(tmp);


@@ -179,18 +209,21 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                                        t_Factura fact;
                                        void *save;
                                        memset(&fact, '*', sizeof(t_Factura));
                                        t_Factura fact;
                                        void *save;
                                        memset(&fact, '*', sizeof(t_Factura));

-                                       fact.numero = atoi(xml_get_prop(node, "NroFac"));
-                                       fact.procdoi = atof(xml_get_prop(node, "PorcDoI"));
-                                       fact.numero_remito = atoi(xml_get_prop(node, "NroRemito"));
-                                       strcpy(fact.emision, xml_get_prop(node, "FechaEmisión"));
-                                       strcpy(fact.vencimiento, xml_get_prop(node, "FechaVto"));
-                                       strcpy(fact.estado, xml_get_prop(node, "Estado"));
-                                       strcpy(fact.fp, xml_get_prop(node, "FP"));
-                                       strcpy(fact.ctacte, xml_get_prop(node, "NroCtaCte"));
-                                       strcpy(fact.cheque, xml_get_prop(node, "NroCheque"));
+                                       prop = xml_get_prop(node, "NroFac");
+                                       fact.numero = atoi(prop); xmlFree(prop);
+                                       prop = xml_get_prop(node, "PorcDoI");
+                                       fact.procdoi = atof(prop); xmlFree(prop);
+                                       prop = xml_get_prop(node, "NroRemito");
+                                       fact.numero_remito = atoi(prop); xmlFree(prop);
+                                       strcpy(fact.emision, prop = xml_get_prop(node, "FechaEmisión")); xmlFree(prop);
+                                       strcpy(fact.vencimiento, prop = xml_get_prop(node, "FechaVto")); xmlFree(prop);
+                                       strcpy(fact.estado, prop = xml_get_prop(node, "Estado")); xmlFree(prop);
+                                       strcpy(fact.fp, prop = xml_get_prop(node, "FP")); xmlFree(prop);
+                                       strcpy(fact.ctacte, prop = xml_get_prop(node, "NroCtaCte")); xmlFree(prop);
+                                       strcpy(fact.cheque, prop = xml_get_prop(node, "NroCheque")); xmlFree(prop);

 
                                        fact.nota = leer_nota(node);
 
                                        fact.nota = leer_nota(node);

-                                       fact.items = leer_items(node, &fact.cant_items, (tipo==3)?10:-1);
+                                       fact.items = leer_items(node, &fact.cant_items, ((tipo-1)==T3)?10:-1);

 
                                        error = 0;
                                        id = tmp->fp_texto->grabar_registro(tmp->fp_texto, fact.nota, strlen(fact.nota)+1, &error);
 
                                        error = 0;
                                        id = tmp->fp_texto->grabar_registro(tmp->fp_texto, fact.nota, strlen(fact.nota)+1, &error);


@@ -210,30 +243,54 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {

-/*             tmp->fp = emufs_abrir("articulos");*/
+               PERR("Voy a recuperar desde un archivo");
+               tmp->fp = emufs_abrir("facturas");
+               if (tmp->fp == NULL) {
+                       PERR("No se pudo cargar archivo de facturas!");
+                       free(tmp);
+                       lst_facturas = NULL;
+                       return NULL;
+               }
+               tmp->fp_texto = emufs_abrir("notas");
+               if (tmp->fp_texto == NULL) {
+                       PERR("No se pudo cargar archivo de notas!");
+                       emufs_destruir(tmp->fp);
+                       free(tmp);
+                       lst_facturas = NULL;
+                       return NULL;
+               }
+

                /* Ahora trato de recuperar la info */
                /* Ahora trato de recuperar la info */

-/*             indices = emufs_idx_get(tmp->fp, &indices_cant);
+               indices = emufs_idx_get(tmp->fp, &indices_cant);

                for(i=0; i<indices_cant; i++) {
                for(i=0; i<indices_cant; i++) {

-                       t_Articulo art;
-                       void *save;*/
+                       t_Factura art;
+                       void *save;

                        /* Leo el registro */
                        /* Leo el registro */

-/*                     save = tmp->fp->leer_registro(tmp->fp, indices[i], &size, &error);
-                       if (procesar_leer_articulo(&art, save, size, tmp) == 1) {
-                               agregar_nodo_articulo(tmp, crear_nodo_articulo(indices[i], art.numero));
+                       save = tmp->fp->leer_registro(tmp->fp, indices[i], &size, &error);
+                       if (procesar_leer_factura(&art, save, size, tmp) == 1) {
+                               agregar_nodo_factura(tmp, crear_nodo_factura(indices[i], art.reg_nota, art.numero));

                                free(save);
                        }
                }
                                free(save);
                        }
                }

-               free(indices);*/
+               free(indices);

        }
        }

+
+       PERR("Facturas todo Ok");

        return lst_facturas;
 }
 
 int fact_liberar(t_LstFacturas *l)
 {
        return lst_facturas;
 }
 
 int fact_liberar(t_LstFacturas *l)
 {

+       t_Reg_Factura *del;

        if (l == NULL) l = lst_facturas;
        if (l == NULL) return 1;
 
        emufs_destruir(l->fp);
        if (l == NULL) l = lst_facturas;
        if (l == NULL) return 1;
 
        emufs_destruir(l->fp);

+       while (l->primero) {
+               del = l->primero;
+               l->primero = l->primero->sig;
+               free(del);
+       }

        free(l);
 
        lst_facturas = NULL;
        free(l);
 
        lst_facturas = NULL;


@@ -448,7 +505,6 @@ void fact_agregar(char *s)
        form_ejecutar(form, 1,1);
 
        fact.numero = form_obtener_valor_int(form, "Numero de Factura");
        form_ejecutar(form, 1,1);
 
        fact.numero = form_obtener_valor_int(form, "Numero de Factura");

-       fprintf(stderr, "Agregando numero %d\n", fact.numero);

        strcpy(fact.emision, form_obtener_valor_char(form, "Fecha Emision"));
        strcpy(fact.vencimiento, form_obtener_valor_char(form, "Fecha Vto"));
        fact.numero_remito = form_obtener_valor_int(form, "Nro Remito");
        strcpy(fact.emision, form_obtener_valor_char(form, "Fecha Emision"));
        strcpy(fact.vencimiento, form_obtener_valor_char(form, "Fecha Vto"));
        fact.numero_remito = form_obtener_valor_int(form, "Nro Remito");


@@ -552,17 +608,18 @@ void *procesar_guardar_factura(t_Factura *f, t_LstFacturas *lst, EMUFS_REG_SIZE
                        memset(tmp, 0, *size);
                        /* Ahora copio la info */
                        memcpy(tmp, &f->numero, i[0]);
                        memset(tmp, 0, *size);
                        /* Ahora copio la info */
                        memcpy(tmp, &f->numero, i[0]);

-                       memcpy(tmp, &f->procdoi, i[1]);
-                       memcpy(tmp, &f->numero_remito, i[2]);
-                       memcpy(tmp, &f->cant_items, i[3]);
-                       memcpy(tmp, &f->reg_nota, i[4]);
+                       memcpy(tmp+i[0], &f->procdoi, i[1]);
+                       memcpy(tmp+i[0]+i[1], &f->numero_remito, i[2]);
+                       memcpy(tmp+i[0]+i[1]+i[2], &f->cant_items, i[3]);
+                       memcpy(tmp+i[0]+i[1]+i[2]+i[3], &f->reg_nota, i[4]);

                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], f->emision, i[5]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5], f->vencimiento, i[6]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6], f->estado, i[7]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7], f->fp, i[8]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8], f->ctacte, i[9]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9], f->cheque, i[10]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], f->emision, i[5]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5], f->vencimiento, i[6]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6], f->estado, i[7]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7], f->fp, i[8]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8], f->ctacte, i[9]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9], f->cheque, i[10]);

-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9]+i[10], f->items, i[11]);
+                       if (i[11] != 0)
+                               memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9]+i[10], f->items, i[11]);

                break;
                case T3:
                        (*size) = sizeof(t_Factura)-sizeof(char *)-sizeof(t_Item *) + f->cant_items*sizeof(t_Item);
                break;
                case T3:
                        (*size) = sizeof(t_Factura)-sizeof(char *)-sizeof(t_Item *) + f->cant_items*sizeof(t_Item);