Ayuda con script de guestbook

karlingaz · #1 (**permalink**) 31/07/2009, 19:28

Que tal amigos, tengo el siguiente script para un guestbook, pero resulta que no lee el txt donde guardo los comentarios, ni siquiera los escribe ya probé con permisos en 775 y 777, les dejo script si me pueden ayudar se los agradecería.

<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<p>Tu comentario:</p>
<p><textarea name="comentario" id="comentario" cols="60" rows="5"></textarea></p>
<p>Tu nombre:</p>
<input name="nombre" type="text" id="nombre" size="40">
<p>Tu e-mail:</p>
<input name="email" type="text" id="email" size="40">
<input type="submit" name="Publicar" id="Publicar" value="Publicar">
</form>
<p><span class="text_3"><strong>Mostrar todos los comentarios</strong></span></p>
<?php
//Guarda el nombre del archivo en la variable
$file = "libro.txt";
//Variable comentario definida? El nombre y email no estan vacios?
if (isset($_POST['cometario'] ) && $_POST['nombre'] != "" && $_POST['email'] != ""){
$comentario = $_POST['comentario'];
$nombre = $_POST['nombre'];
$email = $_POST['email'];
//El archivo se abre para lectura-escritura
$fp = fopen($file, "r+");
//Leer todos los datos y almacenar en $old
$old = fread($fp, filesize($file));
//Se crea el vinculo de email
$email = "<a href=\"mailto:$email\">$email </a>";
//Se incluye la fecha y se le da formato
$dateOfEntry = date("Y-n-j");
//Ocultar caracteres HTML, eliminar slashes, mantener saltos de linea
$comentario = htmlspecialchars($comentario);
$comentario = stripslashes(n12br($comentario));
//Montar la entrada (entry) del libro de visitas
$entry ="<p><b>$nombre</b> ($email) wrote on <i>$dateOfEntry</i>: <br>$comentario</p>\n";
//El cursor invisible salta al principio
rewind($fp);
//Escribir la nueva entrada antes de las antiguas
fputs($fp, "$entry \n $old");
//Cerrar archivo
fclose($fp);
}
//Mostrar archivo completo
readfile($file);
?>

jam1138 · #2 (**permalink**) 31/07/2009, 21:06

Tema movido desde POO.

karlingaz · #3 (**permalink**) 01/08/2009, 10:37

Gracias Jam por mover el tema a un lugar mas apropiado, espero aquí me puedan ayudar

saludoss

gjx2 · #4 (**permalink**) 01/08/2009, 10:56

Vamos a ver con esto

Código PHP:

Ver original<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<p>Tu comentario:</p>
<p><textarea name="comentario" id="comentario" cols="60" rows="5"></textarea></p>
<p>Tu nombre:</p>
<input name="nombre" type="text" id="nombre" size="40">
<p>Tu e-mail:</p>
<input name="email" type="text" id="email" size="40">
<input type="submit" name="Publicar" id="Publicar" value="Publicar">
</form> 
<p><span class="text_3"><strong>Mostrar todos los comentarios</strong></span></p>
<?php
//Guarda el nombre del archivo en la variable
$file = "libro.txt";
//Variable comentario definida? El nombre y email no estan vacios?
if (isset($_POST['comentario'] ) && !empty($_POST['nombre']) && !empty($_POST['email'])){
$comentario = $_POST['comentario'];
$nombre = $_POST['nombre'];
$email = $_POST['email'];
//El archivo se abre para lectura-escritura
$fp = @fopen($file, "a+");
//Leer todos los datos y almacenar en $old
$old = @fread($fp, filesize($file));
//Se crea el vinculo de email
$email = "<a href=\"mailto:$email\">$email </a>";
//Se incluye la fecha y se le da formato
$dateOfEntry = date("Y-n-j");
//Ocultar caracteres HTML, eliminar slashes, mantener saltos de linea
$comentario = htmlspecialchars($comentario);
//$comentario = stripslashes(n12br($comentario));
//Montar la entrada (entry) del libro de visitas
$entry ="<p><b>$nombre</b> ($email) wrote on <i>$dateOfEntry</i>: <br>$comentario</p>\n";
//El cursor invisible salta al principio
@rewind($fp);
//Escribir la nueva entrada antes de las antiguas
@fputs($fp, "$entry \n $old");
//Cerrar archivo
@fclose($fp);
} 
//Mostrar archivo completo
@readfile($file);
?>

