Validar captcha antes de enviar email

[kimmy]

Hola, tengo un mini formulario en el home que permite enviar un mensaje pero quiero agregar un captcha y no me valida. El form está aqui:

http://vamonosdevacaciones.com.ve/

Este es el código del form:

Código PHP:

$code = trim($_POST['code']);
if(isset($_POST['save']))
    {
        if($_SESSION['captcha'] == $code)
        {
            echo "<span style='color:green'>Es correcto el captcha ingresado</span>";
        }
        else
        {
            echo "<span style='color:red'>Es incorrecto el captcha ingresado</span>";
        }   
    }
 
 
        <form method="post" action="envio.php" id="myform" name="myform" onSubmit="return limpiar()" >
        <fieldset id="caja_mensaje">
        <legend>Contacto</legend>
        <ol>
            <li><label>Nombre:</label><input type="text" size="30" name="fname" /></li>
            <li><label>E-mail:</label><input type="text" size="30" name="femail" /></li>
            <li><label>Mensaje:</label><textarea name="fcomment" id="fcomment" rows="3" cols="25" onkeyup="limitarCaracter(this.id,80,'charTexto');"></textarea></li>
            <li><div style="font-size:10px;" id="charTexto"></div></li><br />
            <li><label></label><img src="captcha.php" width="100" height="30" vspace="3"></li><br />
            <li><input name="code" type="text"/></li>
            <li><input type="submit" name="save" value="Enviar"></li>
        </ol>
      </fieldset>
</form>
 

este es el archivo de envio.php:

Código PHP:

$mensaje="";
$mensaje.="Contacto desde:". "\n\n";
$mensaje.="Nombre: ".$_POST['fname']."\n";
$mensaje.="E-mail: ".$_POST['femail']."\n";
$mensaje.="Mensaje: ".$_POST['fcomment']."\n";
// definimos a quien se lo enviamos
$email_destiny="[email protected]";
$subject="Mensaje";
// verificamos si se envió
if (mail($email_destiny,$subject,$mensaje,"From: Contact<".$_POST['femail'].">")) {
    echo '<p align="center" style="color:#263e88; font-family:Arial; font-size:11px;"><b>Gracias por su mensaje</b></p>';
} else {
    echo '<p align="center">Error '.$_POST['fname'].'</p>';
}
 

y este es el codigo captcha.php:

Código PHP:

session_start();
function texto($ancho)
{
     $cadena = "0123456789abcdefghijkmnñlopqrstwxyz";
     for($i = 0; $i<$ancho;$i++)
     {
          $muestra .= $cadena{rand(0,35)};
     }
     return $muestra;
}
 
// Define el ancho del texto usando un parametro (5) para la función creada anteriormente.
$_SESSION['captcha'] = texto(5);
 
// La imagen gif de fondo.
$captcha = imagecreatefromgif("images/bgcaptcha.gif");
 
// La localizacion de la imagen, lo dejamos en 0,0,0
$letras = imagecolorallocate($captcha,0,0,0);
 
// Unir el texto en la imagen gif creada.
imagestring($captcha,5,16,7,$_SESSION['captcha'],$letras);
 
// Pone la imagen en cabezera.
header("Content-type: images/gif");
 
// Muestra la imagen.
imagegif($captcha);
 

No entiendo porque no funciona.

Será que tengo que poner todo el código en envio.php?

y si es así, cómo debo colocarlo???

Gracias

[maxpower2008]

La validacion la debes hacer en la pagina q recibe los valores
obviamente debe ser en el archivo envio.php

Código PHP:

if(isset($_POST['save']))
    {
        if($_SESSION['captcha'] == $code)
        {
            echo "<span style='color:green'>Es correcto el captcha ingresado</span>";
//realizas todo lo del envio
        }
        else
        {
            echo "<span style='color:red'>Es incorrecto el captcha ingresado</span>";
        }  
    }
 

Esto debe ir en envio.php

[kimmy]

hola, lo puse como me indicaste pero no valida nada. Si quieres puedes probarlo...

[Dradi7]

OJO que tu codigo de email debe ir en la condicional en donde verifica que el captcha es correcto

Saludos

[maxpower2008]

Iniciado por kimmy

hola, lo puse como me indicaste pero no valida nada. Si quieres puedes probarlo...

el session_start();
tambien lo pones en el archivo envio.php??...

[kimmy]

antes no lo tenia pero ahora lo puse. Este es el código que no me funciona:

Código PHP:

session_start();
$code = trim($_POST['code']);
if(isset($_POST['save']))
    {
        if($_SESSION['captcha'] == $code)
        {
            echo "<span style='color:green'>Es correcto el captcha ingresado</span>";
            
        // definimos el mensaje
$mensaje="";
$mensaje.="Contacto desde:". "\n\n";
$mensaje.="Nombre: ".$_POST['fname']."\n";
$mensaje.="E-mail: ".$_POST['femail']."\n";
$mensaje.="Mensaje: ".$_POST['fcomment']."\n";
// definimos a quien se lo enviamos
$email_destiny="[email protected]";
$subject="Mensaje";
// verificamos si se envió
if (mail($email_destiny,$subject,$mensaje,"From: Contact<".$_POST['femail'].">")) {
    echo '<p align="center" style="color:#263e88; font-family:Arial; font-size:11px;"><b>Gracias por su mensaje</b></p>';
} else {
    echo '<p align="center">Error '.$_POST['fname'].'</p>';
}
    
            
        }
        else
        {
            echo "<span style='color:red'>Es incorrecto el captcha ingresado</span>";
        }   
    }
 

