[SOLVED] User Login timestamp or datetime

[adam291086]

if revraz is correct then use update. To keep thing simple you can always use update. When i a new user signs up add in the insert last loged in then. Because theoretically that have sign up and therefore logged in.

[giraffemedia]

I tried that revraz and got a blank page.

[revraz]

Echo the query so we can check it for errors before you run mysql_query on it.

[giraffemedia]

How do I do that (i'm just starting out!). Do I use:

 

echo $_SESSION['SESS_MEMBER_ID']);

 

[adam291086]

you have somehting like

 

$query = "select * FROM adam";

$adam = mysql_query($query);

 

we need you to echo $query and show us the result

[giraffemedia]

Aaaah - I get it. The result is"

 

SELECT member_id FROM members WHERE login='james_payne' AND passwd='3de0746a7d2762a87add40dac2bc95a0'

 

If I echo the session id I get the correct member_id as well.

[adam291086]

echo the update query. Thats where the problem is .   

[revraz]

Wrong one, we want the update query.

 

$query="UPDATE members SET last_visits = (NOW()) WHERE member_id ='".$_SESSION['SESS_MEMBER_ID'])"' ";
  
echo $query;
exit;
$result = mysql_query($query);

[tapos]

Please have a check with the following code

<?php
//Start session
session_start();

include('../includes/config.php');
include('../includes/opendb.php');

//Sanitize the value received from login field
//to prevent SQL Injection
if(!get_magic_quotes_gpc()) {
	$login=mysql_real_escape_string($_POST['login']);
}else {
	$login=$_POST['login'];
}

//Create query
$query="SELECT member_id FROM members WHERE login='$login' AND passwd='".md5($_POST['password'])."'";
$result=mysql_query($query);
//Check whether the query was successful or not
if($result) {
	if(mysql_num_rows($result)>0) {
		//Login Successful

		session_regenerate_id();
		$member=mysql_fetch_assoc($result);
		$_SESSION['SESS_MEMBER_ID']=$member['member_id'];
		session_write_close();

		$query="update members set last_visit = NOW() WHERE member_id ='$member[member_id]'";
			$result=mysql_query($query);


		header("location: /admin.php");
		exit();
	}

	else {
		//Login failed
		header("location: login_failed.php");
		exit();
	}
}else {
	die("Query failed");
}



?>

 

--

Thanks,

Tapos Pal

http://tapos.wordpress.com

[giraffemedia]

Wrong one, we want the update query.

 

$query="UPDATE members SET last_visits = (NOW()) WHERE member_id ='".$_SESSION['SESS_MEMBER_ID'])"' ";
   
echo $query;
exit;
$result = mysql_query($query);

 

Get a blank page again.

[revraz]

If your fieldname is last_visit then I made a typo and used last_visits

[giraffemedia]

Please have a check with the following code

<?php
//Start session
session_start();

include('../includes/config.php');
include('../includes/opendb.php');

//Sanitize the value received from login field
//to prevent SQL Injection
if(!get_magic_quotes_gpc()) {
	$login=mysql_real_escape_string($_POST['login']);
}else {
	$login=$_POST['login'];
}

//Create query
$query="SELECT member_id FROM members WHERE login='$login' AND passwd='".md5($_POST['password'])."'";
$result=mysql_query($query);
//Check whether the query was successful or not
if($result) {
	if(mysql_num_rows($result)>0) {
		//Login Successful

		session_regenerate_id();
		$member=mysql_fetch_assoc($result);
		$_SESSION['SESS_MEMBER_ID']=$member['member_id'];
		session_write_close();

		$query="update members set last_visit = NOW() WHERE member_id ='$member[member_id]'";
			$result=mysql_query($query);


		header("location: /admin.php");
		exit();
	}

	else {
		//Login failed
		header("location: login_failed.php");
		exit();
	}
}else {
	die("Query failed");
}



?>

 

--

Thanks,

Tapos Pal

http://tapos.wordpress.com

 

HOORAY - It works!!!!

 

Thanks Tapos

[tapos]

It's my pleasure.

U should make the post solved.

 

--

Thanks,

Tapos Pal

http://tapos.wordpress.com

[giraffemedia]

Thanks for all your help guys. I understand I need to use the UPDATE query instead of INSERT, but why did:

 

WHERE member_id ='$member[member_id]'";

 

work instead of:

 

WHERE member_id ='".$_SESSION['SESS_MEMBER_ID'])"' ";

 

Does this mean the session is not being set correctly?

[revraz]

No, my parenthesis were off.  The session would work fine I'm sure.

 

$query="UPDATE members SET last_visit = NOW() WHERE member_id ='".$_SESSION['SESS_MEMBER_ID']"' ";

 

[adam291086]

i think its because you are treating the session like a string with the  "  " around it.

 

To see if the session is set just echo it out

 

echo $_SESSION['SESS_MEMBER_ID']

[revraz]

You probably have error reporting turned off in your php.ini, so if you check your Log file, you would have received a syntax error instead of a blank screen.

[giraffemedia]

i think its because you are treating the session like a string with the  "  " around it.

 

To see if the session is set just echo it out

 

echo $_SESSION['SESS_MEMBER_ID']

 

I did try it without the " " but got a blank page.

 

echo $_SESSION['SESS_MEMBER_ID'];

 

That works fine and the session is being set.

 

I guess I need to learn more about debugging. I'm ordering the VTC training cd on php so hopefully i'll be able to learn a lot more soon.

 

Thanks for all you help.

 

James