Brain dead from messing with this...

[VooDooRT]

I've been having problems with this code for a while now. Anyone else want to take a crack at it?

I'm trying to get online users...

The code isn't setting the users in the database at all...

function onlineusers($user) {
global $site_prefs;
$date = date("ymd");
$hour = date("H");
$minute = date("i");
if ($user <> "none") {
$sql = "select * from online_user where USERid = '$user'";
$result = mysql_query($sql);


echo mysql_error();
if (mysql_num_rows($result) == 0)
$sql = "insert into online_user values('$user','$date','$hour','$minute')";
else
$sql = "update online_user set date = $date, hour = $hour, minute = $minute where USERid = '$user'";
mysql_query($sql);
echo mysql_error();
}
$logout = $site_prefs[logouttime];
$logouthours = round($logout / 60);
$logoutminutes = $logout - $logouthours * 60;

$sql = "delete from online_user where date <> $date";
mysql_query($sql);

$sql = "select * from online_user";
$result = mysql_query($sql);
while ($row = mysql_fetch_object($result)) {

if ($logout > 60) {
	if ($row->hours < $hour - $logouthours)
	mysql_query("delete from online_user where USERid = \"$row->USERid\"");
	elseif ($row->hours >= $hour - $logouthours) {
		if ($row->minute < $minute - $logoutminutes)
		mysql_query("delete from online_user where USERid = \"$row->USERid\"");
	}

}
else {
if ($row->minute < $minute - $logoutminutes)
mysql_query("delete from online_user where USERid = \"$row->USERid\"");
}
}
}