formulario en php5

elita24 · #1 (**permalink**) 19/12/2008, 11:17

Hola:

Tengo un problemita con mi forumlario, pues ahora que el servidor esta con php5 y mi formulario esta en php4, pues ahora no funciona necesito que me ayuden porfavor es urgente..

Gracias...

este es el codigo en mi respuesta.php

<?
if ($validar=="comprobar")
?>
<?
$display="[email protected]";
$asunto = "Has Recibido un correo de ". $_GET['email'];
$cuerpo="Nombres: ".$_GET['nombres']." Apellidos: ".$_GET['apellidos']." Telefono: ".$_GET['telefono']." 
Email : ".$_GET['email']. " Comentarios: ".$_GET['comentarios'];

//para el envio en formato HTML
$headers = "MIME-Version: 1.0\r\n";
$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";

//direccion del remitente
$headers .= "From: ". $_GET['email'];

mail($display,$asunto,$cuerpo,$headers)
?>


<? echo $nombres ?> 
Gracias por tus comentarios ...

</center>
<?
//Envio de Correo a Usuario
$respuesta=" <a href=http://www.angelica.com.pe><img src=http://www.prueba.com.pe/iconos/email.jpg width=600 height=134 border=0></a>";
mail("$_GET['email']","Gracias por escribir a Prueba",$respuesta,"From: [email protected]\r\nContent-type: text/html;\nX-Mailer:PHP/", '-f $email');
?>

pateketrueke · #2 (**permalink**) 19/12/2008, 11:22

debes cambiar <? por <?php y usar $_POST['validar'] en lugar de $validar, etc, etc...

deberías leer acerca de register_globals y short_open_tags

demás, esta bien que tu if no tenga un bloque ??? (llaves { } )

osea... así debería ser

Código PHP:

  if (/* ... */)
{
  // ...
}

lo digo, por que al faltar el bloque... solo ejecuta la primer sentencia después del if, condicionada al if ... y el resto, sin importar la condición pues... esta fuera del bloque !!!?

franklin85 · #3 (**permalink**) 19/12/2008, 11:43

De esta forma debe de salir:

Código PHP:

  <?
if (isset($HTTP_GET_VARS["nombres"])){ $nombres=$HTTP_GET_VARS["nombres"];}else{ $nombres="";}
if (isset($HTTP_GET_VARS["apellidos"])){ $apellidos=$HTTP_GET_VARS["apellidos"];}else{ $apellidos="";}
if (isset($HTTP_GET_VARS["telefono"])){ $telefono=$HTTP_GET_VARS["telefono"];}else{ $telefono="";}
if (isset($HTTP_GET_VARS["email"])){ $email=$HTTP_GET_VARS["email"];}else{ $email="";}
if (isset($HTTP_GET_VARS["comentarios"])){ $comentarios=$HTTP_GET_VARS["comentarios"];}else{ $comentarios="";}

if ($validar=="comprobar")
?>
<?
$display="[email protected]";
$asunto = "Has Recibido un correo de ". $email;
$cuerpo="<b>Nombres:</b> ".$nombres."<br><b>Apellidos:</b> ".apellidos."<br><b>Telefono:</b> ".$telefono."<br>
<b>Email :</b> ".$email. "<br><b>Comentarios:</b> ".$comentarios;

//para el envio en formato HTML
$headers = "MIME-Version: 1.0\r\n";
$headers .= "Content-type: text/html; charset=iso-8859-1\r\n";

//direccion del remitente
$headers .= "From: ". $email;

mail($display,$asunto,$cuerpo,$headers)
?>

pateketrueke · #4 (**permalink**) 19/12/2008, 12:26

estamos hablando de un buen PHP5 .... y eso, es sin usar variables largas (HTTP_*_VARS)

deberías usar $_GET en lugar de $HTTP_GET_VARS, etc, etc...

(no hagamos malos hábitos, que por eso vuelven los problemas)

joanfont · #5 (**permalink**) 19/12/2008, 13:47

supongo que tu correo és [email protected]

porque sino deberias cambiarlo en email

Ronruby · #6 (**permalink**) 19/12/2008, 22:10

Se olvidan de esta parte:

Código PHP:

 
if ($validar=="comprobar")
?>
<?

En vez de $validar, usa $_GET['validar']
¿Para que cierras y abres las etiquetas de PHP?

Y no uses GET como metodo de formulario, usa POST (Usa $_POST para recoger los datos).