gjx2 · #5 (**permalink**) 01/08/2009, 11:00

Veo que se duplican los mensajes
es por esta linea

@fputs($fp, "$entry \n $old");

quitale el $old

karlingaz · #6 (**permalink**) 01/08/2009, 11:17

gjx2

ya probé con las modificaciones que le hiciste al script pero sigue sin funcionar, no marca ningún error pero no escribe en el txt

lo eh probado en el server local y en un hosting que tengo, pero no funciona en ningun lado

jonatanc · #7 (**permalink**) 01/08/2009, 11:24

Cita:

Iniciado por karlingaz

gjx2

ya probé con las modificaciones que le hiciste al script pero sigue sin funcionar, no marca ningún error pero no escribe en el txt

lo eh probado en el server local y en un hosting que tengo, pero no funciona en ningun lado

Hola , prueba con el siguiente code y fijate si te da algun error o algo :).

Código php:

Ver original<html>
<head></head>
<body>
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<p>Tu comentario:</p>
<p><textarea name="comentario" id="comentario" cols="60" rows="5"></textarea></p>
<p>Tu nombre:</p>
<input name="nombre" type="text" id="nombre" size="40">
<p>Tu e-mail:</p>
<input name="email" type="text" id="email" size="40">
<input type="submit" name="Publicar" id="Publicar" value="Publicar">
</form> 
<p><span class="text_3"><strong>Mostrar todos los comentarios</strong></span></p>
</body>
</html>
<?php
//Guarda el nombre del archivo en la variable
$file = "libro.txt";
//Variable comentario definida? El nombre y email no estan vacios?
if (isset($_POST['comentario'] ) && !empty($_POST['nombre']) && !empty($_POST['email'])){
$comentario = $_POST['comentario'];
$nombre = $_POST['nombre'];
$email = $_POST['email'];
//El archivo se abre para lectura-escritura
$fp = fopen($file, "a+");
//Leer todos los datos y almacenar en $old
$old = fread($fp, filesize($file));
//Se crea el vinculo de email
$email = "<a href=\"mailto:$email\">$email </a>";
//Se incluye la fecha y se le da formato
$dateOfEntry = date("Y-n-j");
//Ocultar caracteres HTML, eliminar slashes, mantener saltos de linea
$comentario = htmlspecialchars($comentario);
//$comentario = stripslashes(n12br($comentario));
//Montar la entrada (entry) del libro de visitas
$entry ="<p><b>$nombre</b> ($email) wrote on <i>$dateOfEntry</i>: <br>$comentario</p>\n";
//El cursor invisible salta al principio
rewind($fp);
//Escribir la nueva entrada antes de las antiguas
fputs($fp, "$entry \n");
//Cerrar archivo
fclose($fp);
} 
//Mostrar archivo completo
readfile($file);
?>

Saludos!

gjx2 · #8 (**permalink**) 01/08/2009, 11:26

Tienes que copiar el codigo
tal cual como te lo presento

Por que vi que tenias un problema en isset($_POST['comentario'])
y algunos mas.

gjx2 · #9 (**permalink**) 01/08/2009, 11:29

Este sr. jonatanc esta empenado en no usar la @ para los wrn . :s

Sr. jonatanc
si el archivo no existe como piensa usted que lo vas a leer con fread() ?
se va a crear el archivo , pero te va adar un wrn .

al menos que uses
file_exists()

MasterXNicoX · #10 (**permalink**) 01/08/2009, 11:32

Cita:

Este sr. jonatanc esta empenado en no usar la @ para los wrn . :s

