Error Help/

[Colin-Roberts]

I've looked over this and no idea why.
If i login using the correct info it say's loginning in but then goes back to the form instead of showing the links..

<?PHP



 $connection = mysql_connect("",
                            "",
                            "");
mysql_select_db("", $connection);












$id = $_REQUEST['id'];
switch($id){
















default:
if($_SESSION['zetolic_user']){
$select = mysql_query("SELECT * FROM `users` WHERE `username`='$_SESSION[zetolic_user]'");
$fetch = mysql_fetch_array($select);
echo"<center><font face='verdana' size='1'>
<b>Welcome,<a href='?page=member&id=" . $_SESSION[zetolic_user] . "'> $_SESSION[zetolic_user]</b></a><br>
Current  Zoints: <b>$fetch[coins]</b><br><br>
<a href='?page=function&id=trade'>Trade  Zoints</a><br>
<a href='?page=function&id=profile'>Edit Profile</a><br>";
if($fetch[rank] == 3){
if($_SESSION[zetolic_admin]){
    echo"<a href='?page=admin&id=list'>UserList</a><br>
<a href='?page=admin&id=search'>Search Users</a><br>
";
}
else {
    
















echo"<a href='?page=admin'>Admin Login</a><br>";
    }
}
















echo"
<a href='?page=function&id=logout'>Logout</a>
</font></center>";
}
else {
echo"<center><font face='verdana' size='1'><form action='?page&id=login' method='POST'>
Username :<br><input name='zetolic_user'><br><br>
Password :<br><input type='password' name='zetolic_pass'><br>
<input type='Submit' value='Login'></form><br><br>
<a href='?page=register'>Register</a></font></center>";
















}
















break;
















case "login":
$user = $_POST['zetolic_user'];
$pass = md5($_POST['zetolic_pass']);
$select = mysql_query("SELECT * FROM `users` WHERE `username`='$user' AND `password`='$pass'");
$count = mysql_num_rows($select);
if($count == '0'){
echo"<meta http-equiv='REFRESH' content='0;url=?'>";
echo"<center><font face='verdana' size='1'>Invalid Username/Password</center></font>";
}
else {
echo"<meta http-equiv='REFRESH' content='0;url=?'>";
echo"<center><font face='verdana' size='1'>Loggining in</center></font>";
$_SESSION['zetolic_user'] = $user;
}
break;
































}

[Dentafrice]

I'd be happy to help you, but that coding is just too awful.. not to mention messy.. <?PHP -> <?php first off

Try to clean it up a bit, and remove the 6 miles of whitespace.

[Jahova]

Horrible PHP coding.
That is worse than mine and mine is pretty bad.
Man, try and clean up that, or no-one will help you.

[Excellent]

<?php
$connection = mysql_connect("","","");
mysql_select_db("", $connection);
$id = $_REQUEST['id'];
switch($id){

default:
if($_SESSION['zetolic_user']){
$select = mysql_query("SELECT * FROM `users` WHERE `username`='$_SESSION[zetolic_user]'");
$fetch = mysql_fetch_array($select);
echo"<center><font face='verdana' size='1'>
<b>Welcome,<a href='?page=member&id=" . $_SESSION[zetolic_user] . "'> $_SESSION[zetolic_user]</b></a><br>
Current  Zoints: <b>$fetch[coins]</b><br><br>
<a href='?page=function&id=trade'>Trade  Zoints</a><br>
<a href='?page=function&id=profile'>Edit Profile</a><br>";
if($fetch[rank] == 3){
if($_SESSION[zetolic_admin]){
echo"<a href='?page=admin&id=list'>UserList</a><br>
<a href='?page=admin&id=search'>Search Users</a><br>";
}
else {
    
echo"<a href='?page=admin'>Admin Login</a><br>";

    }
}

echo" <a href='?page=function&id=logout'>Logout</a>
</font></center>";
}
else {
echo"<center><font face='verdana' size='1'><form action='?page&id=login' method='POST'>
Username :<br><input name='zetolic_user'><br><br>
Password :<br><input type='password' name='zetolic_pass'><br>
<input type='Submit' value='Login'></form><br><br>
<a href='?page=register'>Register</a></font></center>";

}

break;
case "login":
$user = $_POST['zetolic_user'];
$pass = md5($_POST['zetolic_pass']);
$select = mysql_query("SELECT * FROM `users` WHERE `username`='$user' AND `password`='$pass'");
$count = mysql_num_rows($select);
if($count == '0'){
echo"<meta http-equiv='REFRESH' content='0;url=?'>";
echo"<center><font face='verdana' size='1'>Invalid Username/Password</center></font>";
}
else {
echo"<meta http-equiv='REFRESH' content='0;url=?'>";
echo"<center><font face='verdana' size='1'>Loggining in</center></font>";
$_SESSION['zetolic_user'] = $user;
}
break;
} 

?>

Personal preference.

[Jahova]

Much cleaner =D

[Excellent]

Much cleaner =D

If the coding was better I would have spent more time cleaning it up.

[Jackboy]

I think it looks neat with capitals..

I got some addiction now of doing Mysql_Fetch_Array(); etc

It looks smart i think? :S

but yeh anyway

And you have coded it weirdly imo

[Invent]

That looks awful Jack

[PreProcessor]

As said that has been coded terribly, I can't even help you because I cba going through thousands of blank lines. You should try recoding it.

[Independent]

That looks awful Jack

Agreed.

I thought functions we're case sensitive