Whats wrong with this?

[dudejma]

<?
/**
* User has already logged in, so display relavent links, including
* a link to the admin center if the user is an administrator.
*/
if($session->logged_in){
   echo "<h1>Logged In</h1>";
   echo "Welcome <b>$session->username</b>, you are logged in. <br><br>"
       ."[<a href=\"userinfo.php?user=$session->username\">My Account</a>]   "
       ."[<a href=\"useredit.php\">Edit Account</a>]   ";
   if($session->isAdmin()){
      echo "[<a href=\"admin/admin.php\">Admin Center</a>]   ";
   }
   echo "[<a href=\"process.php\">Logout</a>]";
       
}
else{
?>

<? if($session->logged_in){
   echo "<h2>Logged In</h2>";
      ."<a href="http://www.virtualamerican.org/filepirep.html">File a PIREP</a>";
      ."<a href="http://www.virtualamerican.org/logbooks/select.html">Logbook</a>";
?>

 

 

Mostly it's the second <? ?> part. If I change anything in that section, it tells me that the the problem is somewhere in where ever I move it. 

[Alex]

You have too many semicolons. You only place a semicolon at the end of a statement. Since you're concatenating a string, you don't put them there. You also need to escape th double quotes in the string.

 

echo "<h2>Logged In</h2>"
      ."<a href=\"http://www.virtualamerican.org/filepirep.html\">File a PIREP</a>"
      ."<a href=\"http://www.virtualamerican.org/logbooks/select.html\">Logbook</a>";