DaniWeb Forum Index > Web Development > PHP

Users have to login and logout twice, why?

Please support our PHP advertiser: PostgreSQL or MySQL? Compare and contrast the two most popular open source databases

•

Join Date: Jul 2009

Posts: 20

Reputation: eantz is an unknown quantity at this point

Solved Threads: 0

eantz

Offline

Newbie Poster

Users have to login and logout twice, why?

Jul 15th, 2009

Hi..
I'm newbie here..

I have a problem with my login and logout script..

Here is the login script:

 Help with Code Tags 
 PHP Syntax (Toggle Plain Text) 
<?php
$q = "SELECT * FROM members WHERE username='%s' AND password='%s'";
$q = sprintf($q, $username, $password);
$q = mysql_query($q);
if (mysql_num_rows($q) > 0) {
$_SESSION['id'] = $q[0]['id'];
$IS_LOGGED_IN = true;
}
?>
<?php
$q = "SELECT * FROM members WHERE username='%s' AND password='%s'";
$q = sprintf($q, $username, $password);
$q = mysql_query($q);
if (mysql_num_rows($q) > 0) {
$_SESSION['id'] = $q[0]['id'];
$IS_LOGGED_IN = true;
}
?>

here is the logout script:

 Help with Code Tags 
 PHP Syntax (Toggle Plain Text) 
<?php
session_start();
unset($_SESSION['userid']);
session_destroy();
$_SESSION = array();
header('Location: index.php');
?>
<?php
session_start();
unset($_SESSION['userid']);
session_destroy();
$_SESSION = array();
header('Location: index.php');
?>

and I have a variable which is called in every page that define some one logged in:

$IS_LOGGED_IN = (isset($_SESSION['userid'] && $_SESSION['userid'] !== '');
With that code, users have to login twice before he is in real login state. It happens to with logout.

Why?

Thank's for helping me..

•

Join Date: Aug 2009

Posts: 1

Reputation: No Box is an unknown quantity at this point

Solved Threads: 0

No Box

Offline

Newbie Poster

Re: Users have to login and logout twice, why?

Aug 18th, 2009

Check all of your links to the other web pages. I had the same problem with the session resetting itself when I had the <a href="http://www.yoursite.com/page.php"> instead of <a href="page.php"> I set all of them the same way no mixing and matching them together to navigate through the page and it worked.