What is this line doing??

[Struggling]

I am trying to make it so that only people with access rights of A can access a particular part of the site. But I ma struggling to get it to work.

Where am I going wrong?? The code below is what I am trying to use

Cheers

<?php
session_start();
if ($_SESSION["accessrights"] === 'B' || 'C'){
header ( 'location: /' );
exit;
}
?>

[World Wide Weird]

It checks whether a session variable called "accessrights" is either "B" or "C", and if so, redirects the user to the site's home page and terminates the script.

[spikeZ]

pretty sure you cant short cut in that way.

if ($_SESSION["accessrights"] == 'B' || $_SESSION["accessrights"] == 'C'){ 

and === means identical.

[Struggling]

right so it should work shouldn't it. If i have access right so f A then i should be able to get past that page, but if i have b or c then i can't and will not be able to enter.

Cheers

[imaginethis]

I am trying to make it so that only people with access rights of A can access a particular part of the site. But I ma struggling to get it to work.

Where am I going wrong?? The code below is what I am trying to use

Cheers

<?php
session_start();
if ($_SESSION["accessrights"] === 'B' || 'C'){
header ( 'location: /' );
exit;
}
?>

if ($_SESSION["accessrights"] === 'B' || 'C') is illogical. You cant use compound statements like this in branch declarations. The correct statement,

if ($_SESSION["accessrights"] === 'B' || $_SESSION["accessrights"] === 'C')

[Struggling]

cheers guys go that sorted now.

Thanks

[Stormrider]

If you only want people with access rights of A to access it, wouldnt it be better to use

$_SESSION["accessrights"] !== 'A' 

?

What happens if you add a D later on? They would be able to access the page as well, and you probably don't want them to.

[Cups]

How about:

if ($_SESSION["accessrights"] !== 'A' ){
// relocate and exit

} else {
// loads of code 

}