[SOLUCIONADO] Problema con formulario de contacto

[dicarlo]

No doy con la tela, no se que es lo que falla, no entiendo mucho de PHP, agradecería enormemente la ayuda, aquí os dejo el código:

<?php

$mail='[email protected]';



$apo_name = $_POST['apo_name'];
$apo_phone = $_POST['apo_phone'];
$apo_email = $_POST['apo_email'];
$apo_date = $_POST['apo_date'];
$message = $_POST['message'];


$message = "
<p><img src='images/logo2.png' alt='' /></p>
<p><br /></p>
<p>Los datos recibidos son:</p>
<p>
nombre:" .$apo_name."<br />
dia de la cita:" .$apo_date."<br />
telefono:" .$apo_phone."<br />
email:" .$apo_email."<br /><br />
mensaje:" .$message;



$headers = "MIME-Version: 1.0\n";
$headers .= "Content-type: text/html; charset=iso-8859-1\n";
$headers .= "From: ".$apo_email."\n";
$headers .= "Reply-To: ".$apo_email."\n";
// y luego

if mail($apo_mail,"Formulario de Consulta",$message,$headers)
{
echo "Muchas gracias por realizar su consulta. En breve le responderemos en la dirección que nos ha indicado en el
formulario";
}
else {
echo "Oops!. Se produjo un error al procesar lo solicitado. Vuelve a intentarlo en unos minutos";
}
}
?>

[repara2]

No dices cual es el error pero sobreescribes ma variable $message, salu2

[dicarlo]

Gracias por responder repara2, el error que me da es el siguiente:

Parse error: syntax error, unexpected T_STRING, expecting '(' in /htdocs/public/www/demo/vitaller/submit.php on line 33

linea 33= if mail($apo_mail,"Formulario de Consulta",$message,$headers)

Gracias de nuevo por ayudarme.

[nicolasberro]

El if debe tener un parentesis:

if(mail($mail)){
//codigo;
}

[dicarlo]

Iniciado por nicolasberro

El if debe tener un parentesis:

if(mail($mail)){
//codigo;
}

Muchas gracias NicolasBerro, ya no me da el error... me sale el mensaje de echo de que esta todo bien, pero no me llega el correo!!! y el mail esta bien puesto

[nicolasberro]

Bueno viendolo simplemente aqui no le estas enviando el mail a nadie.

Tienes tres variables de mails: $apo_email, $apo_mail y $mail... Definete cual usaras y que quieres hacer con cada una. (CUIDADO CON LOS ERRORES DE TIPEO):

Código PHP:

  <?php
if(mail($apo_mail,$x,$x1,$x2)){
    // LA VARIABLE $APO_MAIL NO ESTA DEFINIDA
    //codigo
}
?>   

Considero que tienes una forma muy directa de trabajar. Tal vez haciendo mas comprobaciones separadas (imprimiento cosas como "Este paso n va bien") podras asegurarte de que funcione, y luego si todo es correcto puedes aplicar la version corta.

Luego, al final del mensaje, te quedara: "mensaje:"
Te sugiero que asignes la variable mensaje y luego la pegues donde corresponda, no concatenes todo.

PD: Keep it simple(mantenlo simple) no significa mantenlo corto! Cuidado con eso.

[dicarlo]

Solucionado, gracias a todos por la ayuda, así da gusto!!!

[repara2]

debe ser:

if (mail($apo_mail,"Formulario de Consulta",$message,$headers)) {
//lo que sea
}