* 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 b9b798a0be1c80167b00d6854606c38372f149f1..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)


@@ -91,7 +92,7 @@ t_Item *leer_items(xmlNode *node, int *cant, int size)
                                if (strcmp(node->name, "ITEMVENTA") == 0) {
                                        count++;
                                        tmp = realloc(tmp, sizeof(t_Item)*count);
                                if (strcmp(node->name, "ITEMVENTA") == 0) {
                                        count++;
                                        tmp = realloc(tmp, sizeof(t_Item)*count);

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

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


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


@@ -195,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 {


@@ -215,12 +217,18 @@ 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.items = leer_items(node, &fact.cant_items, ((tipo-1)==T3)?10:-1);
 
                                        fact.nota = leer_nota(node);
                                        fact.items = leer_items(node, &fact.cant_items, ((tipo-1)==T3)?10:-1);


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


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


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


@@ -498,6 +518,10 @@ 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");
        strcpy(fact.emision, form_obtener_valor_char(form, "Fecha Emision"));
        strcpy(fact.vencimiento, form_obtener_valor_char(form, "Fecha Vto"));
        fact.numero = form_obtener_valor_int(form, "Numero de Factura");
        strcpy(fact.emision, form_obtener_valor_char(form, "Fecha Emision"));
        strcpy(fact.vencimiento, form_obtener_valor_char(form, "Fecha Vto"));


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


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


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


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


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