if statement on session data codeigniter

Question

So I am trying to hide or show options if the user is logged in or not.

I have this simple statement, its always showing no regardless if I am logged in or not.

if( !isset($_SESSION) ){
                    echo "yes";
                }
                else {
                    echo "no";
                }

I have also tried

function __construct()
{
parent::__construct();
$this->is_logged_in();
}


    if(is_logged_in())

        {
            echo "yes";
        }

    else
    {
        echo "no";                      
    }

Neither works, I also think the first one if simpler, but I am not sure what method would be better.

I would suggest codeigniter's session class, its a bit more robust IMHO: https://www.codeigniter.com/user_guide/libraries/sessions.html — dmgig, Mar 14 '16 at 15:35

Nabin Kunwar · Accepted Answer · 2016-03-14T15:41:21.190

2

isset($_SESSION) checks if the variable is set or not and here $_SESSION is already defined.

so in your case !(isset($_SESSION)) is false coz isset($_SESSION) is true and !true is false

To check for the session value try isset($_SESSION['key_you_set']). This will check if the key_you_set exists or not.

edited Mar 14 '16 at 15:41

answered Mar 14 '16 at 15:34

Nabin Kunwar

1,965
14
29

Ok thank you, so using session data like this wont give me the result i need – Beep Mar 14 '16 at 15:39
try like isset($_SESSION['key_you set']) – Nabin Kunwar Mar 14 '16 at 15:40
And would be better trying CI's session library. :) – Nabin Kunwar Mar 14 '16 at 15:50
like `if $this->session->userdata('id'); {echo "yes";}` – Beep Mar 14 '16 at 15:51
1

like if($this->session->userdata('id')) {echo "yes";} after saving userdata with key id – Nabin Kunwar Mar 14 '16 at 15:53
1

Got ya, Thank you dude, your a star. – Beep Mar 14 '16 at 15:55

santosh · Answer 2 · 2016-03-14T15:48:31.653

1

Assuming you have set an session with name session_id You can retrieve your session information in codeigniter like,

$session_id = $this->session->userdata('session_id');

Now You can check like below

if($session_id==""){
echo "session not set";
}
else{
echo "session set";
}

edited Mar 14 '16 at 15:48

answered Mar 14 '16 at 15:39

santosh

799
5
17

= will assign value to the variable so your expression will always return true – Nabin Kunwar Mar 14 '16 at 15:46
@NabinKunwar yeah, thats what im thinking – Beep Mar 14 '16 at 15:47
sorry that was an typo mistake .. i have updated my answer .. @NabinKunwar thank you for reminding.. – santosh Mar 14 '16 at 15:49

score -1 · Answer 3 · answered Mar 14 '16 at 17:20

-1

I think this helps you ..

$session_id = $this->session->userdata('session_id');

less code but more secure

echo (!empty($session_id ) && isset($session_id) ) ? "session set" : "session not set" ;  // ternary operator

answered Mar 14 '16 at 17:20

Aman Kumar

4,533
3
18
40

`!empty($var) && isset($var)` is an antipattern that no one should ever use in any project for any reason. https://stackoverflow.com/a/4559976/2943403 – mickmackusa Jul 14 '21 at 04:52

if statement on session data codeigniter

3 Answers3

I think this helps you ..

less code but more secure