PHP and jQuery: show/hide div if user logged in or not

Question

EDIT: thanks a lot for all your comments/replies. Even if i remove the JS, and use the PHP if/else statements only, i still get the same error.

i keep on running into the same error:

The 2 divs below are visible whereas only one of them should be shown at a time (whether or not the user is logged in).

For simplicity purposes, i removed the html inside the 2 divs.

<div id="header_wrapper">

<div id="loginbutton"></div>

<div id="menu">
<ul>
<li></li>
<li></li>
<li></li>    
</ul>
</div>

<?php 
if (!isset($_SESSION['id']) || empty($_SESSION['id'])){  ?>

<script>
document.getElementById("menu").style.display = "none";
document.getElementById("loginbutton").style.display = "show";
</script>
<?php }
else {  ?>
<script>
document.getElementById("loginbutton").style.display = "none";
document.getElementById("menu").style.display = "show";
</script>

<?php 
}
?>
</div>

The $_SESSION['id'] is not empty. Indeed if i echo $_SESSION['id'] on the page, i get the correct session #.

Am I missing something?

Don't use else, use if (isset($_SESSION['id'])) and btw, it should be ...dislpay = "visible"; — Sergio
– Sergio, Commented Jun 27, 2013 at 5:40
I'm not sure if this will matter but try changing <script> to <script type="text/javascript"> — JunM
– JunM, Commented Jun 27, 2013 at 5:41
thanks for your help, it's still buggy, going to check the js console — Caro_deb
– Caro_deb, Commented Jun 27, 2013 at 5:59

user2511671 · Accepted Answer · 2013-06-27 06:45:12Z

3

If you use session like following exaple you can hide the div

<?php if(isset($_SESSION['id'])) { ?><div>div content</div><?php }?>

answered Jun 27, 2013 at 6:45

user2511671

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

Comments

M Khalid Junaid · Accepted Answer · 2013-06-27 06:34:23Z

2

What about this if you put your html inside php if blocks , or it is necessary to use javascript you can do it also by using only php then why adding JS

<div id="header_wrapper">
<?php 
if (!isset($_SESSION['id']) || empty($_SESSION['id'])){  ?>
<div id="loginbutton"></div>
<?php }
else {  ?>
<div id="menu">
<ul>
<li></li>
<li></li>
<li></li>    
</ul>
</div>
<?php 
}
?>
</div>

edited Jun 27, 2013 at 6:34

answered Jun 27, 2013 at 5:54

M Khalid Junaid

64.6k10 gold badges93 silver badges120 bronze badges

4 Comments

Caro_deb Over a year ago

PHP is what I tried at first, but no luck, it still shows the 2 divs at the same time.

M Khalid Junaid Over a year ago

How in if/else only one condition will run how both conditions run at same time think man ?

Caro_deb Over a year ago

Thanks a lot for your input, i don't understand either ! i ve never had such a problem before !

Jan Sverre Over a year ago

This is the correct answer, but you have to use another iftest. When the user logs in, set $_SESSION['LoggedIn'] = true. Then you can check if(isset($_SESSION['LoggedIn']) && $_SESSION['LoggedIn']) { around the <div>

bestprogrammerintheworld · Accepted Answer · 2013-06-27 05:45:13Z

1

Change

.style.display = "show";

TO

.style.display = "block";

OR

.style.display = "";

AND change

<script>

TO

<script type="text/javascript">

There is no valid value called show in this context. If you have Firebug (or similar) check in the console for further js-errors. If you don't, download it right away :-)

answered Jun 27, 2013 at 5:45

bestprogrammerintheworld

5,5299 gold badges48 silver badges76 bronze badges

4 Comments

random Over a year ago

<script> alone is valid under HTML5

bestprogrammerintheworld Over a year ago

@random - yes, but there's nothing indicating it's a html5-document.

Caro_deb Over a year ago

thanks for your help, it's still buggy, going to check the js console

bestprogrammerintheworld Over a year ago

@CharlieHarris - you're welcome. What message/warning/errors do you get?

drarkayl · Accepted Answer · 2013-06-27 06:19:00Z

1

Edit:

<div id="loginbutton"></div>

<div id="menu">
<ul>
<li></li>
<li></li>
<li></li>    
</ul>
</div>

<?php 
if (isset($_SESSION['id']) && !empty($_SESSION['id'])){  ?>
<script>
document.getElementById("loginbutton").style.display = "none";
document.getElementById("menu").style.display = "show";
</script>
<?php }
else {  ?>

<script>
document.getElementById("menu").style.display = "none";
document.getElementById("loginbutton").style.display = "show";
</script>

<?php 
}
?>
</div>

this should work

edited Jun 27, 2013 at 6:19

answered Jun 27, 2013 at 5:43

drarkayl

1,0177 silver badges17 bronze badges

4 Comments

drarkayl Over a year ago

can you tell me what exactly is happening? like when a user is logged in what is being shown?

Caro_deb Over a year ago

EDIT: When the user is logged in, both the divs are visible. As soon as the user logs out, only #menu is visible whereas it should be #loginbutton

drarkayl Over a year ago

try making it 'and' instead of 'or' in the if statement

drarkayl Over a year ago

try my updated code and are u deleting the sessions after the user logs out?

Collectives™ on Stack Overflow

PHP and jQuery: show/hide div if user logged in or not

4 Answers 4

Comments

4 Comments

4 Comments

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

4 Comments

4 Comments

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related