用php登录时隐藏div

Question

I am trying to hide a div once a user has logged into there account of my online store (built with cartweaver 4 php) but i just cant seem to get it to work.

the php code for defining wether the user is logged in or not is as followed:

if(!strlen($_SESSION["cwclient"]["cwCustomerID"]) ||
    $_SESSION["cwclient"]["cwCustomerID"] === 0 ||
    $_SESSION["cwclient"]["cwCustomerID"] === "0" ||
    $_SESSION["cwclient"]["cwCustomerType"] == 0 ||
    (isset($_SESSION["cwclient"]["cwCustomerCheckout"]) &&
       strtolower($_SESSION["cwclient"]["cwCustomerCheckout"]) == "guest"))

and the div i would like to hide upon login, which contains a table, have been given an id and class of:

  <div id="newcustomertable" class='newcustomer'>

I have tried applying this method using css: Show Hide div if, if statement is true

but that ended up giving me an undefined variable error on my testing server.

I admit i am a bit of a php newbie, so if anyone who is able to make more sense of this could help out and possibly find a solution it would be much appreciated.

Thank you.

Answer 1

You've got at least 3 variables there that could be throwing your undefined variable warning:

$_SESSION["cwclient"]
$_SESSION["cwclient"]["cwCustomerID"]
$_SESSION["cwclient"]["cwCustomerType"]

you need to isset() each of those before doing any tests.

Answer 2

For starters, the strlen function in PHP returns an INT, not a boolean.

Change from:

(!strlen($_SESSION["cwclient"]["cwCustomerID"])

to:

(strlen($_SESSION["cwclient"]["cwCustomerID"]) > 0)

Answer 3

First of all could that not just be simplified to the following if statement to make it easier

 if (($_SESSION["cwclient"]["cwCustomerID"]<>NULL)&&($_SESSION["cwclient"]["cwCustomerCheckout"]) == "guest")) { 
#code here 
}

Second of all to hide the div if they are logged in you could do it like this

     if (($_SESSION["cwclient"]["cwCustomerID"]<>NULL)&&($_SESSION["cwclient"]["cwCustomerCheckout"]) == "guest")) { 
    $shouldhide="style='visibility:hidden;'";
    } else {
$shouldhide=""; 
}
echo"<div id='hidethis' $shouldhide>"; 

Although if the content is a security risk it'd be better to not output it at all as it's still visible in the source code.

Answer 4

<div id ="newcustomertable"></div>
<?
if(!isset($_SESSION["cwclient"]["cwCustomerID"]) || 
!strlen(@$_SESSION["cwclient"]["cwCustomerID"]) ||
@$_SESSION["cwclient"]["cwCustomerID"] === 0 ||
@$_SESSION["cwclient"]["cwCustomerType"] == 0 ||
(isset($_SESSION["cwclient"]["cwCustomerCheckout"]) &&
strtolower(@$_SESSION["cwclient"]["cwCustomerCheckout"]) == "guest"))
{
?>
<script>
document.getElementById("newcustomertable").style.display = "none";
</script>
<?  
}
?>

Make sure that the condition check is AFTER the div element getting rendered. Because if you put it before the element the page would not have fully rendered and so the script will not be able to get the div that you want to hide.

REVERSE Condition based on OP's request. I know this is a kluge, folks. Dont flame me!

<div id ="newcustomertable"></div>
<?
if(!isset($_SESSION["cwclient"]["cwCustomerID"]) || 
!strlen(@$_SESSION["cwclient"]["cwCustomerID"]) ||
@$_SESSION["cwclient"]["cwCustomerID"] === 0 ||
@$_SESSION["cwclient"]["cwCustomerType"] == 0 ||
(isset($_SESSION["cwclient"]["cwCustomerCheckout"]) &&
strtolower(@$_SESSION["cwclient"]["cwCustomerCheckout"]) == "guest"))
{
 //do nothing
}
else
{
?>
<script>
document.getElementById("newcustomertable").style.display = "none";
</script>
<?  
}
?>