Ayuda a encontrar fallo en codigo

ElJudas · #1 (**permalink**) 01/03/2011, 12:01

Hola, tengo este código:

Código PHP:

Ver original<?php 
# FileName="Connection_php_mysql(dot)htm"
# Type="MYSQL"
# HTTP="true"
$hostname = "localhost";
$username = "user";
$password = "pwd";
$nombre = $_POST['nombre'];  
$telefono = $_POST['telefono'];
$dia_mes = $_POST['dia_mes'];
$hora = $_POST['hora'];
$cita = "('$dia_mes', '$hora')";
 
$conx = mysql_pconnect($hostname, $username, $password) or trigger_error(mysql_error(),E_USER_ERROR); 
$bd = "gspyafmr_AGENDA2011";
mysql_select_db("gspyafmr_AGENDA2011", $conx);
$novacita = "INSERT INTO [citas] ([hora], [nombre], [telefono] VALUES ('$cita', '$nombre', '$telefono')";
$peticion = mysql_query ($novacita, $conx);
 
$header = 'From: ' . $nombre . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje .= "Nombre: " . $nombre . " \r\n";
$mensaje .= "Teléfono: " . $telefono . " \r\n"; 
$mensaje .= "Dia ". $dia_mes . " \r\n";
$mensaje .= "Hora ". $hora . " \r\n"; 
$mensaje .= "Cita pedida el " . date('d/m/Y', time()); 
 
 
$para = "[email protected]"; 
$asunto = 'Cita previa'; 
 
 
mail($para, $asunto, utf8_decode($mensaje), $header); 
 
 
?> 
</h2> 
<h2 align="center">Gracias!</h2> 
 
<p align="center">Tu cita ha sido registrada correctamente.</p> 
<form id="form" name="form" method="post" action="">
  <label>
    <div align="center">
      <input type="button" name="Cerrar" id="Cerrar" value="Cerrar" onclick="window.close();"/>
    </div>
  </label>
</form>
<p align="center"> </p> 
<p><span style="color:red;font-size:150%;font-weight:bold;"><?php print $email; ?></span></p>
<noscript>
</noscript>

alguien podría hecharme una mano a ver dode está el fallo, es el archivo enviar.php de un form y la idea es que envie los datos por mail (eso funciona bien) y los introduzca en la bd mysql.

Gracias a todos.

ocp001a · #2 (**permalink**) 01/03/2011, 12:23

Y cuál es el problema? te da algún error?

ElJudas · #3 (**permalink**) 01/03/2011, 12:59

No me inserta los datos en la BD
y gracias por la rapida respuesta ;)

bUllan9ebrio · #4 (**permalink**) 01/03/2011, 13:38

como duda , para que es esto???

