simple if statement

[keyser_soze]

this is supposed to on;y shot the admin panel for one user (geronimo), but it ends up showing the panel for everyone.

<?
          if ($valid_user == "geronimo" || "Geronimo")
     {
 ?>


 <br><!-- admin panel -->

	<table width="400" cellspacing="0" cellpadding="0" border=0>
		<!-- table -->
	<tr><td height="4"></td></tr>
	  <tr><td>
		<table border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse" width="175">
        <tr>
          <td width="5" height="5">
          <img border="0" src="images/sub2-topleft.gif" width="5" height="5"></td>
          <td background="images/sub2-top.gif"></td>
          <td width="5" height="5">
          <img border="0" src="images/sub2-topright.gif" width="5" height="5"></td>
        </tr>
        <tr>
          <td width="5" background="images/sub2-left.gif"></td>
          <td bgcolor="#515151" valign="top">
		  <div align="center">
          <table border="0" cellpadding="2" cellspacing="0" style="border-collapse: collapse" width="100%">
            <tr>
              <td width="100%">
			  <table width="100%" cellspacing="0" cellpadding="1" class="sub">
			  <tr><td colspan="2" align=center><p class="head">Admin Panel<br></td></tr>
			  <tr><td><table width=100% border=0><p>
			  - <a href="admin/frame_news.html" target="iframe2">News</a> <br>
			  - <a href="admin/frame_battles.html" target="iframe2">Battles</a> <br>
			  - <a href="admin/frame_ladders.html" target="iframe2">Ladders</a><br>
			  - <a href="admin/frame_schedule.html" target="iframe2">Members Schedule</a>
			  </table>
						</td></tr>
			  </table>				  
			  </td>
            </tr>
          </table>
		  </div>
          </td>
          <td width="5" background="images/sub2-right.gif"></td>
        </tr>
        <tr>
          <td width="5" height="5">
          <img border="0" src="images/sub2-bottomleft.gif" width="5" height="5"></td>
          <td background="images/sub2-bottom.gif"></td>
          <td width="5" height="5">
          <img border="0" src="images/sub2-bottomright.gif" width="5" height="5"></td>
        </tr>
		</table>          
		</td>

		<td>
		<table border="0" cellpadding="0" cellspacing="0" style="border-collapse: collapse" width="175">
        <tr>
          <td width="5" height="5">
          <img border="0" src="images/sub2-topleft.gif" width="5" height="5"></td>
          <td background="images/sub2-top.gif"></td>
          <td width="5" height="5">
          <img border="0" src="images/sub2-topright.gif" width="5" height="5"></td>
        </tr>
        <tr>
          <td width="5" background="images/sub2-left.gif"></td>
          <td bgcolor="#515151" valign="top">
		  <div align="center">
          <table border="0" cellpadding="2" cellspacing="0" style="border-collapse: collapse" width="100%">
            <tr>
              <td width="100%">
			  <table width="100%" cellspacing="0" cellpadding="1" class="sub">
			  <tr><td colspan="2" align=center><p class="head">Admin Panel<br></td></tr>
			  <tr><td><table width=100% border=0>
			  <iframe src="admin/blank.html" height=60 width=150 frameborder=no scrolling=auto name="iframe2"></iframe>
			  </table>
						</td></tr>
			  </table>				  
			  </td>
            </tr>
          </table>
		  </div>
          </td>
          <td width="5" background="images/sub2-right.gif"></td>
        </tr>
        <tr>
          <td width="5" height="5">
          <img border="0" src="images/sub2-bottomleft.gif" width="5" height="5"></td>
          <td background="images/sub2-bottom.gif"></td>
          <td width="5" height="5">
          <img border="0" src="images/sub2-bottomright.gif" width="5" height="5"></td>
        </tr>
		</table>          
		</td>

		</tr>
		<!-- /table -->
		</table>

		<!-- Admin panel -->


 <?
     }
	 else
	 {
	 echo "&nbsp;";
	 }
?>

i have no clue what to do.
thanks

[nnichols]

Change your if to this -

<?
          if ($valid_user == 'geronimo' || $valid_user == 'Geronimo')
     {
 ?>

Hope this helps 😉

If it doesn't work try echoing $valid_user before the if so you can check that the variable is set in the code.

[demonicfoetus]

dang, u posted just before i was about to, oh well.
You could also try

<?
if ($valid_user == 'geronimo' || $valid_user == 'Geronimo')
{
  echo "Your html but replace all " (speech marks) with \"); 
else
  echo "That is not a valid username";
?>

[keyser_soze]

yup that worked

thanks

[suntra]

Maybe you don't want this, but... but !

You could also use this code to see if $valid_user is geronimo, whatever it has been typed with capital letters (GERONIMO)...

<?
if(strtolower($valid_user) == 'geronimo') {
// blablabla
} else {
// blablabla
}
?>