Aca esta mi bugfixing, para que le llevemos algo que le guste mas.

Bastante bronca conque me haya cagado una boludez.
NO DESCUBRI el error.
Lo que hice fue:
  1) Cambie lo de la moficacion, y no la permito mas.
     Simplemente le pido por JScript mucha confirmacion sobre lo que esta grabando. Y listo.
  2) En un lugar si o si tengo que hacer modificacion ( que es cuando updateo el ranking ). Asi que cambie el algoritmo, a uno mucho mas eficiente. Si bien no me tiraba el error seguro esto lo arreglo. ESPERO!

  En cuanto pueda poner Slackware les juro que lo pruebo. Sorry que no anduvo :(
  Saludos

diff --git a/src/info.ingresarComentario.php b/src/info.ingresarComentario.php
index 6d0f472c0ef53dc7d3a7832f021f785fcc1fe687..139ebb777e192bb284f50b430a4cd10b55b4406a 100644 (file)
--- a/src/info.ingresarComentario.php
+++ b/src/info.ingresarComentario.php
@@ -36,10 +36,8 @@ if (!$_SESSION['user']->esAdmin()) {
        //isset($_POST['hidInfoID']) && \r
        if (isset($_POST['txtComentario'])) {\r
         $__hidID = info_ingresar_comentario( $__hidID, $id, $_POST['txtComentario'], $_SESSION['user']->getId() );\r
-               info_form_ingresar_comentario( $__hidID, $id, $_POST['txtComentario'] );\r
-       } else {\r
-               info_form_ingresar_comentario( $__hidID, $id, "" ); \r
        }\r
+       info_form_ingresar_comentario( -1, $id, "" );\r
 } else {\r
     echo warn("No esta habilitado para dar de alta comentarios, debe ser socio o asesor");\r
 }\r

diff --git a/src/info.ingresarInfo.php b/src/info.ingresarInfo.php
index cd71ccd1ad156370a72d8d4e459a35e80746ff77..d546d020d9b6a1e5caecf55147ee93f5158fc0fe 100644 (file)
--- a/src/info.ingresarInfo.php
+++ b/src/info.ingresarInfo.php
@@ -27,10 +27,8 @@ if (!$_SESSION['user']->esAdmin()) {
 \r
        if (isset($_POST['txtTema']) && isset($_POST['txtURL']) && isset($_POST['txtComentario'])) {\r
         $__hidID = info_ingresar_info( $__hidID, $_POST['txtTema'], $_POST['txtURL'], $_SESSION['user']->getId(), $_POST['txtComentario'] );\r
-               info_form_ingresar_info( $__hidID, $_POST['txtTema'], $_POST['txtURL'], $_POST['txtComentario'] );\r
-       } else {\r
-               info_form_ingresar_info( $__hidID, "", "", "" ); \r
        }\r
+       info_form_ingresar_info( -1, "", "", "" ); \r
 } else {\r
     echo warn("No esta habilitado para dar de alta informacion, debe ser socio o asesor");\r
 }\r

diff --git a/src/info.ingresarTemas.php b/src/info.ingresarTemas.php
index 7e8a2772535dbb8e48103565d3dd9be0adcc5f88..fe103afe52a6456e103c7fdcac37296129f29323 100644 (file)
--- a/src/info.ingresarTemas.php
+++ b/src/info.ingresarTemas.php
@@ -27,10 +27,8 @@ if ($_SESSION['user']->esAdmin()) {
 \r
        if (isset($_POST['txtNombre']) && isset($_FILES['txtIcono']) && isset($_POST['txtDescripcion'])) {\r
         $__hidID = info_ingresar_tema( $__hidID, $_POST['txtNombre'], $_SESSION['user']->getId(), $_POST['txtDescripcion'], $_FILES['txtIcono']['tmp_name'] );\r
-               info_form_ingresar_tema( $__hidID, $_POST['txtNombre'], $_FILES['txtIcono'], $_POST['txtDescripcion'] );\r
-       } else {\r
-               info_form_ingresar_tema( $__hidID, "", "", "" ); \r
        }\r
+       info_form_ingresar_tema( -1, "", "", "" );\r
 } else {\r
        echo warn("No esta habilitado para dar de alta un tema, debe ser administrador del sistema");\r
 }\r

diff --git a/src/lib/Comentario.php b/src/lib/Comentario.php
index 9a412ef5548eb5035f3d8dbed724a4cbd396f945..786d37b750ab74e51545aa1fe28f9fa4b3a3b7c1 100644 (file)
--- a/src/lib/Comentario.php
+++ b/src/lib/Comentario.php
@@ -54,28 +54,6 @@ class Comentario extends Item
                return fappendcsv($this->comentarioCSVConst, array( $this->id, $this->autor, $this->texto, $this->fecha ));
        }
 
