Cual es el error en esto?

[den_22]

Hola, sigo con este codigo que no lo puedo hacer andar. La cuestion es que tratando de hacer un modificar perfil, con este codigo puedo obtener los datos de la bd, pero no los puedo actualizar.
Les paso el codigo:

Código PHP:

<?php
 
session_name('pLogin');
session_set_cookie_params(2*7*24*60*60);
session_start();
 
 
include("function.php");
include("getdata.php");
include("connect.php"); //incluimos el connect.php que contiene los datos de la conexión a la db y la sesión
 
$usuario = $_SESSION['usuario'];
 
 
 
 
$query=mysql_query("SELECT * FROM prueba WHERE usuario='$usuario' ");
 
if(mysql_num_rows($query)>0){
 
$query=mysql_fetch_array($query);
if($save)
 {
    query("update prueba set firstname='$firstname',lastname='$lastname' where usuario='$usuario'");
 };
 
//todo comprobado, ahora solo falta mostrar los datos
echo 'Bienvenid@ <b>'.$query['usuario'].'</b><br>'; 
 
}
 
?>
 

Y el html con el form:

Código PHP:

<form name="form1" method="post" action="modificarperfil.php">
  <table width="573" border="0">
  <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[firstname];  ?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="lastname"  id="lastname" value=" <?php echo $query[lastname];?> " /></td>
    </tr>
    </table>
    <div id="container_submit">
    <input name="save" type="hidden" id="save" value="ok">
       <input type=image height=23 alt=save width=87 src="images/save.jpg" border=0 name=Save> </div>
  </form>
 
</div>
 

Cual puede ser la solucion a mi problema?

Gracias de antemano

[Hidek1]

query("update prueba set firstname='$firstname',lastname='$lastname' where usuario='$usuario'");

ahi le entregas variables.. y esas variables no estan definidas o polomenos en el codigo que muestras no sale nada ..

[zaqpz]

Las variables $firstname, $lastname y $save no están definidas tal como te indicó Hidek1. Aparentemente lo que quieres hacer es recibir sus valores desde los campos de formularios. Si ese es el caso deberías hacerlo con $_POST['lastname'] y demás.

Por otra parte nunca, nunca jamáz hagas una consulta a base de datos que contengan variables que pueden haber sido manipuladas por un usuario externo. Siempre filtra y valida esas variables primero!

Saludos.

PD: mi mensaje nº 400! ^^

[den_22]

Tengo estos codigos el function.php:

Código PHP:

<?php 
function getfname($usuario)
{
    $query = mysql_query("select * from prueba where usuario='$usuario'");
    $getname= mysql_fetch_array($query);
    $firstname= $getname["firstname"];
    return $firstname;
}
 
function getlname($usuario)
{
    $query =mysql_query("select * from prueba where usuario='$usuario' ");
    $getname= mysql_fetch_array($query);
    $lastname=$getname["lastname"];
    return $lastname;
}
 
?>
 

El getdata.php:

Código PHP:

<?php
session_start();
include ("connect.php");
$usuario=$_SESSION['usuario'];
 
$query=mysql_query("select * from prueba where usuario='$usuario'");
$getusuario=mysql_fetch_array($query);
$firstname=getfname('firstname');
$lastname=getlname('lastname');
 
?>
 

[Hidek1]

felicitaciones por el mensaje 400 ;)


agrego:

aparte de lo que te dice zaqpz fijate <?php echo $query[lastname];?> le estas entregando una constante en ves de un string para reconocer el key de ese array, aunque en algunos servidores funciona (ni idea porque) no es recomendable!

<?php echo $query["lastname"];?>

[den_22]

Digamos que tendria que poner

Código PHP:

$firstname=$_POST['firstname']
 

?

[den_22]

Hola ahora cambié el codigo
el modificarperfil.php:

Código PHP:

<?php
 error_reporting(E_ALL ^ E_NOTICE);
session_name('pLogin');
session_set_cookie_params(2*7*24*60*60);
session_start();
 
include("connect.php"); //incluimos el connect.php que contiene los datos de la conexión a la db y la sesión
 
$usuario = $_SESSION['usuario'];
 
if(!isset($usuario)){
 
echo 'No se ha seleccionado ningun usuario.'; 
 
}else{ 
 
 
$query=mysql_query("SELECT * FROM prueba WHERE usuario='$usuario' ");
 
if(mysql_num_rows($query)>0){
$user_ok=mysql_fetch_array($query);
 
 
 
 
//todo comprobado, ahora solo falta mostrar los datos
echo 'Bienvenid@ <b>'.$user_ok['usuario'].'</b><br>'; 
 
}else{
 
echo 'La id de usuario seleccionada no existe';
}
}
 
?>
 

y hay otro que se llama cuando se da en editar perfil, update_cuenta.php:

Código PHP:

<?
include('connect.php'); //incluimos el config.php que contiene los datos de la conexi&oacute;n a la db
 
$firstname=$_POST['firstname']; 
$lastname=$_POST['lastname'];
 
 
//introduccimos la modificacion hecha por el usuario
 
 
 
mysql_query("
UPDATE prueba
SET firstname='$firstname',lastname='$lastname'
WHERE usuario='$usuario'  ");
echo '<span class="Estilo24">Datos Guardados con exito</span>';
 
 
 
 
 
?>
 

el form en la tabla

Código PHP:

<form action="update_cuenta.php" method="post">
  <table width="573" border="0">
        <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $user_ok[firstname];  ?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="lastname"  id="lastname" value=" <?php echo $user_ok[lastname];?> " /></td>
    </tr>
</table>
    <div id="container_submit">
    <input name="Submit" type="submit" class="titulo" value="EDITAR PERFIL"> </div>
  </form>
 

Como hago sigue sin andarme, toma los datos de la bd, pero no actualiza los datos

[zaqpz]

Iniciado por Hidek1

felicitaciones por el mensaje 400 ;)

Gracias!

Ahora en la nueva versión del código que has colocado en update_cuenta.php no veo que estés obteniendo en algún lado la variable $usuario que usas en el WHERE de tu UPDATE.

No te olvides de filtar los datos antes de realizar una consulta en base de datos con ellos.

Saludos.

[den_22]

deberia poner en el update_cuenta.php

$usuario = $_SESSION['usuario'];

[den_22]

Hola, haciendolo asi tampoco sale, por favor alguien me puede decir donde declaro esa variable?

Código PHP:

<?
 
include('connect.php'); //incluimos el config.php que contiene los datos de la conexi&oacute;n a la db
 
$firstname=$_POST['firstname']; 
$lastname=$_POST['lastname'];
 
 
//introduccimos la modificacion hecha por el usuario
$usuario=$_SESSION['usuario'];
 
 
 
mysql_query("
UPDATE prueba
SET firstname='$firstname',lastname='$lastname'
WHERE usuario='$usuario'  ");
echo '<span class="Estilo24">Datos Guardados con &eacute;xito</span>';
 
 
 
 
 
?>
 

Gracias

[CdG]

te falta session_start(); en update_cuenta.php
Revisa la query con var_dump();

[den_22]

No, poniendole session_start(); no me anda :(

[zaqpz]

Por alguno de tus códigos vi que llamabas a la sesión con un nombre distinto al de por defecto con session_name()... ¿la has colocado a esta función también?

[den_22]

Hola, gracias por haber contestado, al final le encontré la solución, el problema era que estaba usando las etiquetas

Código PHP:

<? ?>
 

y debian ser

Código PHP:

<?php ?>
 

Muchas gracias.

den_22