Consulta formulario duplicado

[paganiluciano]

Hola que tal soy nuevo en este foro, tengo un problema y me gustaria consultarlo con uds a ver si me pueden ayudar, tengo unos formularios en php en un servidor win, funcionan bien y me envia los datos correctamente, el problema surge ahora que quiero migrar el hosting a linux, la pagina funciona bien pero los datos del formulario me los envia duplicados, es decir me repite en el mismo email los datos.
Dejo aca abajo los codigos fuente del form. php y del mail.php
Desde ya muchas gracias por cualquier ayuda

http://www.paganiluciano.com.ar/prueba/negocioindep.rar

[shakaran]

Hola,

Primero recomendarte que postees el código en modo texto para la próxima vez, no es buena práctica poner el codigo así (sobre todo para futuras consultas) y más siendo solo dos archivos.

El error lo tienes en la linea 109 del archivo mail.php:

Código:

mail($sdestinatario,$ssubject,$shtml,$sheader);

Debes eliminarlo ya que previamente ya has enviado un email en la linea 78:

Código:

$sentmail = mail($to, $subject, $comentario, $headers);

Aparte hay otro fallo que llamas a la variable "mail" en la 111, pero no tiene un dolar delante para ser considerada variable de php:

Como bonus, te aconsejo cambiar los:

Código:

 echo" <meta http-equiv=refresh content=0;URL=gracias.html> ";

Por:

Código:

header('Location: gracias.html');

[paganiluciano]

Hola gracias por la pronta respuesta modifique lo que me aconsejaste, aca posteo como quedo corregido

<?php
$nombre = $_POST['nombre'];
$edad = $_POST['edad'];
$mail = $_POST['mail'];
$cb_pais = $_POST['cb_pais'];
$est = $_POST['est'];
$cp = $_POST['cp'];
$ciudad = $_POST['ciudad'];
$diurno = $_POST['diurno'];
$nocturno = $_POST['nocturno'];
$hora_contacto = $_POST['hora_contacto'];
$profesion = $_POST['profesion'];
$estudios = $_POST['estudios'];
$idiomas = $_POST['idiomas'];
$motivacion = $_POST['motivacion'];
$oe_ideal = $_POST['oe_ideal'];
$situacion_actual = $_POST['situacion_actual'];
$puesto_reciente = $_POST['puesto_reciente'];
$razones_negocio = $_POST['razones_negocio'];
$lider = $_POST['lider'];
$url = $_POST['url'];
$atractivos = $_POST['atractivos'];
$beneficios = $_POST['beneficios'];
$disp_economica = $_POST['disp_economica'];
$cb_equipamiento = $_POST['cb_equipamiento'];
$dedicacion = $_POST['dedicacion'];
$linea_te = $_POST['linea_te'];
$internet = $_POST['internet'];
$preferencias_contacto = $_POST['preferencias_contacto'];
$referido = $_POST['referido'];
$opinion = $_POST['opinion'];
$usuario = $_POST['usuario'];
$pertenecer = $_POST['pertenecer'];

$to = '[email protected]';
$subject = 'CONSULTA de negocioindep.com.ar';
$headers = 'From: [email protected]' . "\r\n" .
'Reply-To: [email protected]' . "\r\n" .
$comentario="Mi nombre es: ".$nombre."
Mi edad es: ".$edad."
Mi email es: ".$mail."
Pais: ".$cb_pais."
Estado o Provincia: ".$est."
Código Postal: ".$cp."
Ciudad: ".$ciudad."
Teléfono diurno o celular: ".$diurno."
Teléfono nocturno: ".$nocturno."
Mejor hora para localizarlo: ".$hora_contacto."
---------------------------------------------
Profesión: ".$profesion."
Estudios: ".$estudios."
Idiomas: ".$idiomas."
---------------------------------------------
Motivación: ".$motivacion."
Ingreso deseado: ".$oe_ideal."
Situación actual: ".$situacion_actual."
Puesto reciente: ".$puesto_reciente."
Razones del emprendimiento: ".$razones_negocio."
----------------------------------------------
Se siente lider: ".$lider."
Web: ".$url."
Que lo atrajo: ".$atractivos."
Beneficio buscado: ".$beneficios."
Disponibilidad Económica: ".$disp_economica."
Equipamiento que dispone: ".$cb_equipamiento."
Tiempo a dedicar: ".$dedicacion."
Cantidad de líneas telefónicas: ".$linea_te."
Tipo de conexion: ".$internet."
Preferencia de contacto: ".$preferencias_contacto."
Por donde llego al sitio: ".$referido."
Opinion del sitio: ".$opinion."
Conocimiento informático: ".$usuario."
-----------------------------------------------
Comentarios: ".$pertenecer;