-       function saveExisting() {
-               if (($f = fopen($this->comentarioCSVConst, 'r')) == false) return false; // error
-               $index = 0;
-               $indexSave = 0;
-        while (!feof($f))
-               {
-                       $d = fgetcsv($f, 4096);
-                       $index++;
-                       if ($d[0] == $this->id)
-                       {
-                fclose($f);
-                               if (($f = fopen($this->comentarioCSVConst, 'r+')) == false) return false; // error
-                               while ($indexSave!=($index-1)) { fgetcsv($f, 4096); $indexSave++; }
-                fputcsv($f, array( $this->id, $this->autor, $this->texto, $this->fecha ));
-                               fclose($f);
-                               return true;
-                       }
-               }
-               fclose($f);
-               return false;
-       }
-
        function saveLoadThis( $pId, $pInfo_ID, $pComentario, $pAutor ) {
                $this->id                       = $pId;
                $this->autor            = $pAutor;
@@ -96,18 +74,6 @@ class Comentario extends Item
                return $resultado;
        }
 
-       function modificar( $pId, $pInfo_ID, $pComentario, $pAutor ) {
-               $this->saveLoadThis( $pId, $pInfo_ID, $pComentario, $pAutor );
-               $resultado = "";
-        if (!$this->saveExisting()) {
-                       $resultado = "<strong>ERROR</strong> al modificar el comentario. Revise que los parametros sean los adecuados";
-                       file_log_add($pAutor,$resultado);
-        } else {
-                       file_log_add($pAutor,"Se modificó el comentario");
-               }
-               return $resultado;
-       }
-
        function toHTML() {
         ?><tr>
                        <?php

diff --git a/src/lib/Info.php b/src/lib/Info.php
index d2bbbaaf7c229b2e1e3685e88b36e176ac3d8575..a6d2baa30fea78c1119495122c11dc750906fe84 100644 (file)
--- a/src/lib/Info.php
+++ b/src/lib/Info.php
@@ -69,19 +69,16 @@ class Info extends Item
        }\r
 \r
        function saveExisting() {\r
-               if (($f = fopen($this->infoCSVConst, 'r')) == false) return false; // error\r
+               if (($f = fopen($this->infoCSVConst, 'r+')) == false) return false; // error\r
                $index = 0;\r
-               $indexSave = 0;\r
         while (!feof($f))\r
                {\r
+                       $index = ftell($f);\r
                        $d = fgetcsv($f, 4096);\r
-                       $index++;\r
-                       if ($d[0] == $this->id)\r
+            if ($d[0] == $this->id)\r
                        {\r
-                fclose($f);\r
-                               if (($f = fopen($this->infoCSVConst, 'r+')) == false) return false; // error\r
-                               while ($indexSave!=($index-1)) { fgetcsv($f, 4096); $indexSave++; }\r
-                fputcsv($f, array($this->id, $this->tema, $this->url, $this->autor, $this->comentario, $this->ranking, $this->fecha));\r
+                fseek($f,$index);\r
+                               fputcsv($f, array($this->id, $this->tema, $this->url, $this->autor, $this->comentario, $this->ranking, $this->fecha));\r
                                fclose($f);\r
                                return true;\r
                        }\r
@@ -113,18 +110,6 @@ class Info extends Item
                return $resultado;\r
        }\r
 \r
-       function modificar( $pId, $pTema, $pURL, $pAutor, $pComentario ) {\r
-        $this->saveLoadThis( $pId, $pTema, $pURL, $pAutor, $pComentario );\r
-               $resultado = "";\r
-        if (!$this->saveExisting()) {\r
-                       $resultado = "<strong>ERROR</strong> al modificar la informacion. Revise que los parametros sean los adecuados";\r
-                       file_log_add($pAutor,$resultado);\r
-        } else {\r
-                       file_log_add($pAutor,"Se modificó la información");\r
-               }\r
-               return $resultado;\r
-       }\r
-\r
        function toHTML() {\r
         ?><tr>\r
                        <td><img src="<?php echo $this->tema_icono;?>" border="0" alt="icono tema"/></td>\r

diff --git a/src/lib/Tema.php b/src/lib/Tema.php
index 72ea56d065181dbcea15a8a22dc1a27c1ad002b5..1fbb27324e7fe24adad6a7c650dd4cb3e7f26f0d 100644 (file)
--- a/src/lib/Tema.php
+++ b/src/lib/Tema.php
@@ -57,28 +57,6 @@ class Tema extends Item
                return fappendcsv($this->temasCSVConst, array($this->id, $this->autor, $this->nombre, $this->icono, $this->texto, $this->fecha));\r
        }\r
 \r
-       function saveExisting() {\r
-               if (($f = fopen($this->temasCSVConst, 'r')) == false) return false; // error\r
-               $index = 0;\r
-               $indexSave = 0;\r
-        while (!feof($f))\r
-               {\r
-                       $d = fgetcsv($f, 4096);\r
-                       $index++;\r
-                       if ($d[0] == $this->id)\r
-                       {\r
-                fclose($f);\r
-                               if (($f = fopen($this->temasCSVConst, 'r+')) == false) return false; // error\r
-                               while ($indexSave!=($index-1)) { fgetcsv($f, 4096); $indexSave++; }\r
-                fputcsv($f, array($this->id, $this->autor, $this->nombre, $this->icono, $this->texto, $this->fecha));\r
-                               fclose($f);\r
-                               return true;\r
-                       }\r
-               }\r
-               fclose($f);\r
-               return false;\r
-       }\r
-\r
        function saveLoadThis( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono ) {\r
                $this->id               = $pId;\r
                $this->autor    = $pAutor;\r
@@ -105,24 +83,6 @@ class Tema extends Item
                return $resultado;\r
        }\r
 \r
-       function modificar( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono ) {\r
-        $this->saveLoadThis( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono );\r
-               $resultado = "";\r
-        //Se obtiene el ID mas alto actual\r
-               if (!$this->saveExisting()) {\r
-                       $resultado = "<strong>ERROR</strong> al modificar el Tema '" . $pNombre . "'. Revise que los parametros sean los adecuados";\r
-                       file_log_add( $pAutor, $resultado);\r
-               } else {\r
-            if (!copy($pIcono, "temas/" . $pId)) {\r
-                               $resultado = "No se pudo modificar la foto.";\r
-                               file_log_add( $pAutor, $resultado);\r
-                       } else {\r
-                               file_log_add( $pAutor, "Se modificó el el '" . $pNombre . "'");\r
-                       }\r
-               }\r
-               return $resultado;\r
-       }\r
-\r
        function toHTML() {\r
                $objUser = new Usuario($this->autor);\r
                ?><tr>\r

diff --git a/src/lib/info.forms.php b/src/lib/info.forms.php
index ba3d940e147e5def90c0c2e03c21175800afe624..a5b258840ee1b398011df5b8e4496ce0d1a4cb17 100644 (file)
--- a/src/lib/info.forms.php
+++ b/src/lib/info.forms.php
@@ -16,17 +16,11 @@ function info_form_ingresar_tema( $__hidID, $nombre, $icono, $descripcion ) {
                 <!--\r
                 function info_validar_ingresar_tema(f)\r
                 {\r
-                    if ( (!check_vacio(f.txtNombre, 'Nombre')) || (!check_min_max_cant(f.txtURL, 'URL', 5, 20)) ) return false;\r
+                                       if ( (!check_vacio(f.txtNombre, 'Tema')) || (!check_min_max_cant(f.txtNombre, 'Tema', 5, 20)) ) return false;\r
                                        if ( (!check_vacio(f.txtIcono, 'Icono')) ) return false;\r
-                                       if ( (!check_vacio(f.txtDescripcion, 'Descripcion')) || (!check_min_max_cant(f.txtURL, 'URL', 5, 200)) ) return false;\r
-                    return true;\r
+                                       if ( (!check_vacio(f.txtDescripcion, 'Descripcion')) || (!check_min_max_cant(f.txtDescripcion, 'Descripcion', 5, 200)) ) return false;\r
+                    return confirm("Confirma que los datos son correctos?\n\nTema: " + f.txtNombre.value + "\nIcono: " + f.txtIcono.value + "\nDescripcion: " + f.txtDescripcion.value + "\n");\r
                 }\r
-\r
-                               function info_blankActual(f) {\r
-                                       if (confirm("Seguro desea perder la posibilidad de modificar?")) {\r
-                                               location.href = "<?php echo $_SERVER['PHP_SELF'] . '?' . $_SERVER['QUERY_STRING']; ?>";\r
-                                       }\r
-                               }\r
                 -->\r
             </script>\r
                        <table>\r
@@ -43,14 +37,10 @@ function info_form_ingresar_tema( $__hidID, $nombre, $icono, $descripcion ) {
                                        <td><textarea name="txtDescripcion" rows="3" cols="50"><?php echo $descripcion ?></textarea></td>\r
                                </tr>\r
                                <tr>\r
-                     <? if ( $__hidID == -1 ) { ?>\r
-                                               <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Tema"/></td>\r
-                                       <? } else { ?>\r
-                                               <td colspan="2">\r
-                                                       <input type="submit" name="btnSubmit" value="Modificar Tema"/>&nbsp;\r
-                                                       <input type="button" name="btnNew" value="Nuevo Tema" onclick="info_blankActual(this.form);"/>&nbsp;\r
-                                               </td>\r
-                                       <? } ?>\r
+                                       <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Tema"/></td>\r
+                               </tr>\r
+                               <tr>\r
+                    <td colspan="2"><i>Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.</i></td>\r
                                </tr>\r
                        </table>\r
        </form>\r
@@ -84,14 +74,8 @@ function info_form_ingresar_info( $__hidID, $tema, $url, $comentario ) {
                                        if ( (!check_vacio(f.txtTema, 'Tema')) ) return false;\r
                     if ( (!check_vacio(f.txtURL, 'URL')) || (!check_min_max_cant(f.txtURL, 'URL', 5, 200)) ) return false;\r
                     if ( (!check_vacio(f.txtComentario, 'Comentario')) || (!check_min_max_cant(f.txtComentario, 'Comentario', 10, 500)) ) return false;\r
-                    return true;\r
+                    return confirm("Confirma que los datos son correctos?\n\nTema: " + f.txtNombre.value + "\nIcono: " + f.txtIcono.value + "\nDescripcion: " + f.txtDescripcion.value + "\n");\r
                 }\r
-\r
-                               function info_blankActual(f) {\r
-                                       if (confirm("Seguro desea perder la posibilidad de modificar?")) {\r
-                                               location.href = "<?php echo $_SERVER['PHP_SELF'] . '?' . $_SERVER['QUERY_STRING']; ?>";\r
-                                       }\r
-                               }\r
                 -->\r
             </script>\r
                        <table>\r
@@ -108,14 +92,10 @@ function info_form_ingresar_info( $__hidID, $tema, $url, $comentario ) {
                                        <td><textarea name="txtComentario" rows="3" cols="50"><?php echo $comentario ?></textarea></td>\r
                                </tr>\r
                                <tr>\r
-                    <? if ( $__hidID == -1 ) { ?>\r
-                                               <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Información"/></td>\r
-                                       <? } else { ?>\r
-                                               <td colspan="2">\r
-                                                       <input type="submit" name="btnSubmit" value="Modificar Información"/>&nbsp;\r
-                                                       <input type="button" name="btnNew" value="Nueva Información" onclick="info_blankActual(this.form);"/>&nbsp;\r
-                                               </td>\r
-                                       <? } ?>\r
+                                       <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Información"/></td>\r
+                               </tr>\r
+                               <tr>\r
+                    <td colspan="2"><i>Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.</i></td>\r
                                </tr>\r
                        </table>\r
        </form>\r
@@ -135,14 +115,8 @@ function info_form_ingresar_comentario( $__hidID, $info_id, $comentario ) {
                 function info_validar_ingresar_comentario(f)\r
                 {\r
                                        if ( (!check_vacio(f.txtComentario, 'Comentario') || (!check_min_max_cant(f.txtComentario, 'Comentario', 10, 500)) ) ) return false;\r
-                    return true;\r
+                    return confirm("Confirma que los datos son correctos?\n\nComentario: " + f.txtComentario.value + "\n");\r
                 }\r
-\r
-                               function info_blankActual(f) {\r
-                                       if (confirm("Seguro desea perder la posibilidad de modificar?")) {\r
-                                               location.href = "<?php echo $_SERVER['PHP_SELF'] . '?' . $_SERVER['QUERY_STRING']; ?>";\r
-                                       }\r
-                               }\r
                 -->\r
             </script>\r
                        <table>\r
@@ -151,14 +125,10 @@ function info_form_ingresar_comentario( $__hidID, $info_id, $comentario ) {
                                        <td><textarea name="txtComentario" rows="3" cols="50"><?php echo $comentario ?></textarea></td>\r
                                </tr>\r
                                <tr>\r
-                                       <? if ( $__hidID == -1 ) { ?>\r
-                                               <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Comentario"/></td>\r
-                                       <? } else { ?>\r
-                                               <td colspan="2">\r
-                                                       <input type="submit" name="btnSubmit" value="Modificar Comentario"/>&nbsp;\r
-                                                       <input type="button" name="btnNew" value="Nuevo Comentario" onclick="info_blankActual(this.form);"/>&nbsp;\r
-                                               </td>\r
-                                       <? } ?>\r
+                    <td colspan="2"><input type="submit" name="btnSubmit" value="Guardar Comentario"/></td>\r
+                               </tr>\r
+                               <tr>\r
+                    <td colspan="2"><i>Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.</i></td>\r
                                </tr>\r
                        </table>\r
        </form>\r

diff --git a/src/lib/info.functions.php b/src/lib/info.functions.php
index 91ed33f744276fc295a2b51ac91a6f4e369e0dd9..9716ca419f71d3342ea3d59b33a2072ad3380f75 100644 (file)
--- a/src/lib/info.functions.php
+++ b/src/lib/info.functions.php
@@ -38,23 +38,13 @@ function info_obtener_mayor_id() {
 \r
 function info_ingresar_tema( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono ) {\r
        $objTema = new Tema(0,0);\r
-    if ($__hidID == -1 ) {\r
-               $__hidID = info_obtener_mayor_id() + 1;\r
-               $resultado = $objTema->ingresar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se agregó el Tema '" . $pNombre . "'");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+    $__hidID = info_obtener_mayor_id() + 1;\r
+       $resultado = $objTema->ingresar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono );\r
+       if ( $resultado == "" ) {\r
+               echo ok("Se agregó el Tema '" . $pNombre . "'");\r
        } else {\r
-               $resultado = $objTema->modificar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se modificó el Tema '" . $pNombre . "'");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+               echo warn($resultado);\r
        }\r
-\r
        return $__hidID;\r
 }\r
 \r
@@ -124,21 +114,12 @@ function info_obtener_mayor_id_info() {
 \r
 function info_ingresar_info( $__hidID, $pTema, $pURL, $pAutor, $pComentario ) {\r
        $objInfo = new Info(0,0);\r
-       if ($__hidID == -1 ) {\r
-               $__hidID = info_obtener_mayor_id_info() + 1;\r
-               $resultado = $objInfo->ingresar( $__hidID, $pTema, $pURL, $pAutor, $pComentario );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se agregó la información.");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+       $__hidID = info_obtener_mayor_id_info() + 1;\r
+       $resultado = $objInfo->ingresar( $__hidID, $pTema, $pURL, $pAutor, $pComentario );\r
+       if ( $resultado == "" ) {\r
+               echo ok("Se agregó la información.");\r
        } else {\r
-               $resultado = $objInfo->modificar( $__hidID, $pTema, $pURL, $pAutor, $pComentario );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se modificó la información.");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+               echo warn($resultado);\r
        }\r
        return $__hidID;\r
 }\r
@@ -196,22 +177,12 @@ function info_obtener_mayor_id_comentario($pInfo_ID) {
 \r
 function info_ingresar_comentario( $__hidID, $pInfo_ID, $pComentario, $pAutor ) {\r
        $objInfo = new Comentario(0,0,0);\r
-\r
-       if ($__hidID == -1 ) {\r
-               $__hidID = info_obtener_mayor_id_comentario($pInfo_ID) + 1;\r
-               $resultado = $objInfo->ingresar( $__hidID, $pInfo_ID, $pComentario, $pAutor );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se agregó el comentario.");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+       $__hidID = info_obtener_mayor_id_comentario($pInfo_ID) + 1;\r
+       $resultado = $objInfo->ingresar( $__hidID, $pInfo_ID, $pComentario, $pAutor );\r
+       if ( $resultado == "" ) {\r
+               echo ok("Se agregó el comentario.");\r
        } else {\r
-               $resultado = $objInfo->modificar( $__hidID, $pInfo_ID, $pComentario, $pAutor );\r
-               if ( $resultado == "" ) {\r
-                       echo ok("Se modificó el comentario.");\r
-               } else {\r
-                       echo warn($resultado);\r
-               }\r
+               echo warn($resultado);\r
        }\r
        return $__hidID;\r
 }\r