Select and delete from database

Prøv at sætte ' eller " om $_POST[overskrift].

Kan se at du hopper meget ind og ud af php. Hver gang koster det lidt tid (plus det tager længere tid at skrive ;o)), det betyder nok ikke så meget i dit tilfælde, men i et andet sammenhæng kunne det måske være relevant.

Eks.

    } ?>
    <?php
      }
      ?>

I stedet for:
    }
      }
      ?>

Og i stedet for at afslutte hver gang man skal have html, kan man (en smagssag hvad man vil):

echo "<option value="".$_POST['overskrift']."">".$_POST['overskrift']."</option>";

Grunden til jeg går så meget ud og ind af php, er egentlig mere så jeg bedre lige kan overskue hvilke tuborg-klammer der hører til hvad. Men det med html er bare fordi jeg ikke rigtig kan finde ud af at skrive det ordentligt mht. at lave ' og ".

Det har jeg prøvet :\

Du har også glemt at sætte en action på:

if(isset($_POST['Slet'])){
mysql_query("DELETE FROM nyheder WHERE overskrift='".$_POST['overskrift']."'") or die(mysql_error());
}


Noget á la det.

@michael
Se spørgers holdning til point:
http://www.eksperten.dk/spm/929657

Det er fordi der er ingen andre jeg kunne trykke "svar" til..
Og folk klager også hvis man har point der bare står ude.

Hvad er dit problem? Giver med glæde de her virtuelle points, hvis der er nogen der kan hjælpe mig.

Og forresten så har jeg fundet en løsning  selv :)

1. Så kan du jo bede folk om at lægge et svar, som du så kan acceptere. Det er normalt at man kun lægger svar hvis man har absolut endelige løsning, eller at man er blevet bedt om det.

2. Folk klager hvis man har mange point ude, uden at have bedt folk om at lægge svar eller vende tilbage. Ikke når der er blevet bedt om et svar hos en og der så ikke er sket noget i en uge.

3. Ekspertens FAQ siger at man i så fald at man har fundet løsningen selv skal skrive den, på den måde kan det gavne andre der står i samme problem.

Fair, fandt selv ud af den her:

<?php 
 
    $qry="SELECT * FROM members WHERE login='".$_SESSION['SESS_LOGIN']."' AND admin='ja'"; 
    $result=mysql_query($qry); 
   
    if ($_SESSION['SESS_ADMIN'] == "ja") { 
  ?>
    <h1>Slet nyheder</h1> 


<?php 
              if (!isset($_POST["id"])) { 
              ?>
                  <form method="post" action=""> 

        <select name="id" id="id"> 
        <?php
             
    $foresp = mysql_query("SELECT id, overskrift FROM nyheder ORDER BY dato DESC"); 
    while ($data = mysql_fetch_array($foresp)) { 
     
?>
            <option value="<?php echo $data["id"]; ?>"><?php echo $data["overskrift"]; ?></option> 
<?php 
    } 
?> 
        </select> 
<br />
        <input value="Delete" type="submit"> 
    </form> 

<?php 

        }  else {
            $id = mysql_real_escape_string($_POST["id"]); 
            mysql_query("DELETE FROM nyheder WHERE id='" . $id . "'"); 
            echo "The news has been deleted";
            ?>
            <meta http-equiv="refresh" content="2; URL=?<?php echo $sted; ?>=delnews">
            <?php
        }
   
    } else { 
        echo "No access granted."; 
    } 
?>

..

Det er fint det virker. Men det er vist ikke helt W3C valid at du har et meta-tag i din body. Den skal være i headeren.