From 3cfdc427066ffa563f292c8267ecb8db8e1fd9d0 Mon Sep 17 00:00:00 2001 From: Sebastian Arena Date: Sun, 22 May 2005 16:24:18 +0000 Subject: [PATCH 1/1] 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 --- src/info.ingresarComentario.php | 4 +-- src/info.ingresarInfo.php | 4 +-- src/info.ingresarTemas.php | 4 +-- src/lib/Comentario.php | 34 ------------------ src/lib/Info.php | 25 +++---------- src/lib/Tema.php | 40 --------------------- src/lib/info.forms.php | 64 +++++++++------------------------ src/lib/info.functions.php | 59 ++++++++---------------------- 8 files changed, 40 insertions(+), 194 deletions(-) diff --git a/src/info.ingresarComentario.php b/src/info.ingresarComentario.php index 6d0f472..139ebb7 100644 --- a/src/info.ingresarComentario.php +++ b/src/info.ingresarComentario.php @@ -36,10 +36,8 @@ if (!$_SESSION['user']->esAdmin()) { //isset($_POST['hidInfoID']) && if (isset($_POST['txtComentario'])) { $__hidID = info_ingresar_comentario( $__hidID, $id, $_POST['txtComentario'], $_SESSION['user']->getId() ); - info_form_ingresar_comentario( $__hidID, $id, $_POST['txtComentario'] ); - } else { - info_form_ingresar_comentario( $__hidID, $id, "" ); } + info_form_ingresar_comentario( -1, $id, "" ); } else { echo warn("No esta habilitado para dar de alta comentarios, debe ser socio o asesor"); } diff --git a/src/info.ingresarInfo.php b/src/info.ingresarInfo.php index cd71ccd..d546d02 100644 --- a/src/info.ingresarInfo.php +++ b/src/info.ingresarInfo.php @@ -27,10 +27,8 @@ if (!$_SESSION['user']->esAdmin()) { if (isset($_POST['txtTema']) && isset($_POST['txtURL']) && isset($_POST['txtComentario'])) { $__hidID = info_ingresar_info( $__hidID, $_POST['txtTema'], $_POST['txtURL'], $_SESSION['user']->getId(), $_POST['txtComentario'] ); - info_form_ingresar_info( $__hidID, $_POST['txtTema'], $_POST['txtURL'], $_POST['txtComentario'] ); - } else { - info_form_ingresar_info( $__hidID, "", "", "" ); } + info_form_ingresar_info( -1, "", "", "" ); } else { echo warn("No esta habilitado para dar de alta informacion, debe ser socio o asesor"); } diff --git a/src/info.ingresarTemas.php b/src/info.ingresarTemas.php index 7e8a277..fe103af 100644 --- a/src/info.ingresarTemas.php +++ b/src/info.ingresarTemas.php @@ -27,10 +27,8 @@ if ($_SESSION['user']->esAdmin()) { if (isset($_POST['txtNombre']) && isset($_FILES['txtIcono']) && isset($_POST['txtDescripcion'])) { $__hidID = info_ingresar_tema( $__hidID, $_POST['txtNombre'], $_SESSION['user']->getId(), $_POST['txtDescripcion'], $_FILES['txtIcono']['tmp_name'] ); - info_form_ingresar_tema( $__hidID, $_POST['txtNombre'], $_FILES['txtIcono'], $_POST['txtDescripcion'] ); - } else { - info_form_ingresar_tema( $__hidID, "", "", "" ); } + info_form_ingresar_tema( -1, "", "", "" ); } else { echo warn("No esta habilitado para dar de alta un tema, debe ser administrador del sistema"); } diff --git a/src/lib/Comentario.php b/src/lib/Comentario.php index 9a412ef..786d37b 100644 --- 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 = "ERROR 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() { ?> infoCSVConst, 'r')) == false) return false; // error + if (($f = fopen($this->infoCSVConst, 'r+')) == false) return false; // error $index = 0; - $indexSave = 0; while (!feof($f)) { + $index = ftell($f); $d = fgetcsv($f, 4096); - $index++; - if ($d[0] == $this->id) + if ($d[0] == $this->id) { - fclose($f); - if (($f = fopen($this->infoCSVConst, 'r+')) == false) return false; // error - while ($indexSave!=($index-1)) { fgetcsv($f, 4096); $indexSave++; } - fputcsv($f, array($this->id, $this->tema, $this->url, $this->autor, $this->comentario, $this->ranking, $this->fecha)); + fseek($f,$index); + fputcsv($f, array($this->id, $this->tema, $this->url, $this->autor, $this->comentario, $this->ranking, $this->fecha)); fclose($f); return true; } @@ -113,18 +110,6 @@ class Info extends Item return $resultado; } - function modificar( $pId, $pTema, $pURL, $pAutor, $pComentario ) { - $this->saveLoadThis( $pId, $pTema, $pURL, $pAutor, $pComentario ); - $resultado = ""; - if (!$this->saveExisting()) { - $resultado = "ERROR al modificar la informacion. Revise que los parametros sean los adecuados"; - file_log_add($pAutor,$resultado); - } else { - file_log_add($pAutor,"Se modificó la información"); - } - return $resultado; - } - function toHTML() { ?> icono tema diff --git a/src/lib/Tema.php b/src/lib/Tema.php index 72ea56d..1fbb273 100644 --- 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)); } - function saveExisting() { - if (($f = fopen($this->temasCSVConst, '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->temasCSVConst, 'r+')) == false) return false; // error - while ($indexSave!=($index-1)) { fgetcsv($f, 4096); $indexSave++; } - fputcsv($f, array($this->id, $this->autor, $this->nombre, $this->icono, $this->texto, $this->fecha)); - fclose($f); - return true; - } - } - fclose($f); - return false; - } - function saveLoadThis( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono ) { $this->id = $pId; $this->autor = $pAutor; @@ -105,24 +83,6 @@ class Tema extends Item return $resultado; } - function modificar( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono ) { - $this->saveLoadThis( $pId, $pNombre, $pAutor, $pDescripcion, $pIcono ); - $resultado = ""; - //Se obtiene el ID mas alto actual - if (!$this->saveExisting()) { - $resultado = "ERROR al modificar el Tema '" . $pNombre . "'. Revise que los parametros sean los adecuados"; - file_log_add( $pAutor, $resultado); - } else { - if (!copy($pIcono, "temas/" . $pId)) { - $resultado = "No se pudo modificar la foto."; - file_log_add( $pAutor, $resultado); - } else { - file_log_add( $pAutor, "Se modificó el el '" . $pNombre . "'"); - } - } - return $resultado; - } - function toHTML() { $objUser = new Usuario($this->autor); ?> diff --git a/src/lib/info.forms.php b/src/lib/info.forms.php index ba3d940..a5b2588 100644 --- 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 ) { @@ -43,14 +37,10 @@ function info_form_ingresar_tema( $__hidID, $nombre, $icono, $descripcion ) { - - - - - + + + +
-   -   -
Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.
@@ -84,14 +74,8 @@ function info_form_ingresar_info( $__hidID, $tema, $url, $comentario ) { if ( (!check_vacio(f.txtTema, 'Tema')) ) return false; if ( (!check_vacio(f.txtURL, 'URL')) || (!check_min_max_cant(f.txtURL, 'URL', 5, 200)) ) return false; if ( (!check_vacio(f.txtComentario, 'Comentario')) || (!check_min_max_cant(f.txtComentario, 'Comentario', 10, 500)) ) return false; - return true; + return confirm("Confirma que los datos son correctos?\n\nTema: " + f.txtNombre.value + "\nIcono: " + f.txtIcono.value + "\nDescripcion: " + f.txtDescripcion.value + "\n"); } - - function info_blankActual(f) { - if (confirm("Seguro desea perder la posibilidad de modificar?")) { - location.href = ""; - } - } --> @@ -108,14 +92,10 @@ function info_form_ingresar_info( $__hidID, $tema, $url, $comentario ) { - - - - - + + + +
-   -   -
Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.
@@ -135,14 +115,8 @@ function info_form_ingresar_comentario( $__hidID, $info_id, $comentario ) { function info_validar_ingresar_comentario(f) { if ( (!check_vacio(f.txtComentario, 'Comentario') || (!check_min_max_cant(f.txtComentario, 'Comentario', 10, 500)) ) ) return false; - return true; + return confirm("Confirma que los datos son correctos?\n\nComentario: " + f.txtComentario.value + "\n"); } - - function info_blankActual(f) { - if (confirm("Seguro desea perder la posibilidad de modificar?")) { - location.href = ""; - } - } --> @@ -151,14 +125,10 @@ function info_form_ingresar_comentario( $__hidID, $info_id, $comentario ) { - - - - - + + + +
-   -   -
Una vez guardados los datos no pueden ser modificados. Verifique los mismos antes de hacerlo.
diff --git a/src/lib/info.functions.php b/src/lib/info.functions.php index 91ed33f..9716ca4 100644 --- a/src/lib/info.functions.php +++ b/src/lib/info.functions.php @@ -38,23 +38,13 @@ function info_obtener_mayor_id() { function info_ingresar_tema( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono ) { $objTema = new Tema(0,0); - if ($__hidID == -1 ) { - $__hidID = info_obtener_mayor_id() + 1; - $resultado = $objTema->ingresar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono ); - if ( $resultado == "" ) { - echo ok("Se agregó el Tema '" . $pNombre . "'"); - } else { - echo warn($resultado); - } + $__hidID = info_obtener_mayor_id() + 1; + $resultado = $objTema->ingresar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono ); + if ( $resultado == "" ) { + echo ok("Se agregó el Tema '" . $pNombre . "'"); } else { - $resultado = $objTema->modificar( $__hidID, $pNombre, $pAutor, $pDescripcion, $pIcono ); - if ( $resultado == "" ) { - echo ok("Se modificó el Tema '" . $pNombre . "'"); - } else { - echo warn($resultado); - } + echo warn($resultado); } - return $__hidID; } @@ -124,21 +114,12 @@ function info_obtener_mayor_id_info() { function info_ingresar_info( $__hidID, $pTema, $pURL, $pAutor, $pComentario ) { $objInfo = new Info(0,0); - if ($__hidID == -1 ) { - $__hidID = info_obtener_mayor_id_info() + 1; - $resultado = $objInfo->ingresar( $__hidID, $pTema, $pURL, $pAutor, $pComentario ); - if ( $resultado == "" ) { - echo ok("Se agregó la información."); - } else { - echo warn($resultado); - } + $__hidID = info_obtener_mayor_id_info() + 1; + $resultado = $objInfo->ingresar( $__hidID, $pTema, $pURL, $pAutor, $pComentario ); + if ( $resultado == "" ) { + echo ok("Se agregó la información."); } else { - $resultado = $objInfo->modificar( $__hidID, $pTema, $pURL, $pAutor, $pComentario ); - if ( $resultado == "" ) { - echo ok("Se modificó la información."); - } else { - echo warn($resultado); - } + echo warn($resultado); } return $__hidID; } @@ -196,22 +177,12 @@ function info_obtener_mayor_id_comentario($pInfo_ID) { function info_ingresar_comentario( $__hidID, $pInfo_ID, $pComentario, $pAutor ) { $objInfo = new Comentario(0,0,0); - - if ($__hidID == -1 ) { - $__hidID = info_obtener_mayor_id_comentario($pInfo_ID) + 1; - $resultado = $objInfo->ingresar( $__hidID, $pInfo_ID, $pComentario, $pAutor ); - if ( $resultado == "" ) { - echo ok("Se agregó el comentario."); - } else { - echo warn($resultado); - } + $__hidID = info_obtener_mayor_id_comentario($pInfo_ID) + 1; + $resultado = $objInfo->ingresar( $__hidID, $pInfo_ID, $pComentario, $pAutor ); + if ( $resultado == "" ) { + echo ok("Se agregó el comentario."); } else { - $resultado = $objInfo->modificar( $__hidID, $pInfo_ID, $pComentario, $pAutor ); - if ( $resultado == "" ) { - echo ok("Se modificó el comentario."); - } else { - echo warn($resultado); - } + echo warn($resultado); } return $__hidID; } -- 2.43.0