'X-Mailer: PHP/' . phpversion();

$sentmail = mail($to, $subject, $comentario, $headers);

$sfrom="[email protected]";
$sdestinatario="$mail";
$ssubject="Respuesta de Sistema de Negocio Independiente - Siguiente Paso";
$shtml="<html>
<head>
<title>negocioindep.com.ar</title>
</head>
<body>
<table width='590' border='0'>
<tr>
<td><p align='left'><font size='3' face='Tahoma'><strong><img src='http://www.negocioindep.com.ar/topemail.gif' alt='negocioindep.com.ar' /><br />
<font size='4'>Gracias por ingresar a nuestro SISTEMA DE NEGOCIOS.</font></strong></font><br />
<br />
<font size='2' face='Tahoma'>La informaci&oacute;n que necesita ahora la encontrar&aacute; haciendo click en: </font><font size='2' face='Tahoma'><a href='http://www.negocioindep.com.ar/inicio' target='_blank'>PAQUETE DE INICIO</a>.</font><br />
<font size='2' face='Tahoma'>El contenido del mismo le ser&aacute; de utilidad para poder tomar una decisi&oacute;n </font><font size='2' face='Tahoma'>acertada para Ud. y los suyos, en base a conocer m&aacute;s el funcionamiento </font><font size='2' face='Tahoma'>y respaldo de nuestro Sistema de Negocio. </font></p>
<p align='left'><font size='2' face='Tahoma'>En breve ser&aacute; contactado por su asesor de Negocios para asistir </font><font size='2' face='Tahoma'>a la Pr&oacute;xima Presentaci&oacute;n.</font></p>
<p align='left'><font size='2' face='Tahoma'>A su Disposici&oacute;n, Atte.<br />
Depto. De Negocios Internacional</font><br />
<font size='2' face='Tahoma'>negocioindep.com.ar</font></p>
</td>
</tr>
</table>
</body>
</html>";
$sheader = "From: $sfrom\n"
. "Reply-To: $sfrom\n";
$sheader=$sheader."X-Mailer:PHP/".phpversion()."\n";
$sheader=$sheader."Mime-Version: 1.0\n";
$sheader=$sheader."Content-Type: text/html";


if($mail){
echo" <meta http-equiv=refresh content=0;URL=gracias.html> ";
}
else {
echo" <meta http-equiv=refresh content=0;URL=error.html> ";
}
?>

[shakaran]

Podrías haber cambiado tambien los meta refresh por el header location como te dije ;) Me alegro de que te haya funcionado.

[paganiluciano]

NO compa me olvide de poner que me sigue enviando los datos duplicados, ahh si ese me olvide ahora lo corrijo, pero me sigue haciendo exactamente lo mismo no entiendo

[shakaran]

Pues no deberia dar duplicados, en el form.php no parece haber problemas, prueba a cambiar los meta que te dije, el lo unico que podría estar fallando, aunque no tiene nada que ver.

[paganiluciano]

Iniciado por shakaran

Pues no deberia dar duplicados, en el form.php no parece haber problemas, prueba a cambiar los meta que te dije, el lo unico que podría estar fallando, aunque no tiene nada que ver.

gracias igual amigo quizas otro vea que es lo que pasa, pero muchas gracias nuevamente

[paganiluciano]

Ya esta cambiado el meta que me dijiste y le saque el envio que le hacia a la persona que completaba el formulario, todo para ver donde puede estar el error, ya que no entiendo porque envia 2 veces la cadena $comentario. Aca dejo el codigo depurado como quedo ahora pero lo sigue enviando duplicado