Jajajajajaj siiii desde hoy xD.. Pero bue, cada uno tiene sus formas de trabajar

jonatanc · #11 (**permalink**) 01/08/2009, 11:34

Estimados , la idea de eliminar todo tipo de @ consiste en que salgan todos los errores a la luz , simplemente para eso , mi tecnica de programar consiste en no suprimir nada , sino al contrario en programar correctamente o intentar programar correctamente.

Saludos!

gjx2 · #12 (**permalink**) 01/08/2009, 11:41

Bueno lo felicito por esa idea tan brillante.
el problema es que el codigo aqui presente no fue hecho por usted
fue hecho por una persona que quiere resolver su problema no buscar un problema a otro problema.

se lo digo con todo respeto .

karlingaz · #13 (**permalink**) 01/08/2009, 11:41

Cita:

Iniciado por gjx2

Tienes que copiar el codigo
tal cual como te lo presento

Por que vi que tenias un problema en isset($_POST['comentario'])
y algunos mas.

si use el código tal como me lo das, ahora el archivo libro.txt si existe en el mismo directorio y tiene propiedades de lectura y escritura.

jonatanc · #14 (**permalink**) 01/08/2009, 11:45

Solamente era para intentar ayudar a ver en que parte le falla ese codigo , no me interesa quien lo hizo o quien no lo hizo , yo probe el codigo acá en local y funciona sin problemas ,pero xq tiene bien los permisos y demás , por eso le dije que pruebe con ese codigo para que salten errores y tratar de ver xq no andaba , pero si ya existe libro.txt y tiene los permisos bien otorgados deberian de andar , por lo menos asi sucede en mi local.

Saludos y tema terminado :) , no tengo ganas de discutir por un arroba pelo....

gjx2 · #15 (**permalink**) 01/08/2009, 11:48

Cita:

Iniciado por jonatanc

Estimados , la idea de eliminar todo tipo de @ consiste en que salgan todos los errores a la luz , simplemente para eso , mi tecnica de programar consiste en no suprimir nada , sino al contrario en programar correctamente o intentar programar correctamente.

Saludos!

Como bien dice el sr. quitale los @ al codigo a ver si te genera algun wrn
Y verifica si al menos te esta guardando algo en el documento.

karlingaz · #16 (**permalink**) 01/08/2009, 11:50

Cita:

Iniciado por jonatanc

Solamente era para intentar ayudar a ver en que parte le falla ese codigo , no me interesa quien lo hizo o quien no lo hizo , yo probe el codigo acá en local y funciona sin problemas ,pero xq tiene bien los permisos y demás , por eso le dije que pruebe con ese codigo para que salten errores y tratar de ver xq no andaba , pero si ya existe libro.txt y tiene los permisos bien otorgados deberian de andar , por lo menos asi sucede en mi local.

Saludos y tema terminado :) , no tengo ganas de discutir por un arroba pelo....

Vaya, siento que se halla armado controversia por las técnicas que usan cada uno de ustedes, lo que no entiendo por que a mi no me funciona, y no marca ningún error, pero no escribe en el txt.

lo he probado local y en el hosting que tengo y en ningún lado me anda.

Gracias a los dos por su ayuda

gjx2 · #17 (**permalink**) 01/08/2009, 12:05

Descuida es solo una charla como amigos

Me gustaria que probaras algo tan simple como esto.

Código PHP:

Ver original$file = "libro.txt";
 
$contenido = "Hola Mundo!";
$fp = fopen($file, "a+");
 
fputs($fp, "$contenido \n");
 
fclose($fp); 
 
readfile($file);

karlingaz · #18 (**permalink**) 01/08/2009, 12:24

Cita:

Iniciado por gjx2

Descuida es solo una charla como amigos

Me gustaria que probaras algo tan simple como esto.

Código PHP:

Ver original$file = "libro.txt";
 
$contenido = "Hola Mundo!";
$fp = fopen($file, "a+");
 
fputs($fp, "$contenido \n");
 
fclose($fp); 
 