[Dradi7]

para mi el error pude estar en el $_POST['save'] verifica que te devuelve este valor realizando un echo

[Triby]

Conmigo no funciona el formulario, el botón enviar ni pío dice!

Iniciado por Firefox -> Consola de errores

Fecha y hora: 25/09/2012 04:05:53 p.m.
Error: TypeError: document.formulario is undefined
Archivo de origen: http://vamonosdevacaciones.com.ve/
Línea: 307

[kimmy]

Hola Triby!!! Si, el formulario no funciona. Antes de poner lo del captcha funcionaba pero ahora no. Por qué????

[Triby]

No es el captcha, sino que tienes un conflicto de Javascripts, por un lado la función limpiar() que busca un formulario inexistente y, por otro lado, que usas jQuery para enviar los datos serializados (?).

Lo primero que deberías hacer es verificar qué estás recibiendo en $_POST, lo que tienes en $_SESSION y analizar el siguiente paso en base a eso.

PHP para procesar el formulario:

Código PHP:

<?php
session_start();
 
var_dump($_POST, $_SESSION);
 

Los campos vienen como los esperas?, todos o falta alguno?

[kimmy]

Gracias. Arreglé la función limpiar y realice lo que me indicaste y me arroja esto:

array(4) {
["fname"]=> string(5) "Kimmy"
["femail"]=> string(18) "[email protected]"
["fcomment"]=> string(5) "fdfdf"
["code"]=> string(0) "" }
array(1) {
["captcha"]=> string(5) "bcngt"
}

[maxpower2008]

Iniciado por kimmy

Gracias. Arreglé la función limpiar y realice lo que me indicaste y me arroja esto:

array(4) {
["fname"]=> string(5) "Kimmy"
["femail"]=> string(18) "[email protected]"
["fcomment"]=> string(5) "fdfdf"
["code"]=> string(0) "" }
array(1) {
["captcha"]=> string(5) "bcngt"
}

ahi te puedes dar cuenta que el code esta llegando vacio, revisalo para ver por q.

[Dradi7]

bueno yo estoy probando tu codigo desde la misma web y ahora si me toma los valores de code

[kimmy]

en el code si pasan los datos lo que pasó fue que no le coloqué nada para ver si daba error en el captcha

[Dradi7]

ya me di cuenta cual es tu error.

Tu error esta en el JavaScript cuando tu usas esto en el ajax

$('#myform').serialize() ya que no esta tomando el valor del input submit es por esto que cuando en tu codigo php estas haciendo esto

if(isset($_POST['save']))

nunca entra y no realiza nada trata de cambiarlo por otra variable para que lo valides

[maxpower2008]

Iniciado por kimmy

en el code si pasan los datos lo que pasó fue que no le coloqué nada para ver si daba error en el captcha

Pues ya esta, ahora solo es enviar

[kimmy]

Gracias a todos, ahora si funciona. Le quité el if(isset($_POST['save'])).

[Triby]

Algo está mal con tu captcha, porque usa cadenas con caracteres "raros", a mí me apareció esto: �g�qk

Por cierto, si usas firefox, instala la extensión HTML Validator para que puedas ver y corregir los errores que tienes en el código; esto ayuda a tratar de mantener compatibilidad entre navegadores y te evita muchos dolores de cabeza.

Por ejemplo, estás agregando dos estilos css con la etiqueta <link ..> justo después de </head>, deberían estar antes.

[Dradi7]

de nada solo aclarar que debes tener cuidado porque a veces el error esta en la parte cliente y no en el servidor es por eso que debes aclarar como estas enviando el formulario

Saludos

[maxpower2008]

cierto esa cadena de capcha te genera caracteres especiales.
quita la ñ de la cadena.

$cadena = "0123456789abcdefghijkmnñlopqrstwxyz";

[Dradi7]

otro error esta tambien que no es $cadena{rand(0,35)}; sino 33 ya que la cadena tiene como longitud 34 pero como el indice empieza en 0 debe ser 33 asi debe quedar

$cadena{rand(0,33)};

[kimmy]

ok maxpower, ya quité la ñ. Triby voy a instalar la extensión que me dices porque tengo un desastre con los css. Mil gracias a ambos.

[kimmy]

Gracias Dradi, también lo corregiré...

[kimmy]

Una pregunta mas: Cómo hago para que cuando el usuario no coloque lo del captcha al presionar enviar no se borren los datos del form y solo se borren si está correcto?

[Dradi7]

en tu form cambia esto

<form method="post" action="envio.php" id="myform" name="myform" onSubmit="return limpiar()" >

por esto

<form method="post" action="envio.php" id="myform" name="myform" >

Solo debes quitar el onsubmit

[kimmy]

No funciona asi. debería ser con alguna condicion.

[maxpower2008]

Iniciado por kimmy

No funciona asi. debería ser con alguna condicion.

en cada campo valida con isset()
algo mas o menos asi
echo "<input type='text' name='nombre' ";
if(isset($_POST['nombre'])) {
echo "value ='".$_POST['nombre']."' ;
}
echo " />";
Esta medio chapuza pero algo asi te serviria

[kimmy]

lo intentaré a ver que pasa. Gracias...