[RESOLVED] conditional statments & array

jose07

Hey guys here is where i'm stuck:

i have adverts on the site but i have set permission on the adverts to be either 'public' or 'restricted' so based on that i want to create a statement with three possible outcomes, it should check the values of the columns and if the is an advert it will check the permission if the permission 0 (public) it shows the ad if permission 1 (restricted) it will direct the user to a log-in page however if the user is logged on then it will just show all the adverts.

These is the code i'm using but it is only executing the else line (so when the user clicks on the ad it takes them straight to the login page even if the ad is set to '0')....i checked the error log and i get the msg 😛HP Notice: Undefined variable: SiteDir in

code:

<?php
$qry_getAdverts = mysql_query("SELECT advertID,accesslevelID FROM adverts_adverts");

        $row_getAdvert = mysql_fetch_assoc($qry_getAdverts);       

    if($qry_getAdverts["AdvertID"] > 0 && ($qry_getAdverts["accesslevelID"] == 0 )) {

        echo($row_getAdvert["Content"]);

    }

else {

echo '<script type="text/javascript">window.location.href=\'' . $SiteURL . '/my-account/login.php\';</script>' ;
}

I tried put it as an array and use a while loop but still it didn't work

Any tips much appreciated!

Thanks

anakadote

Should be:

if($row_getAdvert["AdvertID"] > 0 && ($row_getAdvert["accesslevelID"] == 0 )) {

jose07

Thanks for the reply but unfortunately i tried and it stills behave the same way

anakadote

What do you get from this?

$qry_getAdverts = mysql_query("SELECT advertID,accesslevelID FROM adverts_adverts");
$row_getAdvert = mysql_fetch_assoc($qry_getAdverts);

print_r($row_getAdvert);

jose07

No still not working, the problem is that i think the if statment doesn't get the result of the query and so doesn't compare the values and only returns the value of the else and even if it prints the 'content' it doesn't recognise the accesslevel status.

dagon

anakadote's code is for outputing $row_getAdvert array (which you then post here), so we can see what the actual data is to help with debugging

jose07

Hi, Thanks very much for the reply, unfortunately i still can't get it to work as it seems it doesn't recognise the accesslevel and if it displays it displays all adverts and if it doens't display it doesn't display any advert which is obv showing that the accesslevelID param is not being recognised.

I have attached the whole page so you can have a look and maybe spot the error.

Thanks again for the help...

the code of the whole page

dagon

sigh, asked to see the output from the above code not the whole script.

jose07

I posted the whole script because as i explained earlier, that part of the code is only executing the else part which displays the loging page regardless of the accesslevel set for the advert. If the accesslevel of an article is 0 it should show the article but at the moment it doesn't. Ii i remove the else part it shows all the articles again withou considering accesslevel restriction

jose07

I have another question

I can't user headers because the script already uses other set headers which then conflict if i add another one on the if statement. my question is how can i direct the user to a particular page 'login' from the if statement without using javascript or headers? and if i have to use header how do i set it up so it doesn't conflict with the others already set?

else {

//echo '<script type="text/javascript">window.location.href=\'' . $SiteURL . '/my-account/login.php\';</script>';
header("location: {$SiteURL}/my-account/login.php");
exit();
}

bradgrafelman

jose07;10967872 wrote:
I can't user headers because the script already uses other set headers which then conflict if i add another one on the if statement.

That isn't true. You can set headers as many times as you want in the script as long as you haven't begun outputting anything.