Det er jo også det, som filen viser? Efter alt din PHP kode der afslutter du med hele din login form, så er det da klart at den kommer hver gang.
Det du skal gøre, er at afslutte din php kode af med en if, som starter, og efter din formular, skal du så ende din if.
Jeg har ændret lidt i din kode, her er den:
- <?php
- include("connect.php");
- if($_POST[action] == "login") {
-
- $db_hent11 = mysql_query("select * from bans where ip = '$_SERVER[REMOTE_ADDR]'");
- $antaq = mysql_num_rows($db_hent11);
- $slo = mysql_fetch_array($db_hent11);
- if($antaq > '0'){
- print "<font size=10>Banned!</font><br><br>$slo[why]";
- }else{
- $db_hent = mysql_query("select * from brugere where brugernavn = '$_POST[brugernavn]'");
- $hent = mysql_fetch_array($db_hent);
- $din_ip = "Lukket";
- if($hent[Rang] == $din_ip){
- print "Brugeren er lukket!";
- }else{
- if($hent[r] == '0'){
- print "Din bruger <u>SKAL</u> være aktiveret!<br><b>Sp:</b> Hvorfor?<br><b>Sv:</b> Fordi at vi kun vil have seriøste medlemmer :-)";
- }else{
- if($_POST[brugernavn] == ''){
- print "Fejl!";
- }else{
- if($_POST[brugernavn] == ' '){
- print "Fejl!";
- }else{
- if($_POST['brugernavn'] == $hent['brugernavn'] && $_POST['adgangskode'] == $hent['adgangskode']){
- $_SESSION['brugernavn'] = $hent['brugernavn'];
- $_SESSION['Rang'] = $hent['Rang'];
- $_SESSION['konto'] = $hent['point'];
- $_SESSION['logintid'] = time();
- $tid = time();
- mysql_query("UPDATE brugere SET online = 'ja' WHERE brugernavn = '$_SESSION[brugernavn]'");
- mysql_query("UPDATE brugere SET loggetind = '1' WHERE brugernavn = '$_SESSION[brugernavn]'");
- mysql_query("UPDATE brugere SET logintid = '$tid' WHERE brugernavn = '$_SESSION[brugernavn]'");
- print "test";
- $dato = date("d/m/Y H:i:s");
- mysql_query("INSERT INTO tagwall (af, dato, besked, farve) VALUES ('Robot', '$dato', '<b>$_SESSION[brugernavn]</b> er netop logget ind!', 'orange')") or
-
- die(mysql_error());
- // du er åbenbart logget ind, efter al den mærkelige kode... derfor!
- $logget_ind=1;
- }else{
- print "Forkert brugernavn/adgangskode!";
- }
- }
- }
- }
- }
- }
- }
-
- if($logget_ind!=1)
- {
- // hvis man ikke er logget ind, så skal formularen vises!
- ?>
-
- <form action="index.php" method="POST">
- <input type="hidden" name="action" value="login">
-
- <b>Brugernavn:</b><br>
- <input type="text" name="brugernavn" style="border: 1px solid #8FA38C; background-color: #A5CDA2; top: 1px;">
- <br><br>
-
- <b>Adgangskode:</b><br>
- <input type="password" name="adgangskode" style="border: 1px solid #8FA38C; background-color: #A5CDA2; top: 1px;">
- <br><br>
-
- <input type="submit" name="submit" value="Log ind" style="border: 1px solid #8FA38C; background-color: #A5CDA2;">
- <input type="button" value="Opret" onClick="parent.main.location='community/opret.php'" style="border: 1px solid #8FA38C; background-color: #A5CDA2;">
- </form>
- <?php } ?>
Vil forresten foreslå at lave en mere overskuelig kode, synes den er meget kryptisk, og langstrukken..
Allerede i starten gør du det svært for dig selv, med at starte php koden, og lukke igen, og starte igen, uden noget html eller noget som helst, ind i mellem.
- <?php include("connect.php"); ?>
-
- <?php
- if($_POST[action] == "login") {
Indlæg senest redigeret d. 07.02.2009 21:04 af Bruger #8782