* BUGFIX : Estaba siendo olvidado verificar de no sobrepasar

 el limite de los campos impuesto en el enunciado.

diff --git a/emufs_gui/articulos.c b/emufs_gui/articulos.c
index 598a888427b6a30d82dbc57b906acd269837b219..2e3602a47ba59bdec5fa1d4f90150ba35224c684 100644 (file)
--- a/emufs_gui/articulos.c
+++ b/emufs_gui/articulos.c
@@ -113,12 +113,17 @@ t_LstArticulos *art_cargar(const char *filename, int tipo, int tam_bloque)
                                        prop = xmlGetProp(node, "NroArtículo");
                                        art.numero = atoi(prop);
                                        xmlFree(prop);
                                        prop = xmlGetProp(node, "NroArtículo");
                                        art.numero = atoi(prop);
                                        xmlFree(prop);

-                                       strcpy(art.desc, prop = xmlGetProp(node, "Descripción")); xmlFree(prop);
-                                       strcpy(art.presentacion, prop = xmlGetProp(node, "Presentación")); xmlFree(prop);
-                                       strcpy(art.existencia, prop = xmlGetProp(node, "Existencia")); xmlFree(prop);
+                                       strncpy(art.desc, prop = xmlGetProp(node, "Descripción"), 50); xmlFree(prop);
+                                       art.desc[50] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.presentacion, prop = xmlGetProp(node, "Presentación"), 30); xmlFree(prop);
+                                       art.presentacion[30] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.existencia, prop = xmlGetProp(node, "Existencia"), 8); xmlFree(prop);
+                                       art.existencia[8] = '\0'; /* Me aseguro de que este */

                                        /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/
                                        /*strncpy(tmp->array[cant].ubicacion, xmlGetProp(node, "Ubicacion"), 30);*/

