consulta update problema

dani12345 · #1 (**permalink**) 11/04/2010, 06:54

hola tengo un problema ,estoy haciendo una consulta que actualice un registro en concreto de la base de datos ,el problema es que me actualiza todos, quiero que el usuario al cerrar la sesion actualice el registro de su sesion con la fecha en que ha cerrado dicha sesion pero en realidad me actualiza la fecha de cierre de sesion de todos los registros, se que necesito compararlo con algo mas que el nombre de usuario de la sesion, pero no se que hacer mas, creo que debería tener un id propio de cada sesion o algo asi,me podeis ayudar

<?php
session_start();
session_destroy();
include 'conectar.php';
$date = date("Y-m-d H:i:s");
$query="SELECT FROM historial WHERE usuariodesesion= '$_POST[usuario]' AND numerodesesion= '$numerosesion'";
mysql_db_query("basedatos",$query);
$query2="UPDATE historial SET fechacierresesion= '$date' ";
mysql_db_query("discosrko",$query2);
$query;
$query2;
?>
<html>
<head>
<title>Logout</title>
</head>
<body background="images/background.gif">
</body>
<SCRIPT LANGUAGE="javascript">
alert("Usted ha cerrado la sesión, esperamos volver a verle de de nuevo.");
location.href = "index.php";
</SCRIPT>
</html>

lucasbanegas · #2 (**permalink**) 11/04/2010, 07:32

Le falta la clausula WHERE al UPDATE. O sea, el UPDATE esta bien, pero tenes que ponerle WHERE a = b

Es decir, 'UPDATE historial SET fechacierresesion= '$date' WHERE usuario = '$usuario'";

Tu tabla debería tener una columna donde aparezca el ID del usuario que está cerrando la sesión.

dani12345 · #3 (**permalink**) 11/04/2010, 08:05

la tabla del historial tiene numerodesesion, usuariodesesion, fechainiciosesion y fecha cierre de sesion. el problema es que se cambiarian todas los registros de ese usuario y no el de esa sesion, voy a probarlo pero creo que ocurrira eso, gracias

lucasbanegas · #4 (**permalink**) 13/04/2010, 04:50

Cita:

Iniciado por dani12345

la tabla del historial tiene numerodesesion, usuariodesesion, fechainiciosesion y fecha cierre de sesion. el problema es que se cambiarian todas los registros de ese usuario y no el de esa sesion, voy a probarlo pero creo que ocurrira eso, gracias

Y si, la verdad es que es muy probable. Entonces deberias hacer un doble WHERE. Osea, "WHERE usuario = '$usuario' AND sesion = '$sesion';

dani12345 · #5 (**permalink**) 13/04/2010, 12:39

voy a probar como dices,porque lo probe la otra vez y efectivamente se actualizaban todos los cierre de sesion del mismo usuario, ojala funcione como dices, lo que pasa es que el codigo de sesion yo no lo especifico es automatico y autoincremental,voy a ver,gracias

como va siempre aumentando a mas el numero de sesion habia pensado en esto, que te parece:

<?php
session_start();
include 'conectar.php';
$date = date("Y-m-d H:i:s");
$query="SELECT numerodesesion FROM historial WHERE usuariodesesion='$_SESSION[usuario]'";
$result=mysql_db_query("discosrko",$query);
$MAX=mysql_fetch_array($result);
$query2="UPDATE historial SET fechacierresesion='$date' WHERE usuariodesesion='$_SESSION[usuario]' AND numerodesesion='$MAX[numerodesesion]'";
mysql_db_query("discosrko",$query2);
session_destroy();
?>

finalmente he conseguido lo que quería ,ese es el codigo modificado y funciona correctamente