php - issue with assigning session value

Question

I have a piece of code that is checking to see if a session ID has been set for my site, and if it hasn't, it assigns it a value from my URL. Below is the code I have:

if (!$_SESSION['test_id']){
  $_SESSION['test_id'] = strip_tags($_REQUEST["id"]);
};

My problem is not a huge one, but I am getting a notice that says:

Undefined index: test_id

Is there a better way assign this variable without setting off the notice? This piece fo code is in an Expression Engine platform (Code Igniter) so I don't have total control over the display of notices. This particular page has all error reporting turned off in my code, but it is still showing up.

Thanks so much for any help!

This question is tagged CodeIgniter. Why aren't you using the Session Class? — Jordan Arsenault
– Jordan Arsenault, Commented Feb 24, 2012 at 19:49

Cfreak · Accepted Answer · 2012-02-24 17:37:50Z

3

Test it against isset() as well. That will get rid of the notice.

if (!isset($_SESSION['test_id']) && !$_SESSION['test_id']){
  $_SESSION['test_id'] = strip_tags($_REQUEST["id"]);
};

answered Feb 24, 2012 at 17:37

Cfreak

19.4k6 gold badges52 silver badges64 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Alex · Accepted Answer · 2012-02-24 17:38:54Z

0

yes.. you can do it.. using isset()

if(!isset($_SESSION['test_id']))

This is because 'test_id' doesn't exist when you check if exists.. and with isset is not creating it...

answered Feb 24, 2012 at 17:38

Alex

2,1463 gold badges25 silver badges48 bronze badges

Comments

Michal Borek · Accepted Answer · 2012-02-24 17:40:04Z

0

You should try if(!isset($_SESSION['...'])) { }

or if(!empty($_SESSION['...')) { }

answered Feb 24, 2012 at 17:40

Michal Borek

4,6342 gold badges32 silver badges42 bronze badges

Comments

ircmaxell · Accepted Answer · 2012-02-24 17:41:46Z

0

The equivilant code to what you are looking for is:

if (!empty($_SESSION['test_id']))

The isset($foo) && !$foo is logically equivilant to !empty(). Empty is a language construct, just like isset, but it also checks that it's a non-empty value.

So there's no need for the extract condition, just toss the !empty there, and you're golden...

answered Feb 24, 2012 at 17:41

ircmaxell

166k36 gold badges269 silver badges316 bronze badges

Collectives™ on Stack Overflow

php - issue with assigning session value

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related