DaniWeb Forum Index > Web Development > PHP

if isset ==

Please support our PHP advertiser: PostgreSQL or MySQL? Compare and contrast the two most popular open source databases

Thread Solved

•

Join Date: Jul 2009

Posts: 208

Reputation: CFROG is an unknown quantity at this point

Solved Threads: 14

CFROG

Offline

Posting Whiz in Training

if isset ==

23 Days Ago

I'm trying to display a menu option only for certain session id's and for some reason it displays for ALL session id's ... not just the ones i have indicated. Not too sure why it's doing this, it all looks okay to me.

 Help with Code Tags 
 PHP Syntax (Toggle Plain Text) 
<?PHP 
if(isset($_SESSION['user_id']) && $_SESSION['user_id'] == 1 ||  68)
{?>
<li><span class="qmdivider qmdividery" ></span></li>
<li><a class="qmparent" href="javascript:void(0);">ADMIN</a> 
 
<ul style="width:150px;">
<li><a href="/admin/">Admin Home</a></li>
<li><span class="qmdivider qmdividerx" ></span></li>
<li><span class="qmtitle" >Hit Counter</span></li>
<li><a href="javascript:void(0);">Today&nbsp;<?=$hcount?></a></li>
<li><span class="qmdivider qmdividerx" ></span></li>
</ul></li> 
<? } ?>
<?PHP 
if(isset($_SESSION['user_id']) && $_SESSION['user_id'] == 1 ||  68)
{?>
<li><span class="qmdivider qmdividery" ></span></li>
<li><a class="qmparent" href="javascript<b></b>:void(0);">ADMIN</a> 

<ul style="width:150px;">
<li><a href="/admin/">Admin Home</a></li>
<li><span class="qmdivider qmdividerx" ></span></li>
<li><span class="qmtitle" >Hit Counter</span></li>
<li><a href="javascript<b></b>:void(0);">Today&nbsp;<?=$hcount?></a></li>
<li><span class="qmdivider qmdividerx" ></span></li>
</ul></li> 
<? } ?>

Any ideas?

•

Join Date: Aug 2009

Posts: 47

Reputation: EvolutionFallen is an unknown quantity at this point

Solved Threads: 8

EvolutionFallen EvolutionFallen is offline

Offline

Light Poster

23 Days Ago

Your problem is with the "|| 68": if(isset($_SESSION['user_id']) && $_SESSION['user_id'] == 1 || 68) . It should be:

 Help with Code Tags 
 php Syntax (Toggle Plain Text) 
if(isset($_SESSION['user_id']) && $_SESSION['user_id'] == 1 || $_SESSION['user_id'] == 68)
if(isset($_SESSION['user_id']) && $_SESSION['user_id'] == 1 || $_SESSION['user_id'] == 68)

It's like you're asking if(68) , which will of course return true. You need to compare the 68 to $_SESSION['user_id']

Also, you may want to add another set of parenthesis to group the last two logical operations together:

 Help with Code Tags 
 php Syntax (Toggle Plain Text) 
if(isset($_SESSION['user_id']) && ($_SESSION['user_id'] == 1 || $_SESSION['user_id'] == 68))
if(isset($_SESSION['user_id']) && ($_SESSION['user_id'] == 1 || $_SESSION['user_id'] == 68))