The IF statement

[dcjones]

Hi all,

I need to check the content of a $_SESSION var, and depending on the content execute a statement. At the same time I need to check for the case of the content.

Example: if the content of the $_SESSION var is 'changeme', in UPPER or LOWER CASE, redirect.

This is what I have:

if($_SESSION['update'] == 'changeme')
{
	$_SESSION['changeme'] = $_SESSION['_amember_user']['street'];
	if($_SESSION['changeme'] == 'changeme')
 		{
			 header("Location:profile.php");
			 exit;
 		} 
	if($_SESSION['update'] == 'changeme')
 		{
 			header("Location:profile.php");
			exit;
 		}
}		

Can someone please advise the best way to do this.

[Weedpacket]

Either use [man]strcasecmp[/man] or [man]strtolower[/man] (or [man]strtoupper[/man] if that takes your fancy).

PS: Your second if($_SESSION['update']=='changeme') is guaranteed to test true, and is therefore redundant.

[Ashley_Sheridan]

Yes, like Weedpacket said, the second if is redundant, so the third will never ever be processed. You could change the code to this:

if($_SESSION['update'] == 'changeme')
{
    $_SESSION['changeme'] = $_SESSION['_amember_user']['street'];
     if($_SESSION['update'] == 'changeme')
        {
            header("Location:profile.php");
            exit;
        }
       else
        {
            header("Location:profile.php");
             exit;
        }
} 

Note that I've not incorporated the comparison operators that Weedpacket suggested, so you will still have to do this.

[Weedpacket]

Of course, now both branches of the inner if statement are identical, so that test no longer serves a purpose 🙂

if($_SESSION['update'] == 'changeme')
{
    $_SESSION['changeme'] = $_SESSION['_amember_user']['street'];
    header("Location:profile.php");
    exit;
} 

[Ashley_Sheridan]

Well, I assumed the check was there for a reason, but the location had been mis-typed originally