* Valgrind rulez :-)

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

diff --git a/emufs_gui/facturas.c b/emufs_gui/facturas.c
index f1b60aafc8f2e43521ace8bf9c922708603783fe..7a594d385658265c96b10d3d03ed97160091684f 100644 (file)
--- a/emufs_gui/facturas.c
+++ b/emufs_gui/facturas.c
@@ -22,12 +22,13 @@ t_LstFacturas *fact_get_lst()
 char *xml_get_prop(xmlNode *node, char *nombre)
 {
        char *s;
 char *xml_get_prop(xmlNode *node, char *nombre)
 {
        char *s;

-       if (xmlGetProp(node, nombre) == NULL) {
+       s = xmlGetProp(node, nombre);
+       if (s == NULL) {

                s = malloc(1);
                s[0] = '\0';
                return s;
        }
                s = malloc(1);
                s[0] = '\0';
                return s;
        }

-       return xmlGetProp(node, nombre);
+       return s;

 }
 
 int eliminar_nodo_factura(t_LstFacturas *lst, t_Reg_Factura *nodo)
 }
 
 int eliminar_nodo_factura(t_LstFacturas *lst, t_Reg_Factura *nodo)


@@ -83,8 +84,25 @@ 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], 0, 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);


@@ -157,6 +175,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                PERR("Voy a cargar de un XML");
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {
                PERR("Voy a cargar de un XML");
                document = xmlReadFile(filename, "ISO-8859-1",0);
                if (document == NULL) {

+                       PERR("Error al leer documento!!");

                        free(tmp);
                        lst_facturas = NULL;
                        return NULL;
                        free(tmp);
                        lst_facturas = NULL;
                        return NULL;


@@ -178,7 +197,7 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                /* En el registro no guardo los punteros de nota ni items. Si guardo la cantidad de items
                 * y los items al final del registro.
                 */
                /* En el registro no guardo los punteros de nota ni items. Si guardo la cantidad de items
                 * y los items al final del registro.
                 */

-               if (tipo == 3) {
+               if (tipo == T3) {

                        /* Limito a 10 items en el caso de registro constante! */
                        cant_items = 10;
                } else {
                        /* Limito a 10 items en el caso de registro constante! */
                        cant_items = 10;
                } else {


@@ -198,15 +217,21 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
                                        fact.procdoi = atof(prop); xmlFree(prop);
                                        prop = xml_get_prop(node, "NroRemito");
                                        fact.numero_remito = atoi(prop); xmlFree(prop);
                                        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);
+                                       strncpy(fact.emision, prop = xml_get_prop(node, "FechaEmisión"), 8); xmlFree(prop);
+                                       fact.emision[8] = '\0';
+                                       strncpy(fact.vencimiento, prop = xml_get_prop(node, "FechaVto"), 8); xmlFree(prop);
+                                       fact.vencimiento[8] = '\0';
+                                       strncpy(fact.estado, prop = xml_get_prop(node, "Estado"), 2); xmlFree(prop);
+                                       fact.estado[2] = '\0';
+                                       strncpy(fact.fp, prop = xml_get_prop(node, "FP"), 2); xmlFree(prop);
+                                       fact.fp[2] = '\0';
+                                       strncpy(fact.ctacte, prop = xml_get_prop(node, "NroCtaCte"), 5); xmlFree(prop);
+                                       fact.ctacte[5] = '\0';
+                                       strncpy(fact.cheque, prop = xml_get_prop(node, "NroCheque"), 18); xmlFree(prop);
+                                       fact.cheque[18] = '\0';

 
                                        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);


@@ -264,10 +289,17 @@ t_LstFacturas *fact_cargar(const char *filename, int tipo, int tam_bloque)
 
 int fact_liberar(t_LstFacturas *l)
 {
 
 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);

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

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


@@ -449,8 +481,8 @@ void fact_modificar(char *s)
 
 void fact_agregar(char *s)
 {
 
 void fact_agregar(char *s)
 {

-       WINDOW *win, *items;
-       t_Form *form;
+       WINDOW *win, *items, *nota, *subnota;
+       t_Form *form, *form_nota;

        t_Item *its = NULL;
        t_Factura fact;
        EMUFS_REG_SIZE size;
        t_Item *its = NULL;
        t_Factura fact;
        EMUFS_REG_SIZE size;


@@ -466,6 +498,11 @@ void fact_agregar(char *s)
        wrefresh(win);
 
        items = derwin(win, LINES-20, COLS-4, 15, 1);
        wrefresh(win);
 
        items = derwin(win, LINES-20, COLS-4, 15, 1);

+       nota = derwin(win, 9, COLS-62, 1, 56);
+       subnota = derwin(nota, 7, COLS-64, 1, 1);
+       box(nota, 0, 0);
+       mvwaddstr(nota, 0, 1, "Nota :");
+       wrefresh(nota);

        wrefresh(items);
 
        form = form_crear(win);
        wrefresh(items);
 
        form = form_crear(win);


@@ -481,8 +518,11 @@ void fact_agregar(char *s)
 
        form_ejecutar(form, 1,1);
 
 
        form_ejecutar(form, 1,1);
 

+       form_nota = form_crear(subnota);
+       form_agregar_widget(form_nota, INPUT, "", 255, "");
+       form_ejecutar(form_nota, 0, 0);
+

        fact.numero = form_obtener_valor_int(form, "Numero de Factura");
        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");


@@ -493,6 +533,7 @@ void fact_agregar(char *s)
        strcpy(fact.cheque, form_obtener_valor_char(form, "Cheque Nro"));
 
        form_destruir(form);
        strcpy(fact.cheque, form_obtener_valor_char(form, "Cheque Nro"));
 
        form_destruir(form);

+       form_destruir(form_nota);

 
        form = form_crear(win);
        form_agregar_widget(form, INPUT, "Nro de Articulo (* == fin)", 8, "");
 
        form = form_crear(win);
        form_agregar_widget(form, INPUT, "Nro de Articulo (* == fin)", 8, "");


@@ -503,6 +544,7 @@ void fact_agregar(char *s)
        mvwaddstr(win, 15, 2, "Numero");
        mvwaddstr(win, 15, 11, "CV");
        mvwaddstr(win, 15, 21, "PVU");
        mvwaddstr(win, 15, 2, "Numero");
        mvwaddstr(win, 15, 11, "CV");
        mvwaddstr(win, 15, 21, "PVU");

+       cant = 0;

        do {
                form_set_valor(form, "Nro de Articulo (* == fin)", "");
                form_set_valor(form, "CV", "");
        do {
                form_set_valor(form, "Nro de Articulo (* == fin)", "");
                form_set_valor(form, "CV", "");


@@ -511,7 +553,7 @@ void fact_agregar(char *s)
 
                entrada = form_obtener_valor_char(form, "Nro de Articulo (* == fin)");
 
 
                entrada = form_obtener_valor_char(form, "Nro de Articulo (* == fin)");
 

-               if (entrada[0] != '\0') {
+               if ((entrada[0] != '\0') && (entrada[0] != '*')){

                        y_actual++;
                        if (y_actual > LINES-22) {
                                y_actual = LINES-22;
                        y_actual++;
                        if (y_actual > LINES-22) {
                                y_actual = LINES-22;


@@ -524,17 +566,24 @@ void fact_agregar(char *s)
                        /* Agrego el Item */
                        cant++;
                        its = (t_Item *)realloc(its, cant*sizeof(t_Item));
                        /* Agrego el Item */
                        cant++;
                        its = (t_Item *)realloc(its, cant*sizeof(t_Item));

-                       its[cant-1].numero = form_obtener_valor_int(form, entrada);
-                       strcpy(its[cant-1].cv, form_obtener_valor_char(form, "CV"));
-                       strcpy(its[cant-1].pvu, form_obtener_valor_char(form, "PVU"));
+                       if (its != NULL) {
+                               its[cant-1].numero = atoi(entrada);
+                               strcpy(its[cant-1].cv, form_obtener_valor_char(form, "CV"));
+                               strcpy(its[cant-1].pvu, form_obtener_valor_char(form, "PVU"));
+                       }

                }
                }

-       } while (strcmp(entrada, "*") != 0);
+       } while (entrada[0] != '*');

 
 

-       if (lst_facturas->fp->tipo == 3) {
+       if (lst_facturas->fp->tipo == T3) {

                if (cant != 10) {
                        /* TODO Limitar en la GUI en lugar de truncar! */
                        its = (t_Item *)realloc(its, 10*sizeof(t_Item));
                if (cant != 10) {
                        /* TODO Limitar en la GUI en lugar de truncar! */
                        its = (t_Item *)realloc(its, 10*sizeof(t_Item));

-                       cant = 10;
+                       if (its == NULL) {
+                               cant = 0;
+                       } else {
+                               memset(its+sizeof(t_Item)*cant, 0, (10-cant)*sizeof(t_Item));
+                               cant = 10;
+                       }

                }
        }
        fact.items = its;
                }
        }
        fact.items = its;


@@ -556,6 +605,8 @@ void fact_agregar(char *s)
        werase(win);
        wrefresh(win);
        delwin(items);
        werase(win);
        wrefresh(win);
        delwin(items);

+       delwin(subnota);
+       delwin(nota);

        delwin(win);
 }
 
        delwin(win);
 }
 


@@ -586,17 +637,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);