My session doesnot work properly

Question

I have the problem, that the following if condition always goes into the first if condition, never into the elseif condition.

This is my code below :

if ($this->session->has_userdata('role') == 1) {
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('adashboard');    
} elseif ($this->session->has_userdata('role') == 2) {
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('udashboard');
}

Can you tell my why?

Related, if not duplicate: [Is $this->session->userdata and $this->session->has_userdata same when use for authenticate in CodeIgniter?](https://stackoverflow.com/questions/44411551/is-this-session-userdata-and-this-session-has-userdata-same-when-use-for-a) — GolezTrol, Aug 07 '18 at 13:31
Like mentioned in the accepted answer of the question that @GolezTrol linked in his comment, the function `has_userdata()` just returns a `bool` that indicates whether or not the key exist, not the value behind it. — Philipp Maurer, Aug 07 '18 at 13:36

score 0 · Accepted Answer · answered Aug 07 '18 at 13:54

Hope this will help you :

Use userdata instead of has_userdata : userdata returns Value of the specified item key

if ($this->session->userdata('role') == 1) 
{
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('adashboard');    
} 
elseif ($this->session->userdata('role') == 2) 
{
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('udashboard');
}

For more : https://www.codeigniter.com/user_guide/libraries/sessions.html

Totaly I did that just after posting this question. – Dhiraj Shrestha Aug 07 '18 at 15:10 — Dhiraj Shrestha, Aug 07 '18 at 15:10

score 0 · Answer 2 · answered Aug 08 '18 at 15:59

Hope this will help you

has_userdata() check whether given session index is set or not and it return boolen value.

$this->session->has_userdata('role') ;

this code check whether role is set or not. So in your case it returns true that's equivalent to 1.

In your case you need to use userdata() instead of has_userdata().

$role=$this->session->userdata('role');
if ($role == 1) {
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('adashboard');    
} elseif ($role == 2) {
    $this->session->set_flashdata('success', 'Welcome ' . $userInfo[0]->name . '!');
    redirect('udashboard');
}

My session doesnot work properly

2 Answers2