<?php
$nombre = $_POST['nombre'];
$edad = $_POST['edad'];
$mail = $_POST['mail'];
$cb_pais = $_POST['cb_pais'];
$est = $_POST['est'];
$cp = $_POST['cp'];
$ciudad = $_POST['ciudad'];
$diurno = $_POST['diurno'];
$nocturno = $_POST['nocturno'];
$hora_contacto = $_POST['hora_contacto'];
$profesion = $_POST['profesion'];
$estudios = $_POST['estudios'];
$idiomas = $_POST['idiomas'];
$motivacion = $_POST['motivacion'];
$oe_ideal = $_POST['oe_ideal'];
$situacion_actual = $_POST['situacion_actual'];
$puesto_reciente = $_POST['puesto_reciente'];
$razones_negocio = $_POST['razones_negocio'];
$lider = $_POST['lider'];
$url = $_POST['url'];
$atractivos = $_POST['atractivos'];
$beneficios = $_POST['beneficios'];
$disp_economica = $_POST['disp_economica'];
$cb_equipamiento = $_POST['cb_equipamiento'];
$dedicacion = $_POST['dedicacion'];
$linea_te = $_POST['linea_te'];
$internet = $_POST['internet'];
$preferencias_contacto = $_POST['preferencias_contacto'];
$referido = $_POST['referido'];
$opinion = $_POST['opinion'];
$usuario = $_POST['usuario'];
$pertenecer = $_POST['pertenecer'];

$to = '[email protected]';
$subject = 'CONSULTA de negocioindep.com.ar';
$headers = 'From: [email protected]' . "\r\n" .
'Reply-To: [email protected]' . "\r\n" .
$comentario="Mi nombre es: ".$nombre."
Mi edad es: ".$edad."
Mi email es: ".$mail."
Pais: ".$cb_pais."
Estado o Provincia: ".$est."
Código Postal: ".$cp."
Ciudad: ".$ciudad."
Teléfono diurno o celular: ".$diurno."
Teléfono nocturno: ".$nocturno."
Mejor hora para localizarlo: ".$hora_contacto."
---------------------------------------------
Profesión: ".$profesion."
Estudios: ".$estudios."
Idiomas: ".$idiomas."
---------------------------------------------
Motivación: ".$motivacion."
Ingreso deseado: ".$oe_ideal."
Situación actual: ".$situacion_actual."
Puesto reciente: ".$puesto_reciente."
Razones del emprendimiento: ".$razones_negocio."
----------------------------------------------
Se siente lider: ".$lider."
Web: ".$url."
Que lo atrajo: ".$atractivos."
Beneficio buscado: ".$beneficios."
Disponibilidad Económica: ".$disp_economica."
Equipamiento que dispone: ".$cb_equipamiento."
Tiempo a dedicar: ".$dedicacion."
Cantidad de líneas telefónicas: ".$linea_te."
Tipo de conexion: ".$internet."
Preferencia de contacto: ".$preferencias_contacto."
Por donde llego al sitio: ".$referido."
Opinion del sitio: ".$opinion."
Conocimiento informático: ".$usuario."
-----------------------------------------------
Comentarios: ".$pertenecer;

'X-Mailer: PHP/' . phpversion();

$sentmail = mail($to, $subject, $comentario, $headers);

if($mail){
header('Location: gracias.html');
}
else {
echo" <meta http-equiv=refresh content=0;URL=error.html> ";
}
?>

[shakaran]

Vamos a mejorar tu código un poco, en lugar de declarar tropecientas variables con $_POST y guardarlas, haces un foreach y las "generas" a pelo con:

Código PHP:

foreach($_POST as $k => $v) ${$k} = $v;
 

Prueba con esto ahora que ya esta "mejorado":

Código PHP:

<?
foreach($_POST as $k => $v) ${$k} = $v;
 
$comentario = 'Mi nombre es: '.$nombre.'
Mi edad es: '.$edad.'
Mi email es: '.$mail.'
Pais: '.$cb_pais.'
Estado o Provincia: '.$est.'
Código Postal: '.$cp.'
Ciudad: '.$ciudad.'
Teléfono diurno o celular: '.$diurno.'
Teléfono nocturno: '.$nocturno.'
Mejor hora para localizarlo: '.$hora_contacto.'
---------------------------------------------
Profesión: '.$profesion.'
Estudios: '.$estudios.'
Idiomas: '.$idiomas.'
---------------------------------------------
Motivación: '.$motivacion.'
Ingreso deseado: '.$oe_ideal.'
Situación actual: '.$situacion_actual.'
Puesto reciente: '.$puesto_reciente.'
Razones del emprendimiento: '.$razones_negocio.'
----------------------------------------------
Se siente lider: '.$lider.'
Web: '.$url.'
Que lo atrajo: '.$atractivos.'
Beneficio buscado: '.$beneficios.'
Disponibilidad Económica: '.$disp_economica.'
Equipamiento que dispone: '.$cb_equipamiento.'
Tiempo a dedicar: '.$dedicacion.'
Cantidad de líneas telefónicas: '.$linea_te.'
Tipo de conexion: '.$internet.'
Preferencia de contacto: '.$preferencias_contacto.'
Por donde llego al sitio: '.$referido.'
Opinion del sitio: '.$opinion.'
Conocimiento informático: '.$usuario.'
-----------------------------------------------
Comentarios: '.$pertenecer;
 
