Script PHP, MYSQL UPDATE funktioniert nicht

[j.durini]

Hallo zusammen,

ich habe folgedes Script es Ruft Daten aus der Datebank auf und lässt sie in einer Textaria bearbeiten.

if($_GET["edit"]==1){



$query = mysql_query("SELECT * FROM aktuelles;");
$count = mysql_num_rows($query);
for($i=0;$i < $count; $i++) {
$query1 = mysql_query("SELECT * FROM aktuelles LIMIT $i, $count;");
$result = mysql_fetch_object($query1);
    $datenText = $result->text;
    $datenDatum = $result->datum;
    $datenId = $result->id;
?>
Inhalte <? echo $i; ?><br><br>
<form method="POST" action="formular.php">
    <input name="id" type="hidden" value="<?php echo $datenId; ?>">
    <input name="text" value="<?php echo $datenText; ?>"><br>
    <input name="datum" value="<?php echo $datenDatum; ?>"><br><br>
    <input type=submit name=submit value="Formular absenden">
</form>
<?}?>

 <?php

 
 if(isset($_POST["text"]) && isset($_GET["edit"]))
 {
   $text = $_POST["text"];
   $datum = $_POST["datum"];
   $id = $_POST["id"];
   
   
   //echo("INSERT INTO aktuelles (id, bild, titel, text, datum, aktiv) VALUES ('".$id."', '".$bild."', '".$titel."', '".$text."', '".$datum."', '".$aktiv."')");
   mysql_query("UPDATE aktuelles SET bild = '$bild', titel = '$titel', text = '$text', datum = '$datum', aktiv = '$aktiv' WHERE id = '$id';");
 }
 } //aendern schließen
 ?> 
 <?
  if($_GET["add"]==1){?>
<form method="POST" action="formular.php?add=1">
    <input name="text" value="<?php echo $datenText; ?>"><br>
    <input name="datum" value="<?php echo $datenDatum; ?>"><br><br>
    <input type=submit name=submit value="Formular absenden">
</form>
 <?php

 
 if(isset($_POST["text"]) && isset($_GET["add"]))
 {
   $text = $_POST["text"];
   $datum = $_POST["datum"];
   
   
   
   mysql_query("INSERT INTO aktuelles (id, bild, titel, text, datum, aktiv) VALUES ('0', '".$bild."', '".$titel."', '".$text."', '".$datum."', '".$aktiv."')");
  
 }
 }
  //normal
  ?>
<a href="?edit=1">bearbeiten</a>
<a href="?add=1">hinzuf&uuml;gen</a>

Verbindung und soweiter steht alles. Es können Beiträge hinzugefügt werden aber die Beiträge nicht bearbeitet werden was ist falsch ?

[TessaKavanagh]

mysql_query("INSERT INTO aktuelles (id, bild, titel, text, datum, aktiv) VALUES ('0', '".$bild."', '".$titel."', '".$text."', '".$datum."', '".$aktiv."')");
mysql_query("UPDATE aktuelles SET bild = '$bild', titel = '$titel', text = '$text', datum = '$datum', aktiv = '$aktiv' WHERE id = '$id';"); 

Vergleich doch bitte mal beide Zeilen und sag mir wo der entscheidende Unterschied liegt Als kleiner Tipp es hat mit den Variablen zu tun und sollte durch das Highlighting des Quelltextes durchaus auffallen.

Davon abgesehen solltest du nicht mehr auf my_sql sondern auf my_sgli oder noch besser PDO setzen. Denn my_sql ist veraltet. Ebenfalls ist dein Code anfällig für SQL Injection. Niemals User Eingaben ungeprüft verwenden.

[j.durini]

Ja es sind nur text und Datum belegt..aber mehr wollte ich zum testen nicht belegen ist ja nicht weiter schlimm oder ?

[TessaKavanagh]

Schau dir bitte man die Farben im Code an. Es hat meistens seine Gründe wenn etwas einmal Blau und einmal Rot gehighlightet ist. Es geht hier nicht darum das du nur zwei Variablen verwendest sondern darum das du einmal im Ergebnis gar keine Variablen sondern einen string verwendest.

[j.durini]

Die Punkte sind oben bei den Variablen ja ?

Hä ich stehe gerade voll auf dem Schlauch.

[TessaKavanagh]

Ok also:

mysql_query("INSERT INTO aktuelles (id, bild, titel, text, datum, aktiv) VALUES ('0', '".$bild."', '".$titel."', '".$text."', '".$datum."', '".$aktiv."')"); 
mysql_query("UPDATE aktuelles SET bild = '".$bild."', titel = '".$titel."', text = '".$text."', datum = '".$Datum."', aktiv = '".$aktiv."' WHERE id = '".$id."';"); 

und jetzt schau bitte mal wo der unterschied zu deinem Code ist.

[j.durini]

Habs jetzt selber gelöst war nicht das was du beschrieben hattest sondern hier

<form method="POST" action="formular.php?edit=1"> 

das edit fehlte...Danke trotztdem

[Tera3yte]

Du solltest den Rat von Tessa dennoch beherzigen

EDIT:

Besonders das mit mysqli und PDO.

[tk1234]

Schau dir bitte man die Farben im Code an. Es hat meistens seine Gründe wenn etwas einmal Blau und einmal Rot gehighlightet ist. Es geht hier nicht darum das du nur zwei Variablen verwendest sondern darum das du einmal im Ergebnis gar keine Variablen sondern einen string verwendest.

Dir ist schon klar dass Variablen in doppelten Anführungszeichen ersetzt werden, oder?