¿ Donde está el error ?

Campesino · #1 (**permalink**) 20/04/2010, 02:36

Por favor, tengo este script para modificar datos de una tabla:

<?php
include "../config.php";
$username = $_COOKIE['loggedin'];
if (!isset($_COOKIE['loggedin'])) die("You are not logged in, <a href=../login.htm>click here</a> to login.");
mysql_connect($server, $db_user, $db_pass) or die (mysql_error());
mysql_select_db($database)
or die ("No se pudo seleccionar la base de datos porque ".mysql_error());
mysql_query("UPDATE $table SET name1 = '"$_POST['name1']"' WHERE username = '$username'");
mysql_query("UPDATE $table SET name2 = '".$_POST['name2']."' WHERE username = '$username'");
mysql_query("UPDATE $table SET email = '".$_POST['email']."' WHERE username = '$username'");
mysql_query("UPDATE $table SET password = '".$_POST['newpass']."' WHERE password = '".$_POST['oldpass']."'");
mysql_query("UPDATE $table SET website = '".$_POST['telefono']."' WHERE username = '$username'");
?>

pero al aparecer los datos en el recuadro, en lugar de los datos existente, sale el dato existente precedido de signos como: >Dato size=
El dato en rojo es el campo existente en la tabla. ¿ Como se quitaria esos caracteres que salen junto al dato?
Gracias

Vallu · #2 (**permalink**) 20/04/2010, 04:07

primero de todo simplifica el numero de consultas a la base de datos:

Código PHP:

  mysql_query("UPDATE $table SET name1 = '".$_POST['name1']."',name2 = '".$_POST['name2']."',email = '".$_POST['email']."',website = '".$_POST['telefono']."'  WHERE username = '".$username."'");

mysql_query("UPDATE $table SET password = '".$_POST['newpass']."' WHERE password = '".$_POST['oldpass']."'");

y vigila tambien que en la primera sentencia sql te faltava concatenar el post. A ver si asi funciona.

Tambien haz un print_r($_POST[]) para comprobar que los datos que recibes son los que quieres.

Campesino · #3 (**permalink**) 20/04/2010, 09:33

Perdona, pero no se como hacerlo, ¿ como es eso de concatenar el post ? y ¿ donde pondria el Print_r($_POST[]) para que saliesen los datos dentro del cuadro "imput"

marioms1995 · #4 (**permalink**) 20/04/2010, 09:47

campesino, creo que vallu se referia con concatenar el post a que de esto:

mysql_query("UPDATE $table SET name1 = '"$_POST['name1']"' WHERE username = '$username'");

pases a esto:

mysql_query("UPDATE $table SET name1 = '".$_POST['name1']."' WHERE username = '$username'");

y en cuanto a la sugerencia de ver el contenido del array $_POST tan solo tienes que añadir la sentencia:

print_r($_POST);

al principio del script.
Espero que te sirva de ayuda

margaro · #5 (**permalink**) 20/04/2010, 09:57

Lo que quizo decir es que en la primera consulta te faltaba un punto para concatenar la variable $_POST, y para concatenar debes agragarle un punto "." despues de la cadena. ejemplo:

"cadena "."$_POST['name1']"."Cadena"

y print_r es una funcion que te permite imprimir los valores de un arreglo para visualizarlos mejor. en este caso haz print_r($_POST)....

observo tambien que aperturas cadenas con comillas dobles " y luego cierras con comillas simples ' , fijate lo que haces "UPDATE $table SET name1 = '.... deberias hacerlo con uno o con otro, pero no una mezcla de los dos... te sugiero que leas tutoriales de concatenacion de cadenas..... suerte

Campesino · #6 (**permalink**) 20/04/2010, 11:16

He puesto los puntos que faltaban, los quité probando y olvidé colocarlos, el caso es que el problema de aparecer en el recuadro del imput los datos existentes en la base de datos, no aparecen, en su lugar sale este código dentro del cuadro:

<?phpinclude "../config.php";mysql_connect($server, $db_user, $db_pass) or die (mysql_error()); $result = mysql_db_query($database, "select * from $table WHERE username = '$username'") or die (mysql_error()); while ($qry = mysql_fetch_array($result)) { echo "$qry[username]"; } ?>

marioms1995 · #7 (**permalink**) 20/04/2010, 13:29

por favor postea el codigo completo, puede que el problema este en lo que imprimes

Campesino · #8 (**permalink**) 20/04/2010, 15:24

