Formular Updaten

[Nimmert]

Hallo,

Ich hab ein Formular gemacht womit man eig Updaten sollte was es aber net tut :-/
Vielleicht köönt ihr mir ja sagen wo der fehler ist

Formular.php

<form action="../conf/adminupdate.php" method="post">
Nickname: <input type="text" name = "name" value='<?php echo $_SESSION["user_name"]; ?>'><br>
Passwort: <input type="text" name = "kennwort" value='<?php echo $_SESSION["user_kennwort"]; ?>'><br>
Level:
<select name="level" size="1">
<option value="1">1</option>
<option value="2">2</option>
<option value="3">3</option>
</select><br>
<input type="hidden" name = "id" value='<?php echo $_SESSION["user_id"]; ?>'>
<input type="submit" value="Update">
</form>

adminupdate.php

<?
$id=$_POST['id'];
$name=$_POST['name'];
$kennwort=$_POST['kennwort'];
$level=$_POST['level'];
mysql_connect("versteckt", "nimmert", "A123456Z") or die(mysql_error());
mysql_select_db("nimmert") or die(mysql_error());
mysql_query("UPDATE net_admin SET ('$id', '$name', '$kennwort', '$level') WHERE .$id.");

Print "<font face='Arial'><font color='limegreen'><center>Erfolgreich gesendet</font></font>";
?>

 

[mermshaus]

Lass dir mal die Query ausgeben, bevor du sie abschickst.

UPDATE-Syntax ist hier beschrieben:

- MySQL :: MySQL 5.0 Reference Manual :: 12.2.10 UPDATE Syntax

Außerdem ist dein Code anfällig für SQL-Injections.

- SQL-Injection

 

[Nimmert]

Okay hab sql-injection verstanden.. aber wie wandel ich das jetzt um in update ?

$abfrage = "SELECT '$id', '$name', '$kennwort', '$level'
FROM net_admin
WHERE id = '".mysql_real_escape_string($_POST['id'])."'";
$query = mysql_query($abfrage) or die("Datenbankabfrage ist fehlgeschlagen!");

 

[threadi]

Woher kommt jetzt das Select-Statement? Du hast doch oben geschrieben, dass es um

mysql_query("UPDATE net_admin SET ('$id', '$name', '$kennwort', '$level') WHERE .$id.");

geht? Und das ist falsch aufgebaut, daher auch der Hinweis von mermshaus.

Richtig:

mysql_query("UPDATE net_admin SET name = '$name', kennwort = '$kennwort', level = '$level' WHERE id = ".$id);

wobei Du die Spaltennamen natürlich an deine Anforderungen anpassen musst.