[RESOLVED] Notice: Undefined index

[2CODE]

I'm coding a function to check the user if he logged in or he's a visitor to call it from the other files here is it :

function login_check(){
session_start();
if (!isset($_SESSION['logged']) && $_SESSION['logged'] == 'true') {
require_once('libs/Smarty.class.php');
$smarty = new Smarty();
$smarty->template_dir = 'libs/templates';
$smarty->compile_dir = 'libs/templates_c';
$smarty->config_dir = 'libs/configs';
$smarty->cache_dir = 'libs/cache';
$smarty->display('login.tpl');
die();
}}

So I added this line :

if (isset($_SESSION['logged']))

The problem is gone but the script is not working 🙁

Any ideas ?

The current full code is :

function login_check(){
session_start();
if (isset($_SESSION['logged']))
if (!isset($_SESSION['logged']) && $_SESSION['logged'] == 'true') {
require_once('libs/Smarty.class.php');
$smarty = new Smarty();
$smarty->template_dir = 'libs/templates';
$smarty->compile_dir = 'libs/templates_c';
$smarty->config_dir = 'libs/configs';
$smarty->cache_dir = 'libs/cache';
$smarty->display('login.tpl');
die();
}}

[bradgrafelman]

Your logic doesn't make sense; the outer if() will only evaluate to true if $_SESSION['logged'] isn't defined, and the inner if() statement won't evaluate to true unless it is defined.

In other words, logically speaking the inner if() statement is guaranteed to never be executed.

EDIT: Also note that your logic before the change doesn't make any sense either; you first check to see that $_SESSION['logged'] isn't defined... and then you try to compare its value to a string.

[2CODE]

So what's the best way to do that ?

[2CODE]

I've recoded it again in different way and I got it working.
Resolved.