urgent help with a few things

goodwin69

hi everyone i know this is my first post but i need some help with my site ive built.

for starters , i have on my homepage join and login , when someone logs in i want the join part to dissapear and the login to say profile ...

and also the pics i have on the website , when you go through the website like click on photos and view them you cannot view them if your not logged in , but if you type the url of the picture in you can view it , how can i stop that?

any help would be much appreciated.

thanx in advance

dagon

since there's no code to work with ...

if(how ever you detect they are logged in){
//show profile
}else{
//show login
}

goodwin69

ok heres my code but it says itsa coming up with a unexpected error <

<?
if(!$session->logged_in){
<table width="452" border="0" align="center">
<tr class="nav">
<td width="76"><a href="../index.php">Home</a></td>
<td width="89"><a href="../main.php">Profile</a></td>
<td width="93"><a href="../sets.php">Sets</a></td>
<td width="86"><a href="../videos.php">Videos</a></td>
</tr>
</table>
}else{
<table width="452" border="0" align="center">
<tr class="nav">
<td width="86"><a href="../register.php">Join</a></td>
<td width="89"><a href="../main.php">Login</a></td>
</tr>
</table>
}
?>

Weedpacket

If you use the formatting tags described in the FAQ, you'll see that your code completely misses the distinction between PHP and HTML. Basically, you're short some ?> and <?php.

On a side note, using tables like that is soooo last century; an appropriately-styled list would be more appropriate for displaying a list.

goodwin69

weedpacket thank you for your reply , but the nav fits in with the theme , all i want to know is how to get rid of certain links when logged in but when not logged in i want them to be shown.

example(when not logged in)
JOIN | LOGIN

(when logged in)

HOME | MEMBERS | SETS | VIDEOS

i want to hide certain links from non members.

thanx in advance

goodwin69

any help please? i know its simple but just cant find a way to do it.

dagon

<?php
if(!$session->logged_in){
?>
<table width="452" border="0" align="center">
<tr class="nav">
<td width="76"><a href="../index.php">Home</a></td>
<td width="89"><a href="../main.php">Profile</a></td>
<td width="93"><a href="../sets.php">Sets</a></td>
<td width="86"><a href="../videos.php">Videos</a></td>
</tr>
</table>
<?php
}else{
?>
<table width="452" border="0" align="center">
<tr class="nav">
<td width="86"><a href="../register.php">Join</a></td>
<td width="89"><a href="../main.php">Login</a></td>
</tr>
</table>
<?php
}
?>

Weedpacket

That's what I get for saying two things in one post again...I'll never learn.

johanafm

goodwin, weedpacket did not comment on the table-style menu from any kind of visual design perspective, but rather due to how html is supposed to be used. Since html elements have semantical meaning, you're supposed to stick to tags when they make sense for your usage. A menu is not a table. It is however an ordered or unordered list. And even if you make a table and ul look identical, a screen reader won't notice.

Weedpacket, what you need to do is answer all questions asked. You also need to break up the answers, one per post.
I just had to find out how weedpacket himself reads posts 😉

goodwin69

thanx for all your posts but even with dagons code im still getting , HOME PROFILE SETS and VIDEOS when im not logged in when i should be getting JOIN and LOGIN when im not logged in , its really baffling me now.

goodwin69

nav.php (which is <? include "nav.php"; ?> in index.php)

<?php
if(!$session->logged_in){
?>
<table width="452" border="0" align="center">
<tr class="nav">
<td width="76"><a href="../index.php">Home</a></td>
<td width="89"><a href="../main.php">Profile</a></td>
<td width="93"><a href="../sets.php">Sets</a></td>
<td width="86"><a href="../videos.php">Videos</a></td>
</tr>
</table>
<?php
}else{
?>
<table width="452" border="0" align="center">
<tr class="nav">
<td width="86"><a href="../register.php">Join</a></td>
<td width="89"><a href="../main.php">Login</a></td>
</tr>
</table>
<?php
}
?>

johanafm

if(!$session->logged_in){

! means not. So the english translation of that code is:
if( NOT $session->logged_in) { ...

goodwin69

so what would be the code for it?

goodwin69

ok removing the ! from the session logg_in part has took it so when im logged out it shows JOIN and LOGIN , but when i login it is still showing the same JOIN and LOGIN and not HOME PROFILE SETS VIDEOS.

goodwin69

it seems like i can only have one navigation , i cannot have both, its either home and login or home profile sets and videos. cant have both.

Weedpacket

Then there's something wrong with the value of $session->logged_in: it's not being set correctly.

goodwin69

right the only thing i can see what is wrong with it is that im not including my sessions.php at the head of the page but when i do do that i get a erro saying

Fatal error: Cannot redeclare class Session in blah/blah/blah/include/session.php on line 16

in my sets.php i have this code to make it a private page and it works fine

<?
include("include/session.php");
?>

<html>
<title>Protected Page</title>

<?
/ User is not logged in /
if(!$session->logged_in){
?>

that is how i protect my sets.php being viewed by non members , ive tried the same way but when i include session.php i get that fatal error message.

and the line it says what the error is coming from this this line.

class MySQLDB
{ (this is the error line)
var $connection; //The MySQL database connection
var $num_active_users; //Number of active users viewing site
var $num_active_guests; //Number of active guests viewing site
var $num_members; //Number of signed-up users

johanafm

It's not possible that you are referring to the same line as the php parser reports. If you are showing the correct line NUMBER, then you are not showing the correct FILE. What happens is this

class Session {

}

class Session {

}

Cannot redeclare class Session on line 5.

The most likely reason this is happening is that you're running in circles with includes/requires. You might want to change all of them into include_once and require_once. For something like session.php, I'd definitely go with require, since it would indeed be a fatal error if it is missing, not just a minor warning.
However, while require_once would solve the issue at hand since PHP would deal with not including the same file twice, you obviously have a logical error somewhere along the line or you wouldn't be including that file twice. So you should look over your code again.

Ashley_Sheridan

OK, what happens when you run this code when you are not logged in and when you are?

echo ($session->logged_in)?'123':'456';

Make sure you actually run the test and don't just reply back with what you think you should get.