Formulario en php

[Arkaitz]

Código PHP:

<?php
 
 $EmailFrom = "[email protected]";
 $EmailTo = "[email protected]";
 $Subject = "ANDES/HOTEL formulario contacto";
 $nombre = Trim(stripslashes($_POST['nombre']));
 $pais = Trim(stripslashes($_POST['pais']));
 $pasajeros = Trim(stripslashes($_POST['adultos']));
 $pasajeros = Trim(stripslashes($_POST['ninos']));
 $llegada = Trim(stripslashes($_POST['llegada']));
 $salida = Trim(stripslashes($_POST['salida'])) ;
 $tipo_caba = Trim(stripslashes($_POST['tipo_caba']));
 $fono = Trim(stripslashes($_POST['fono']));
 $mail = Trim(stripslashes($_POST['mail']));
 $comentarios = Trim(stripslashes($_POST['comentarios']));
 
 
 // validation
 $validationOK=true;
 if (!$validationOK) {
 print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
 exit;
 }
 
 // prepare email body text
 $Body = "";
 $Body .= "nombre: ";
 $Body .= $nombre;
 $Body .= "\n";
 
 $Body .= "pais: ";
 $Body .= $pais;
 $Body .= "\n";
 
 $Body .= "adultos: ";
 $Body .= $adultos;
 $Body .= "\n";
 
 $Body .= "niños: ";
 $Body .= $ninos;
 $Body .= "\n";
 
 $Body .= "llegada: ";
 $Body .= $llegada;
 $Body .= "\n";
 
 $Body .= "salida: ";
 $Body .= $salida;
 $Body .= "\n";
 
 $Body .= "tipo cabaña: ";
 $Body .= $tipo_caba;
 $Body .= "\n";
 
 $Body .= "fono: ";
 $Body .= $fono;
 $Body .= "\n";
 
 $Body .= "mail: ";
 $Body .= $mail;
 $Body .= "\n";
 
 $Body .= "comentarios: ";
 $Body .= $comentarios;
 $Body .= "\n";
 
 
 
 // send email 
 $success = mail($EmailTo, $Subject, $Body, "From: <$EmailFrom>");
 
 // redirect to success page 
 if ($success){
 print "<meta http-equiv=\"refresh\" content=\"0;URL=gracias.html\">";
 }
 else{
 print "<meta http-equiv=\"refresh\" content=\"0;URL=error.htm\">";
 }
 
 if(email==NULL||$fono==NULL) {
 ?>
 <script>
 alert("faltan campos por llenar!");
 location.href="contacto.html";//pagina donde esta tu formulario
 </script>
 
 <?php
 }
 if (!preg_match("/^[a-zA-Z0-9_\.\-]+@[a-zA-Z0-9\-]+\.[a-zA-Z0-9\-\.]+$/", $mail)) {
 exit('El formato del email ingresado no es correcto.');
 }
 

El if de la línea 80 debería estar antes de la línea 70 que es donde envías el mail. El código lo interpreta de arriba a abajo por eso envía el mail y luego te dice que hay campos por llenar.

y en la línea 80 sería:

Código PHP:

if ($email == NULL || $fono == NULL){