if(mail('[email protected]', 
        'CONSULTA de negocioindep.com.ar', 
        $comentario, "From: [email protected]\r\nReply-To: [email protected]\r\nX-Mailer: PHP/".phpversion()
          )
   )
    header('Location: gracias.html');
else header('Location: error.html');
?>
 

[paganiluciano]

puff el que sabe sabe y el que no pregunta, muchas gracias amigo me solucionaste quedarme sin pelo, un millon de gracias

[shakaran]

De nada, me "encanta" el karma ;)

[abimaelrc]

Un pequeño detalle, si lo piensas usar así y no vas a hacer nada con el foreach, si no solamente usar el concepto variable variables, puedes usar extract que hace exactamente lo mismo y en una pequeña línea.

Código PHP:

<?php
extract($_POST);
 

[shakaran]

Iniciado por abimaelrc

Un pequeño detalle, si lo piensas usar así y no vas a hacer nada con el foreach, si no solamente usar el concepto variable variables, puedes usar extract que hace exactamente lo mismo y en una pequeña línea.

Código PHP:

<?php
extract($_POST);
 

Toma ya! Esa me la apunto yo que no la sabía ;)

[paganiluciano]

Muchas gracias por la gran ayuda brindada
ahora una consulta mas yo en el mail.php original hacia que le enviara un email a la persona que completaba el formulario, usaba este codigo

$sfrom="[email protected]";
$sdestinatario="$mail";
$ssubject="Respuesta de Sistema de Negocio Independiente - Siguiente Paso";
$shtml="<html>
<head>
<title>negocioindep.com.ar</title>
</head>
<body>
<table width='590' border='0'>
<tr>
<td><p align='left'><font size='3' face='Tahoma'><strong><img src='http://www.negocioindep.com.ar/topemail.gif' alt='negocioindep.com.ar' /><br />
<font size='4'>Gracias por ingresar a nuestro SISTEMA DE NEGOCIOS.</font></strong></font><br />
<br />
<font size='2' face='Tahoma'>La informaci&oacute;n que necesita ahora la encontrar&aacute; haciendo click en: </font><font size='2' face='Tahoma'><a href='http://www.negocioindep.com.ar/inicio' target='_blank'>PAQUETE DE INICIO</a>.</font><br />
<font size='2' face='Tahoma'>El contenido del mismo le ser&aacute; de utilidad para poder tomar una decisi&oacute;n </font><font size='2' face='Tahoma'>acertada para Ud. y los suyos, en base a conocer m&aacute;s el funcionamiento </font><font size='2' face='Tahoma'>y respaldo de nuestro Sistema de Negocio. </font></p>
<p align='left'><font size='2' face='Tahoma'>En breve ser&aacute; contactado por su asesor de Negocios para asistir </font><font size='2' face='Tahoma'>a la Pr&oacute;xima Presentaci&oacute;n.</font></p>
<p align='left'><font size='2' face='Tahoma'>A su Disposici&oacute;n, Atte.<br />
Depto. De Negocios Internacional</font><br />
<font size='2' face='Tahoma'>negocioindep.com.ar</font></p>
</td>
</tr>
</table>
</body>
</html>";
$sheader = "From: $sfrom\n"
. "Reply-To: $sfrom\n";
$sheader=$sheader."X-Mailer:PHP/".phpversion()."\n";
$sheader=$sheader."Mime-Version: 1.0\n";
$sheader=$sheader."Content-Type: text/html";
mail($sdestinatario,$ssubject,$shtml,$sheader);

ahora puedo usar el mismo o deberia cambiarlo
Gracias

[paganiluciano]

listo ya lo solucione gracias nuevamente