El script completo está en el primer comentario que puse en color azul . Si a ese script le pones una base de datos y datos de identificación lo tienes como yo y al entrar en él, deberian salir los datos existentes en la tabla dentro del recuadro de entrada de datos, pero en lugar de salir los datos, sale el código que he colocado antes, dentro del recuadro.
No se si me he explicado, pero si lo ejecutais lo vereis.

millan2525 · #9 (**permalink**) 20/04/2010, 15:40

no habrás dejado algún lado sin comillas o te sobrarán? :S

Campesino · #10 (**permalink**) 20/04/2010, 15:45

¿ Donde ? No tengo ni idea que puede ser

millan2525 · #11 (**permalink**) 20/04/2010, 15:48

se puede saber que hay en el config.php?

Quitando lo privado, pass, usuarios...

Campesino · #12 (**permalink**) 21/04/2010, 01:20

En config.php esta lo siguiente:

<?php
// Server to connect to, the default is 'LocalHost' but if it does not work use your IP address for your server
$server = "localhost";

// The name of the database, when using cPanel you would normally have your user name first like so: username_DATABASE-NAME
$database = "xxxxxxxxxxx";

// MySQL username to access the database with.
$db_user = "xxxxxxxxxx";

// MySQL password to access the database with.
$db_pass = "";

// The table that this script will set up and use, you can change this but recommended to keep the same so if you require any support we can help faster
$table = "usuarios";
?>

marioms1995 · #13 (**permalink**) 21/04/2010, 08:17

con recuadro te refieres al campo de la base de datos??

si es asi imprime el array $_POST

Campesino · #14 (**permalink**) 22/04/2010, 00:43

El problema está en el formulario de modificacion de datos y está en la parte del codigo señalado en azul, pero como vemos está en alfabeto anglosajon ¿ como se cambiaria para ponerlo que no de error ?

<form action="update.php" method="post" >
Usuario  

:     &nb sp;       &nbsp ;
<input name="username" disabled="disabled" value="<?php
include "config.php";
mysql_connect($server, $db_user, $db_pass) or die (mysql_error());
$result = mysql_db_query($database, "select * from $table WHERE username = '$username'") or die (mysql_error());
while ($qry = mysql_fetch_array($result)) {
echo "$qry[username]";
}
?>" size="29" ?>
¿ Cambiar su password ?
Su Password:      &nbsp ; 
<input type="password" name="oldpass" size="30" /> (sigue el scrpt)

Greco2 · #15 (**permalink**) 22/04/2010, 02:23

Cita:

Iniciado por Campesino

El problema está en el formulario de modificacion de datos y está en la parte del codigo señalado en azul, pero como vemos está en alfabeto anglosajon ¿ como se cambiaria para ponerlo que no de error ?

<form action="update.php" method="post" >
Usuario  

:     &nb sp;       &nbsp ;
<input name="username" disabled="disabled" value="<?php
include "config.php";
mysql_connect($server, $db_user, $db_pass) or die (mysql_error());
$result = mysql_db_query($database, "select * from $table WHERE username = '$username'") or die (mysql_error());
while ($qry = mysql_fetch_array($result)) {
echo "$qry[username]";
}
?>" size="29" ?>
¿ Cambiar su password ?
Su Password:      &nbsp ; 
<input type="password" name="oldpass" size="30" /> (sigue el scrpt)

el codigo php que tienes dentro del value del campo debes iniciarlo y cerrarlo con "<?php y ?>" y no con "<?php y ?>" como aparece en tu post, por eso no se ejecuta como codigo php.

tambien al final del codigo tienes size="29" ?> debes quitar ese ? de ahi.

Saludos

Campesino · #16 (**permalink**) 22/04/2010, 04:53

Muy bien, ahora me dice que la funcion mysql_db_query es obsoleta, me lo dice en el cuadro de texto mediente este aviso:
 Deprecated: mysql_db_query() [<a href='function.mysql-db-query'>function.mysql-db-query</a>]: This function is deprecated; use mysql_query() instead in C:\wamp\www\members\profile.php on line 234 dato del registro size=

Me aparece el dato que hay en el registro pero precedido de esta retaila.

Greco2 · #17 (**permalink**) 22/04/2010, 04:59

Cita:

Iniciado por Campesino

Muy bien, ahora me dice que la funcion mysql_db_query es obsoleta, me lo dice en el cuadro de texto mediente este aviso:
 Deprecated: mysql_db_query() [<a href='function.mysql-db-query'>function.mysql-db-query</a>]: This function is deprecated; use mysql_query() instead in C:\wamp\www\members\profile.php on line 234 dato del registro size=

Me aparece el dato que hay en el registro pero precedido de esta retaila.

tienes que usar la funcion mysql_query , como bien dices, la otra función está obsoleta.

Saludos