Actualización de registros

[Alun]

Hola:

Estoy intentando hacer una actualización de unos registros en una base de datos. Consiste en que un usuario pueda modificar sus datos. Para ello, he puesto un formulario con un cuadro por cada dato editable, y los mando otro archivo donde se supone tendría que hacerse la actualización. El problema es que, aunque la página me responde que se ha actualizado todo correctamente, cuando miro la base de datos todo sigue igual.

He probado a hacer la actualización de cada campo por separado, y he conseguido que se actualice el prmier campo, y al hacerlas todas en un bucle he logrado que me diga que los tres primeros registros se actualizan bien, y el resto falla, pero nada cambia. Este es el código:

Código PHP:

  //Conexión con la base de datos
//..
$result=mysql_fetch_row($sql);
$x=array(0=>'user','pwd','nombre','surname1','surname2','mail','phone');//creo un array cuyos valores son los nombres de los campos del formulario
foreach($x as $post)
{
  if(isset($_POST[$post]))
  {
    $update = 'update usuarios set '.$post.' = "'.$_POST[$post].'" where user = "'.$_SESSION['nombre'].'" and pwd = "'.$_SESSION['pwd'].'";';
    $fin = mysql_query($update);
    if($update)
    {
      echo 'Actualización exitosa<br>';
    }
    else
    {
      echo 'Actualización fallida<br>';
    }
  }
} 
   

Dudaba sobre si el fallo era algo relacionado con las variables de sesión, pero las he comprobado y son correctas, así que ya me he perdido por completo.

Gracias por la ayuda.

[ElJavista]

En primer lugar veo innecesario actualizar campo por campo, eso haría demasiadas llamadas al servidor MySQL, basta con una sola sentencia. Inténtalo de nuevo, solo mediante una consulta de actualización, pero esta vez imprime la consulta para que veas qué puede tener mal y si es necesario ejecútala en la consula del MySQL o en el phpMyAdmin (si lo utilizas). Por otra parte, esto si está muy mal:

Código:

if($update) 
    { 
      echo 'Actualización exitosa<br>'; 
    } 
    else 
    { 
      echo 'Actualización fallida<br>'; 
    }

$update es tu sentencia sql no el resultado de la consulta de actualización, si haces eso, siempre va a sali verdadero porque $update siempre es una cadena no nula, lo cual es equivalente a true. Ya te habrás dado cuenta que debiste hacer esto:

Código:

if ($fin) {
    echo 'Actualización exitosa<br>'; 
} else {
    echo 'Actualización fallida<br>'; 

}

[Alun]

Gracias por responder:

Bueno, del fallo de la variable me di cuenta mientras releía el mensaje después de postear. Tengo que quitarme la manía de escribirlos a mano, y usar más copiar/pegar. En el código aparece tal y como lo pusiste abajo.

Por otro lado, no había caido en probarlo fuera de php. Voy a probarlo, y comento con lo que sea.

Muchas gracias por la ayuda.

[Alun]

Hola de nuevo:

Acabo de probarlo desde la consola de mysql (utilizo xampp, por si sirve de algo ponerlo), y efectivamente, esa consulta ejecutada de uno en uno modifica los registros correctamente, aunque, como dijiste, actualizarlos en uno en uno iba a ser un poco tedioso y sobrecargante para el servidor.

Por otra parte, me he dado cuenta de otro fallo: en el formulario, tengo algunos campos con nombres distintos de como aparecen en la base de datos, sea el caso de los apellidos, que en el formulario aparecen como "surname1" y "surname2", y en la base de datos figuran como "apellido1" y "apellido2". Estos eran algunos de los campos en los que daba error la actualización. Los he escrito correctamente, pero ahora no solo no me muestra un error, sino que ni siquiera habla de ellos, como si no se hubieran recibido.

Muchas gracias por la ayuda.

[ElJavista]

Podrías poner algo de tu código para ver que pasa.

[Alun]

Bien, esto es el código tal y como está ahora:

El formulario de actualización:

edit_account.php:

Código PHP:

 //...
<form action="update_account.php?user=<?php echo $_SESSION['nombre']; ?>" method="post" name="update_account">
Nombre de usuario: <?php echo $result[0]; ?><br>
<input type="text" name="user"><br>
Contraseña: <?php echo md5($result[1]); ?><br>
<input type="password" name="pwd"><br>
Nombre: <?php echo $result[2]; ?><br>
<input type="text" name="nombre"><br>
Apellido 1: <?php echo $result[3]; ?><br>
<input type="text" name="apellido1"><br>
Apellido 2: <?php echo $result[4]; ?><br>
<input type="text" name="apellido2"><br>
email: <?php echo $result[5]; ?><br>
<input type="text" name="email"><br>
Teléfono: <?php echo $result[6]; ?><br>
<input type="text" name="telefono"><br>
<input type="submit" value="Actualizar datos">
</form>
//...  

y aquí la página de recepción de datos:

update_account.php:

Código PHP:

  //...
$result=mysql_fetch_row($sql);
$x=array(0=>'user','pwd','nombre','surname1','surname2','mail','phone');//creo un array cuyos valores son los nombres de los campos del formulario
foreach($x as $post)
{
 if(isset($_POST[$post]))
 {
  $update = 'update usuarios set '.$post.' = "'.$_POST[$post].'" where user = "'.$_SESSION['nombre'].'" and pwd = "'.$_SESSION['pwd'].'";';
  $fin = mysql_query($update);
  if($fin)
  {
   echo 'Actualización exitosa<br>';
  }
  else
  {
   echo 'Actualización fallida<br>';
  }
 }
}
//... 
   

[ElJavista]

Espero que esta vez si hayas copiado y pegado. Veo que no haz cambiado nada, que continúas intentando cargar campo por campo, por otra parte veo en el formulario los campos: apellido1 y apellido2, pero en el PHP está surname1 y surname2. Ahora bien, que gracia imprimir los resultados de una consulta mediante índices numéricos, me refiero a esto:

Nombre: <?php echo $result[2]; ?><br>

Si tranquilamente se puede hacer mediante el nombre del campo, siendo así más entendible. En fin. Intenta hacer la consulta de actualización con una sola sentencia, pruebala y si te falla pones tu código para ver que se puede corregir. Suerte!

[Alun]

He encontrado algo: tenía la sospecha de que había algún fallo con las variables de sesión. Efectivamente, al imprimir $_SESSION['pwd'] en el formulario, la contraseña aparece bien, pero al llevármela a la página de actualización, la imprimo y aparece una contraseña que no tiene nada que ver. Evidentemente, si no hay un registro en la base de datos en el que coincidan la contraseña y el nombre, no se va a actualizar nada. Lo mejor es que la he intentado pasar mediante GET, y nada, sigue cambiada. voy a ver dónde ocurre ese cambio y posteo.

[Alun]

Sigo sin encontrar el sitio donde se cambia el valor de la variable. Vi que en alguna página tengo una variable global que se llama "$pwd", y que sospecho que puede crear conflictos, así que he cambiado $_SESSION['pwd'] a $_SESSION['pass'], pero nada, aun así sigue cambiándose el valor. ¿Alguna idea de por qué pasa esto?

Muchas gracias.

[carlos_belisario]

donde llenas la variable $_SESSION['pwd']?? en algun otro sitio le asignas valor?? ya quepuede q este iendo a un sitio donde le des otro valor y por eso te est dando el error

[Alun]

Hola, carlos:

Pues en principio, solamente le asigno valor en la página de identificación:

Código PHP:

if($sql)
{
 $result=mysql_fetch_row($sql);
 if($result)
 {      
  echo "Has sido identificado correctamente, $result[0], con contraseña $result[1]<br>";
  $_SESSION['nombre']=$result[0];
  $_SESSION['pass'] = $result[1]; //como dije arriba, ahora es $_SESSION['pass']
  echo $_SESSION['nombre'].$_SESSION['pass'].'<br>'; //aquí la imprimo para comprobar que es correcta. Es correcta.
  echo '<a href=".">Volver al índice</a>';
 }
}
 

Después de comprobar que se cambia, le he asignado de nuevo un valor en la página que contiene el formulario de actualización:

Código PHP:

//...
$sql=mysql_query('select * from usuarios where user="'.$_GET['user'].'";',$conex) or die(mysql_error());
if($sql)
{
 $result=mysql_fetch_row($sql);
 if($result)
 {
  $_SESSION['pass'] = $result[1];
  echo $_SESSION['pass'].'<br>';
//...
 

Justo después, la imprimo, y sale correcta. No obstante, al pasarla a la página donde se actualiza la BD mediante $_SESSION (antes tampoco se pasaba mediante $_GET, ahora sí), aparece cambiada. incluso he forzado que tenga el valor correcto de esta forma:

Código Javascript:

function update_account()
{
<?php
$passw = mysql_query('select pwd from usuarios where user like "'.$_SESSION['nombre'].'";',$conexion);
$final = mysql_fetch_row($passw);
$_SESSION['pass'] = $final[0];
?>
return true;
}
 

Código PHP:

<form action="update_account.php?user=<?php echo $_SESSION['nombre']; ?>&data=<?php echo $_SESSION['pass']; ?>" method="post" name="update_account" onSubmit="update_account();">
 

y luego en la página de actualización, justo después de iniciar una sesión, hago esto:

Código PHP:

$_SESSION['pass'] = $_GET['data'];
 

Pero nada, en el momento de actualizar, aparece cambiada.


EDITO:

He observado algo curioso: esa variable toma el valor de la contraseña que he asignado a mysql.

[Alun]

Un nuevo dato: tenía session.use_trans_sid puesto a 1. Lo he colocado a 0, y ahora la variable $_SESSION['pass'] toma un valor vacío.