readfile($file);

pues de esta forma si muestra el hola mundo

gjx2 · #19 (**permalink**) 01/08/2009, 12:27

Vamos hacer esto.

Código PHP:

 
<html> 
<head></head> 
<body> 
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post"> 
<p>Tu comentario:</p> 
<p><textarea name="comentario" id="comentario" cols="60" rows="5"></textarea></p> 
<p>Tu nombre:</p> 
<input name="nombre" type="text" id="nombre" size="40"> 
<p>Tu e-mail:</p> 
<input name="email" type="text" id="email" size="40"> 
<input type="submit" name="Publicar" id="Publicar" value="Publicar"> 
</form>  
<p><span class="text_3"><strong>Mostrar todos los comentarios</strong></span></p> 
</body> 
</html> 
<?php 
//Guarda el nombre del archivo en la variable 
$file = "libro.txt"; 
//Variable comentario definida? El nombre y email no estan vacios? 
 
$comentario = $_POST['comentario']; 
$nombre = $_POST['nombre']; 
$email = $_POST['email']; 
//El archivo se abre para lectura-escritura 
$fp = fopen($file, "a+"); 
//Leer todos los datos y almacenar en $old 
$old = fread($fp, filesize($file)); 
//Se crea el vinculo de email 
$email = "<a href=\"mailto:$email\">$email </a>"; 
//Se incluye la fecha y se le da formato 
$dateOfEntry = date("Y-n-j"); 
//Ocultar caracteres HTML, eliminar slashes, mantener saltos de linea 
$comentario = htmlspecialchars($comentario); 
//$comentario = stripslashes(n12br($comentario)); 
//Montar la entrada (entry) del libro de visitas 
$entry ="<p><b>$nombre</b> ($email) wrote on <i>$dateOfEntry</i>: <br>$comentario</p>\n"; 
//El cursor invisible salta al principio 
rewind($fp); 
//Escribir la nueva entrada antes de las antiguas 
fputs($fp, "$entry \n"); 
//Cerrar archivo 
fclose($fp); 
 
//Mostrar archivo completo 
readfile($file); 
?>

karlingaz · #20 (**permalink**) 01/08/2009, 12:54

solo se esta escribiendo en el archivo el nombre, fecha y correo electrónico, sigue sin escribirse el comentario.
te dejo el link para que veas

http://www.iimae.com/daniela/index.php

karlingaz · #21 (**permalink**) 01/08/2009, 13:11

Ya quedo gjx2 muchas gracias por toda tu ayuda y paciencia

Te mando un cordial saludo.

gjx2 · #22 (**permalink**) 01/08/2009, 13:16

No hay problemas

No olvides poner

Código PHP:

Ver originalif (isset($_POST['comentario'] ) && !empty($_POST['nombre']) && !empty($_POST['email'])){

albert1980 · #23 (**permalink**) 01/08/2009, 13:24

no es por molestar pero que pasaría si un usuario introduce en su email esto:

Código:

" onclick="while(1) alert('hola');"

quedaría:
<a href="" onclick="while(1) alert('hola');">" onclick="while(1) alert('hola');"</a>

y cada vez que un usuario pulsaría en ese enlace se ejecutaría código javascript

Saludos :p

para sulucionarlo sustituir

$comentario = htmlspecialchars($comentario);

por

$comentario = htmlspecialchars($comentario, ENT_QUOTES);

MasterXNicoX · #24 (**permalink**) 01/08/2009, 14:59

Cita:

Iniciado por albert1980

no es por molestar pero que pasaría si un usuario introduce en su email esto:

Código:

" onclick="while(1) alert('hola');"

quedaría:
<a href="" onclick="while(1) alert('hola');">" onclick="while(1) alert('hola');"</a>

y cada vez que un usuario pulsaría en ese enlace se ejecutaría código javascript

Saludos :p

para sulucionarlo sustituir

$comentario = htmlspecialchars($comentario);

por

$comentario = htmlspecialchars($comentario, ENT_QUOTES);