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 469d97b7f397cc5e62527a30412fe55a04b8371c..00b5276371093b8394cb20887f044b4a12253d8c 100644 (file)
--- a/emufs_gui/facturas.c
+++ b/emufs_gui/facturas.c
@@ -83,18 +83,36 @@ t_Item *leer_items(xmlNode *node, int *cant, int size)
        int count;
        char *prop;
        if (size == -1) {
        int count;
        char *prop;
        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) {

+                                       memset(&tmp[count], '*', sizeof(t_Item));

                                        prop = xml_get_prop(node, "NroArtículo");
                                        tmp[count].numero = atoi(prop);
                                        xmlFree(prop);
                                        prop = xml_get_prop(node, "NroArtículo");
                                        tmp[count].numero = atoi(prop);
                                        xmlFree(prop);


@@ -103,6 +121,7 @@ t_Item *leer_items(xmlNode *node, int *cant, int size)
                                        count++;
                                }
                        }
                                        count++;
                                }
                        }

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

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


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


@@ -188,7 +208,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                                if (strcmp(node->name, "FACTURA") == 0) {
                                        t_Factura fact;
                                        void *save;
                                if (strcmp(node->name, "FACTURA") == 0) {
                                        t_Factura fact;
                                        void *save;

-                                       memset(&fact, 0, sizeof(t_Factura));
+                                       memset(&fact, '*', sizeof(t_Factura));

                                        prop = xml_get_prop(node, "NroFac");
                                        fact.numero = atoi(prop); xmlFree(prop);
                                        prop = xml_get_prop(node, "PorcDoI");
                                        prop = xml_get_prop(node, "NroFac");
                                        fact.numero = atoi(prop); xmlFree(prop);
                                        prop = xml_get_prop(node, "PorcDoI");


@@ -203,7 +223,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                                        strcpy(fact.cheque, prop = xml_get_prop(node, "NroCheque")); xmlFree(prop);
 
                                        fact.nota = leer_nota(node);
                                        strcpy(fact.cheque, prop = xml_get_prop(node, "NroCheque")); xmlFree(prop);
 
                                        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);


@@ -223,8 +243,22 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {
                xmlFreeDoc(document);
                xmlCleanupParser();
        } else {

+               PERR("Voy a recuperar desde un archivo");

                tmp->fp = emufs_abrir("facturas");
                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");
                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 */
                indices = emufs_idx_get(tmp->fp, &indices_cant);
 
                /* Ahora trato de recuperar la info */
                indices = emufs_idx_get(tmp->fp, &indices_cant);


@@ -240,15 +274,23 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                }
                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;


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


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