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 9e791cf96f0283e3fd106d3ba4b14c077a3595b6..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, id_texto; /*, *indices, indices_cant;*/
+       EMUFS_REG_ID id, *indices, indices_cant;

        
        lst_facturas = NULL;
 
        
        lst_facturas = NULL;
 


@@ -142,8 +171,11 @@ 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) {
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {

+                       free(tmp);
+                       lst_facturas = NULL;

                        return NULL;
                }
 
                        return NULL;
                }
 


@@ -177,26 +209,30 @@ 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);
+                                       fact.reg_nota = id;

                                        save = procesar_guardar_factura(&fact, lst_facturas, &size);
                                        if (save != NULL) {
                                                error = 0;
                                                id = tmp->fp->grabar_registro(tmp->fp, save, size, &error);
                                        save = procesar_guardar_factura(&fact, lst_facturas, &size);
                                        if (save != NULL) {
                                                error = 0;
                                                id = tmp->fp->grabar_registro(tmp->fp, save, size, &error);

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

                                                if (fact.items) free(fact.items);
                                                if (fact.nota) free(fact.nota);
                                                free(save);
                                                if (fact.items) free(fact.items);
                                                if (fact.nota) free(fact.nota);
                                                free(save);


@@ -207,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;


@@ -251,12 +311,14 @@ t_Factura *fact_buscar(t_LstFacturas *lst, int numero, EMUFS_REG_ID *id, EMUFS_R
        while (reg) {
                if (reg->numero == numero) {
                        size = 0;
        while (reg) {
                if (reg->numero == numero) {
                        size = 0;

-                       fprintf(stderr, "Leer me dice que %lu\n", size);

                        error = 0;
                        leo = lst->fp->leer_registro(lst->fp, reg->num_reg, &size, &error);
                        error = 0;
                        leo = lst->fp->leer_registro(lst->fp, reg->num_reg, &size, &error);

-                       fprintf(stderr, "Leer me dice que %lu\n", size);

                        if (leo != NULL) {
                                fact = (t_Factura *)malloc(sizeof(t_Factura));
                        if (leo != NULL) {
                                fact = (t_Factura *)malloc(sizeof(t_Factura));

+                               if (fact == NULL) {
+                                       free(leo);
+                                       return NULL;
+                               }

                                procesar_leer_factura(fact, leo, size, lst);
                                (*id) = reg->num_reg;
                                (*id_texto) = reg->texto_reg;
                                procesar_leer_factura(fact, leo, size, lst);
                                (*id) = reg->num_reg;
                                (*id_texto) = reg->texto_reg;


@@ -415,7 +477,7 @@ void fact_agregar(char *s)
        t_Item *its = NULL;
        t_Factura fact;
        EMUFS_REG_SIZE size;
        t_Item *its = NULL;
        t_Factura fact;
        EMUFS_REG_SIZE size;

-       EMUFS_REG_ID id, id_texto;
+       EMUFS_REG_ID id; /*, id_texto;*/

        int y_actual, cant, error;
        char *entrada;
 
        int y_actual, cant, error;
        char *entrada;
 


@@ -443,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");


@@ -523,7 +584,7 @@ void fact_agregar(char *s)
 void *procesar_guardar_factura(t_Factura *f, t_LstFacturas *lst, EMUFS_REG_SIZE *size)
 {
        char *tmp=NULL;
 void *procesar_guardar_factura(t_Factura *f, t_LstFacturas *lst, EMUFS_REG_SIZE *size)
 {
        char *tmp=NULL;

-       int i[11];
+       int i[12];

 
        switch (lst->fp->tipo) {
                case T1:
 
        switch (lst->fp->tipo) {
                case T1:


@@ -533,29 +594,32 @@ void *procesar_guardar_factura(t_Factura *f, t_LstFacturas *lst, EMUFS_REG_SIZE
                        i[1] = sizeof(float);
                        i[2] = sizeof(int);
                        i[3] = sizeof(int);
                        i[1] = sizeof(float);
                        i[2] = sizeof(int);
                        i[3] = sizeof(int);

-                       i[4] = sizeof(char)*(strlen(f->emision)+1); /* +1 por el \0 para separar */
-                       i[5] = sizeof(char)*(strlen(f->vencimiento)+1); /* +1 por el \0 para separar */
-                       i[6] = sizeof(char)*(strlen(f->estado)+1); /* +1 por el \0 para separar */
-                       i[7] = sizeof(char)*(strlen(f->fp)+1); /* +1 por el \0 para separar */
-                       i[8] = sizeof(char)*(strlen(f->ctacte)+1); /* +1 por el \0 para separar */
-                       i[9] = sizeof(char)*(strlen(f->cheque)+1); /* +1 por el \0 para separar */
-                       i[10] = sizeof(t_Item)*f->cant_items;
-                       (*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9]+i[10];
+                       i[4] = sizeof(EMUFS_BLOCK_ID);
+                       i[5] = sizeof(char)*(strlen(f->emision)+1); /* +1 por el \0 para separar */
+                       i[6] = sizeof(char)*(strlen(f->vencimiento)+1); /* +1 por el \0 para separar */
+                       i[7] = sizeof(char)*(strlen(f->estado)+1); /* +1 por el \0 para separar */
+                       i[8] = sizeof(char)*(strlen(f->fp)+1); /* +1 por el \0 para separar */
+                       i[9] = sizeof(char)*(strlen(f->ctacte)+1); /* +1 por el \0 para separar */
+                       i[10] = sizeof(char)*(strlen(f->cheque)+1); /* +1 por el \0 para separar */
+                       i[11] = sizeof(t_Item)*f->cant_items;
+                       (*size) = i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8]+i[9]+i[10]+i[11];

                        tmp = (char *)malloc(*size);
                        if (tmp == NULL) return NULL;
                        memset(tmp, 0, *size);
                        /* Ahora copio la info */
                        memcpy(tmp, &f->numero, i[0]);
                        tmp = (char *)malloc(*size);
                        if (tmp == NULL) return NULL;
                        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+i[0]+i[1]+i[2]+i[3], f->emision, i[4]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], f->vencimiento, i[5]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5], f->estado, i[6]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6], f->fp, i[7]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7], f->ctacte, i[8]);
-                       memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4]+i[5]+i[6]+i[7]+i[8], f->cheque, 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->items, i[10]);
+                       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]);
+                       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);


@@ -570,6 +634,7 @@ void *procesar_guardar_factura(t_Factura *f, t_LstFacturas *lst, EMUFS_REG_SIZE
 static int procesar_leer_factura(t_Factura *dst, void *src, EMUFS_REG_SIZE size, t_LstFacturas *lst)
 {
        char *ini, *fin;
 static int procesar_leer_factura(t_Factura *dst, void *src, EMUFS_REG_SIZE size, t_LstFacturas *lst)
 {
        char *ini, *fin;

+       int dummy;

 
        switch (lst->fp->tipo) {
                case T1:
 
        switch (lst->fp->tipo) {
                case T1:


@@ -587,6 +652,9 @@ static int procesar_leer_factura(t_Factura *dst, void *src, EMUFS_REG_SIZE size,
                        
                        memcpy(&dst->cant_items, ini, sizeof(int));
                        ini+=sizeof(int);
                        
                        memcpy(&dst->cant_items, ini, sizeof(int));
                        ini+=sizeof(int);

+                       
+                       memcpy(&dst->reg_nota, ini, sizeof(EMUFS_BLOCK_ID));
+                       ini+=sizeof(EMUFS_BLOCK_ID);

 
                        /* Ahora empieza el juego */
                        /* Los \0 son los delimitadores de campo! */
 
                        /* Ahora empieza el juego */
                        /* Los \0 son los delimitadores de campo! */


@@ -624,9 +692,10 @@ static int procesar_leer_factura(t_Factura *dst, void *src, EMUFS_REG_SIZE size,
                                dst->items = (t_Item *)malloc(sizeof(t_Item)*dst->cant_items);
 
                                ini = fin+1;
                                dst->items = (t_Item *)malloc(sizeof(t_Item)*dst->cant_items);
 
                                ini = fin+1;

-                               fprintf(stderr, "trabajo con 1 -> %lu\n", size);

                                fin = (char *)src+size;
                                memcpy(dst->items, ini, fin-ini);
                                fin = (char *)src+size;
                                memcpy(dst->items, ini, fin-ini);

+
+                               dst->nota = lst->fp_texto->leer_registro(lst->fp_texto, dst->reg_nota, (EMUFS_REG_SIZE *)&dummy, &dummy);

                        } else {
                                dst->items = NULL;
                        }
                        } else {
                                dst->items = NULL;
                        }