Notice: Undefined index: admin in C:\xampp\htdocs\deped2\index.php Notice: Undefined index: users in C:\xampp\htdocs\deped2\index.php

Question

This is my code.

<?php

include("include/dbcon.php");
if($_SESSION['admin']){      //this is where the error starts
    header("location:admin/");
}else if($_SESSION['users']){   //and here also
    header('location:employee/');
}else{
    header:('location:/deped2/');
}
?>

Please help me with this code

In addition to the current answer which will solve your issue, I would like to suggest using `exit;` after each `header('Location: ...')`. — Peter, Jan 20 '17 at 14:50

score 3 · Answer 1 · edited Jan 20 '17 at 14:56

3

You are not checking whether the Session Index is set or not. It would be better if you first start the session then check if the variables are present or not.

Try this and adding exit; after each header. Otherwise, your code might want to continue to execute.

<?php

if(!isset($_SESSION)){
session_start();
}
include("include/dbcon.php");
if(isset($_SESSION['admin'])){      //this is where the error starts
    header("location:admin/");
    exit;
}else if(isset($_SESSION['users'])){   //and here also
    header('location:employee/');
    exit;
}else{
    header:('location:/deped2/');
    exit;
}

?>

edited Jan 20 '17 at 14:56

Funk Forty Niner

74,450
15
68
141

answered Jan 20 '17 at 14:41

Abhishek

1,008
1
16
39

I would place the include below the conditional. If that fails, then it will throw a headers sent warning and add `exit;` after each header (for a better answer). – Funk Forty Niner Jan 20 '17 at 14:53
yes you can put include after the condition. – Abhishek Jan 20 '17 at 14:55

Notice: Undefined index: admin in C:\xampp\htdocs\deped2\index.php Notice: Undefined index: users in C:\xampp\htdocs\deped2\index.php

1 Answers1