([hora], [nombre], [telefono]

no es lo mismo que ???

(hora,nombre,telefono)

y asi de pasada te digo que falta cerrar ese parentesis

te lo destaque en Rojo

ElJudas · #5 (**permalink**) 01/03/2011, 13:55

Es para guardar citas en un bd (estoy abierto a sugerencias y mejoras jeje), los [, los he puesto por que son nombre de columnas ( no se si es correcto, corrígeme si me equivoco please).

No me di cuenta de ese paréntesis, probare y comento, gracias de nuevo.

bUllan9ebrio · #6 (**permalink**) 01/03/2011, 14:04

Prueba colocando esto:

Código MySQL:

Ver originalINSERT INTO citas (hora,nombre,telefono) VALUES ('$cita', '$nombre', '$telefono')

ElJudas · #7 (**permalink**) 02/03/2011, 10:02

Lo he intentado y nada, no será alguna coma o doble coma que no debe estar o donde hay simple debe llevar doble?? o me falta alguna orden o algo, no es normal, por favor echen un vistazo a ver si ven algo raro.
Gracias

ElJudas · #8 (**permalink**) 02/03/2011, 10:24

Esto estaría bien???

Código PHP:

Ver original$novacita = "INSERT INTO citas (hora,nombre,telefono) VALUES ('$cita', '$nombre', '$telefono')";
mysql_query ($novacita, $conx);

o como tendría que hacer para llamar a

Código PHP:

Ver original($novacita(que es in INSERT), $cons)

s00rk · #9 (**permalink**) 02/03/2011, 10:56

Bueno a como he visto guardas solo como variable, pero aveces esto podria estar mal o para mi lo esta porque no se ejecuta sino mas bien se guarda en ella

La forma correcta a mi parecer seria asi, o si tambien cambie algo porque declaras variables y ni las usas :/

Código PHP:

Ver original$bd = "gspyafmr_AGENDA2011";
mysql_select_db($bd, $conx);
$novacita = "INSERT INTO ([citas] ([hora], [nombre], [telefono]) VALUES ('$cita', '$nombre', '$telefono')";
 
mysql_query ($novacita, $conx);

Asi es como deveria estar a mi opinion.

ElJudas · #10 (**permalink**) 02/03/2011, 13:54

ok lo probaré, hay un ( que sobra. que ya lo he revisado y está la variable $peticion que sobra, porque iba a usarla y al final no la usé, le he eliminado también, citas lo tengo sin[] porque es el nombre de la tabla, he usado $bd en la función y sigue igual que antes, envía el mail con los datos, pero ni añade los datos a la BD ni tampoco me sale ningún mensaje de error, es como si no hubiera ni intentado el conectarse,

ayuda please

Dejo el código del form por si alguno quiere probarlo, a ver si hay suerte y le apetece a alguien

;)

Gracias a todos.

s00rk · #11 (**permalink**) 02/03/2011, 22:12

Volvi a revisar el codigo y si en el que yo te puse hay 2 errores
1.- No colocamos el nombre de la tabla a la que se ingresaran los datos
2.- No ingresamos la fecha

Otra cosa no entiendo que intentas hacer con esto

Código:

$cita = "('$dia_mes', '$hora')";

ElJudas · #12 (**permalink**) 03/03/2011, 10:14

Simplemente insertar los datos de un formulario y unos datos de la pagina que ha ejecutado el form, esta información me llega bien por mail, pero no consigo que los datos se ingresen en la bd.

La variable $cita, recoge día mes y la hora seleccionada para abrir el form.

PD. Alguien sabe una buena manera de crear una agenda donde hay que introducir los datos desde el form, por php y como estructurar la BD sql????(es lo que estoy intentado pero no se si lo he estructurado bien).

GRACIAS!

ElJudas · #13 (**permalink**) 03/03/2011, 13:39

He modificado un poco el código para saber donde fallaba exactamente, he puesto un if en cada conexión y conecta bien con el host pero el mensaje de error que da, es de la conexión a la DB.

Dejo el código por si le pueden echar un vistazo,

Código PHP:

Ver original$conx2 = mysql_select_db($bd, $conx);
$bd = "AGENDA2011";
mysql_select_db($bd, $conx);
if (!$conx2) {
    die('No conecta a la BD: ' . mysql_error());
    }
echo 'Conectado  satisfactoriamente2';

Gracias

s00rk · #14 (**permalink**) 03/03/2011, 14:58

Bueno ahora si ya revise mejor esto y asi deveria ya funcionar:

Código PHP:

Ver original<?php 
# FileName="Connection_php_mysql(dot)htm"
# Type="MYSQL"
# HTTP="true"
$hostname = "localhost";
$username = "user";
$password = "pwd";
$bd = "gspyafmr_AGENDA2011";
$nombre = $_POST['nombre'];  
$telefono = $_POST['telefono'];
$dia_mes = $_POST['dia_mes'];
$hora = $_POST['hora'];
$cita = $dia_mes."-".$hora;
 
$conx = mysql_connect($hostname, $username, $password) or trigger_error(mysql_error(),E_USER_ERROR); 
mysql_select_db($bd, $conx);
$novacita = "INSERT INTO citas ([hora], [nombre], [telefono]) VALUES ('$cita', '$nombre', '$telefono')";
mysql_query ($novacita, $conx);
 
$header = 'From: ' . $nombre . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje .= "Nombre: " . $nombre . " \r\n";
$mensaje .= "Teléfono: " . $telefono . " \r\n"; 
$mensaje .= "Dia ". $dia_mes . " \r\n";
$mensaje .= "Hora ". $hora . " \r\n"; 
$mensaje .= "Cita pedida el " . date('d/m/Y', time()); 
 
 
$para = "[email protected]"; 
$asunto = 'Cita previa'; 
 
 
mail($para, $asunto, utf8_decode($mensaje), $header); 
 
 
?> 
</h2> 
<h2 align="center">Gracias!</h2> 
 
<p align="center">Tu cita ha sido registrada correctamente.</p> 
<form id="form" name="form" method="post" action="">
  <label>
    <div align="center">
      <input type="button" name="Cerrar" id="Cerrar" value="Cerrar" onclick="window.close();"/>
    </div>
  </label>
</form>
<p align="center"> </p> 
<p><span style="color:red;font-size:150%;font-weight:bold;"><?php print $email; ?></span></p>
<noscript>
</noscript>

SirDuque · #15 (**permalink**) 03/03/2011, 16:17

Usa esto:

mysql_query() or die (error());

ElJudas · #16 (**permalink**) 03/03/2011, 17:02

Muchas gracias a todos sois unos/as cracks ya introduje los datos, pero no se porqué se insertan doble en dos filas y en dos id.
Pongo como a quedado ;)

Código PHP:

Ver original<?php 
# FileName="Connection_php_mysql(dot)htm"
# Type="MYSQL"
# HTTP="true"
$hostname = "localhost:0000";
$username = "usuario";
$password = "contraseña";
$bd = "base de datos";
$nombre = $_POST['nombre'];  
$telefono = $_POST['telefono'];
$dia_mes = $_POST['dia_mes'];
$hora = $_POST['hora'];
$cita = $dia_mes."-".$hora;
 
$conx = mysql_connect($hostname, $username, $password);
if (!$conx) {
    die('No pudo conectarse: ' . mysql_error());
    }
$conx2 = mysql_select_db($bd, $conx);
mysql_select_db($bd, $conx);
if (!$conx2) {
    die('No conecta a la BD: ' . mysql_error());
    }   
$novacita = "INSERT INTO citas (hora ,nombre, telefono) VALUES ('$cita', '$nombre', '$telefono')";
$conx3 = mysql_query($novacita, $conx);
mysql_query($novacita, $conx);
if (!$conx3) {
    die('Entrada NO realizada: ' . mysql_error());
    }   
 
$header = 'From: ' . $nombre . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje .= "Nombre: " . $nombre . " \r\n";
$mensaje .= "Teléfono: " . $telefono . " \r\n"; 
$mensaje .= "Dia ". $dia_mes . " \r\n";
$mensaje .= "Hora ". $hora . " \r\n"; 
$mensaje .= "Cita pedida el " . date('d/m/Y', time()); 
 
 
$para = "[email protected]"; 
$asunto = 'Cita previa'; 
 
 
mail($para, $asunto, utf8_decode($mensaje), $header); 
 
 
?> 
</h2> 
<h2 align="center">Gracias!</h2> 
 
<p align="center">Tu cita ha sido registrada correctamente.</p> 
<form id="form" name="form" method="post" action="">
  <label>
    <div align="center">
      <input type="button" name="Cerrar" id="Cerrar" value="Cerrar" onclick="window.close();"/>
    </div>
  </label>
</form>
<p align="center"> </p> 
<p><span style="color:red;font-size:150%;font-weight:bold;"><?php print $email; ?></span></p>
<noscript>
</noscript>

gracias

s00rk · #17 (**permalink**) 04/03/2011, 03:10

Esque lo que pasa ejecutas dos veces la query

Código PHP:

Ver original$conx3 = mysql_query($novacita, $conx);
mysql_query($novacita, $conx);
if (!$conx3) {
    die('Entrada NO realizada: ' . mysql_error());
   }

Una primero donde solo dice
mysql_query($novacita, $conx); y luego otra vez la ejecuta donde esta la condicion if(!$conx3)

Simplemente elimina
mysql_query($novacita, $conx);
y deja lo demas y funcionara n_n

En total quedaria asi solo cambiando eso

Código PHP:

Ver original<?php 
# FileName="Connection_php_mysql(dot)htm"
# Type="MYSQL"
# HTTP="true"
$hostname = "localhost:0000";
$username = "usuario";
$password = "contraseña";
$bd = "base de datos";
$nombre = $_POST['nombre'];  
$telefono = $_POST['telefono'];
$dia_mes = $_POST['dia_mes'];
$hora = $_POST['hora'];
$cita = $dia_mes."-".$hora;
 
$conx = mysql_connect($hostname, $username, $password);
if (!$conx) {
    die('No pudo conectarse: ' . mysql_error());
    }
$conx2 = mysql_select_db($bd, $conx);
mysql_select_db($bd, $conx);
if (!$conx2) {
    die('No conecta a la BD: ' . mysql_error());
    }   
$novacita = "INSERT INTO citas (hora ,nombre, telefono) VALUES ('$cita', '$nombre', '$telefono')";
$conx3 = mysql_query($novacita, $conx);
if (!$conx3) {
    die('Entrada NO realizada: ' . mysql_error());
    }   
 
$header = 'From: ' . $nombre . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje .= "Nombre: " . $nombre . " \r\n";
$mensaje .= "Teléfono: " . $telefono . " \r\n"; 
$mensaje .= "Dia ". $dia_mes . " \r\n";
$mensaje .= "Hora ". $hora . " \r\n"; 
$mensaje .= "Cita pedida el " . date('d/m/Y', time()); 
 
 
$para = "[email protected]"; 
$asunto = 'Cita previa'; 
 
 
mail($para, $asunto, utf8_decode($mensaje), $header); 
 
 
?> 
</h2> 
<h2 align="center">Gracias!</h2> 
 
<p align="center">Tu cita ha sido registrada correctamente.</p> 
<form id="form" name="form" method="post" action="">
  <label>
    <div align="center">
      <input type="button" name="Cerrar" id="Cerrar" value="Cerrar" onclick="window.close();"/>
    </div>
  </label>
</form>
<p align="center"> </p> 
<p><span style="color:red;font-size:150%;font-weight:bold;"><?php print $email; ?></span></p>
<noscript>
</noscript>

ElJudas · #18 (**permalink**) 07/03/2011, 09:52

Muchas gracias, ahora envia un solo registro, pensé que al estar definiendo una variable, no se ejecutaria, gracias a tod@s