-                                       strcpy(art.pvu, prop = xmlGetProp(node, "PVU")); xmlFree(prop);
-                                       strcpy(art.emin, prop = xmlGetProp(node, "Emín")); xmlFree(prop);
+                                       strncpy(art.pvu, prop = xmlGetProp(node, "PVU"), 8); xmlFree(prop);
+                                       art.pvu[8] = '\0'; /* Me aseguro de que este */
+                                       strncpy(art.emin, prop = xmlGetProp(node, "Emín"), 8); xmlFree(prop);
+                                       art.emin[8] = '\0'; /* Me aseguro de que este */

                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {
                                        /* Ya leido el articulo ahora paso a guardarlo en el archivo y agregarlo a la lista */
                                        save = procesar_guardar_articulo(&art, &size, lst_articulos);
                                        if (save != NULL) {


@@ -443,6 +448,7 @@ int procesar_leer_articulo(t_Articulo *dst, void *src, EMUFS_REG_SIZE size, t_Ls
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;
                        ini = fin+1;
                        fin = ini;
                        while (*fin!='\0') fin++;

+                       fprintf(stderr, "Presentacion = %s\n", dst->presentacion);

                        memcpy(dst->existencia, ini, fin-ini+1);
                        
                        ini = fin+1;
                        memcpy(dst->existencia, ini, fin-ini+1);
                        
                        ini = fin+1;


@@ -485,6 +491,7 @@ void *procesar_guardar_articulo(t_Articulo *src, EMUFS_REG_SIZE *size, t_LstArti
                        memcpy(tmp, &src->numero, i[0]);
                        memcpy(tmp+i[0], src->desc, i[1]);
                        memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);
                        memcpy(tmp, &src->numero, i[0]);
                        memcpy(tmp+i[0], src->desc, i[1]);
                        memcpy(tmp+i[0]+i[1], src->presentacion, i[2]);

+                       fprintf(stderr, "ACA: %s\n", src->presentacion);

                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->pvu, i[4]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->emin, i[5]);
                        memcpy(tmp+i[0]+i[1]+i[2], src->existencia, i[3]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3], src->pvu, i[4]);
                        memcpy(tmp+i[0]+i[1]+i[2]+i[3]+i[4], src->emin, i[5]);

diff --git a/emufs_gui/viewer.c b/emufs_gui/viewer.c
index 1b4c5419f0f50e3f1000b5a6e3d154d1670feacb..c181b9e34997e2c8e1a8ae7770b074a108d97b7c 100644 (file)
--- a/emufs_gui/viewer.c
+++ b/emufs_gui/viewer.c
@@ -14,7 +14,7 @@ static char *procesar_registro_factura_tipo1(EMUFS *emu, char *ptr, EMUFS_REG_SI
 
 static int preguntar_id(WINDOW *win, EMUFS *fp);
 
 
 static int preguntar_id(WINDOW *win, EMUFS *fp);
 

-void mostrar_info(WINDOW *padre, int h, int offset_alto)
+void mostrar_info(WINDOW *padre, int h, int offset_alto, int opt)

 {
        /* Info de teclas */
        wattron(padre, A_BOLD);
 {
        /* Info de teclas */
        wattron(padre, A_BOLD);


@@ -25,20 +25,22 @@ void mostrar_info(WINDOW *padre, int h, int offset_alto)
        mvwaddstr(padre, h-offset_alto+2, 8, "Salir = ENTER");
        mvwaddstr(padre, h-offset_alto+3, 8, "Scroll = A/Z");
        mvwaddstr(padre, h-offset_alto+4, 8, "Seleccionar registros = K/L");
        mvwaddstr(padre, h-offset_alto+2, 8, "Salir = ENTER");
        mvwaddstr(padre, h-offset_alto+3, 8, "Scroll = A/Z");
        mvwaddstr(padre, h-offset_alto+4, 8, "Seleccionar registros = K/L");

-       mvwaddstr(padre, h-offset_alto+5, 8, "Acciones: ");
-       waddstr(padre, "A");
-       wattron(padre, A_BOLD);
-       waddch(padre, 'g');
-       wattroff(padre, A_BOLD);
-       waddstr(padre, "regar ");
-       wattron(padre, A_BOLD);
-       waddstr(padre, "M");
-       wattroff(padre, A_BOLD);
-       waddstr(padre, "ofidicar ");
-       wattron(padre, A_BOLD);
-       waddstr(padre, "E");
-       wattroff(padre, A_BOLD);
-       waddstr(padre, "liminar ");
+       if (opt) {
+               mvwaddstr(padre, h-offset_alto+5, 8, "Acciones: ");
+               waddstr(padre, "A");
+               wattron(padre, A_BOLD);
+               waddch(padre, 'g');
+               wattroff(padre, A_BOLD);
+               waddstr(padre, "regar ");
+               wattron(padre, A_BOLD);
+               waddstr(padre, "M");
+               wattroff(padre, A_BOLD);
+               waddstr(padre, "ofidicar ");
+               wattron(padre, A_BOLD);
+               waddstr(padre, "E");
+               wattroff(padre, A_BOLD);
+               waddstr(padre, "liminar ");
+       }

        mvwaddstr(padre, h-offset_alto+6, 8, "Buscar ID : B");
        
        /* Info de leyenda */
        mvwaddstr(padre, h-offset_alto+6, 8, "Buscar ID : B");
        
        /* Info de leyenda */


@@ -52,7 +54,6 @@ void mostrar_info(WINDOW *padre, int h, int offset_alto)
        mvwaddstr(padre, h-offset_alto+4, 48, "(XXX) = ID de registro");
        mvwaddstr(padre, h-offset_alto+5, 48, "{XXX} = Tam. de registro");
        mvwaddstr(padre, h-offset_alto+6, 48, "  .   = Esp. Libre");
        mvwaddstr(padre, h-offset_alto+4, 48, "(XXX) = ID de registro");
        mvwaddstr(padre, h-offset_alto+5, 48, "{XXX} = Tam. de registro");
        mvwaddstr(padre, h-offset_alto+6, 48, "  .   = Esp. Libre");

-       mvwaddstr(padre, h-offset_alto+7, 48, " < >  = Separador Bloques");

 }
 
 char *juntar_memoria(char *s1, char *s2, char *s3, int size1, int size2, int size3)
 }
 
 char *juntar_memoria(char *s1, char *s2, char *s3, int size1, int size2, int size3)


@@ -165,7 +166,7 @@ void ver_bloques(WINDOW *padre, int w, int h, int cual)
 
        curs_set(0);
 
 
        curs_set(0);
 

-       mostrar_info(padre, h, offset_alto);
+       mostrar_info(padre, h, offset_alto, 0);

        
        mvwaddnstr(actual[1], 0, 0, data, pos_actual);
        wattron(actual[1], A_BOLD);
        
        mvwaddnstr(actual[1], 0, 0, data, pos_actual);
        wattron(actual[1], A_BOLD);


@@ -183,7 +184,9 @@ void ver_bloques(WINDOW *padre, int w, int h, int cual)
                        case 'B':
                                dlg = newwin(4, 50, h/2-2, w/2-25);
                                box(dlg, 0, 0);
                        case 'B':
                                dlg = newwin(4, 50, h/2-2, w/2-25);
                                box(dlg, 0, 0);

-                               preguntar_id(dlg, fp);
+                               indices_actual = preguntar_id(dlg, fp);
+                               if (indices_actual < 0) indices_actual = 0;
+                               if (indices_actual > indices_total) indices_actual = indices_total-1;

                                werase(dlg);
                                wrefresh(dlg);
                                delwin(dlg);
                                werase(dlg);
                                wrefresh(dlg);
                                delwin(dlg);


@@ -318,7 +321,7 @@ void ver_registros(WINDOW *padre, int w, int h, int cual)
 
        curs_set(0);
 
 
        curs_set(0);
 

-       mostrar_info(padre, h, offset_alto);
+       mostrar_info(padre, h, offset_alto, 1);

        
        mvwaddnstr(actual[1], 0, 0, data, pos_actual);
        wattron(actual[1], A_BOLD);
        
        mvwaddnstr(actual[1], 0, 0, data, pos_actual);
        wattron(actual[1], A_BOLD);


@@ -371,7 +374,7 @@ void ver_registros(WINDOW *padre, int w, int h, int cual)
                                indices = emufs_idx_get(fp, &indices_total);
 
                                /* Tengo que re-pintar algunas cosas */
                                indices = emufs_idx_get(fp, &indices_total);
 
                                /* Tengo que re-pintar algunas cosas */

-                               mostrar_info(padre, h, offset_alto);
+                               mostrar_info(padre, h, offset_alto, 1);

                                box(actual[0], 0, 0);
                                wrefresh(actual[0]);
                        break;                  
                                box(actual[0], 0, 0);
                                wrefresh(actual[0]);
                        break;                  


@@ -389,7 +392,7 @@ void ver_registros(WINDOW *padre, int w, int h, int cual)
                                data = procesar(fp, data, &size, &pos_actual, &ancho_registro);
 
                                /* Tengo que re-pintar algunas cosas */
                                data = procesar(fp, data, &size, &pos_actual, &ancho_registro);
 
                                /* Tengo que re-pintar algunas cosas */

-                               mostrar_info(padre, h, offset_alto);
+                               mostrar_info(padre, h, offset_alto, 1);

                                box(actual[0], 0, 0);
                                wrefresh(actual[0]);
                        break;
                                box(actual[0], 0, 0);
                                wrefresh(actual[0]);
                        break;


@@ -616,28 +619,16 @@ char *procesar_registro_articulo_tipo1(EMUFS *emu, char *ptr, EMUFS_REG_SIZE *si
 
 int preguntar_id(WINDOW *win, EMUFS *fp)
 {
 
 int preguntar_id(WINDOW *win, EMUFS *fp)
 {

-       int n=-1, j=0;
+       int n=-1;

        t_Form *form = form_crear(win);
        form_agregar_widget(form, INPUT, "ID a buscar", 8, "");
 
        do {
        t_Form *form = form_crear(win);
        form_agregar_widget(form, INPUT, "ID a buscar", 8, "");
 
        do {

-               if (j != 0) {
-                       curs_set(0);
-                       wattron(win, COLOR_PAIR(COLOR_YELLOW));
-                       wattron(win, A_BOLD);
-                       mvwaddstr(win, 2, 1, "Registro no encontrado!!");
-                       wattroff(win, A_BOLD);
-                       wattroff(win, COLOR_PAIR(COLOR_YELLOW));
-                       wrefresh(win);
-                       getch();
-                       werase(win);
-                       box(win, 0, 0);
-               }
+               form_set_valor(form, "ID a buscar", "");

                form_ejecutar(form, 1,1);
 
                n = form_obtener_valor_int(form, "ID a buscar");
                form_ejecutar(form, 1,1);
 
                n = form_obtener_valor_int(form, "ID a buscar");

-               j = 1;
-       } while (emufs_idx_existe_id(fp, n) != 0);
+       } while (n>0);

 
        form_destruir(form);
        return n;
 
        form_destruir(form);
        return n;