[php/mysql] Update

[Verwijderd]

Ik heb een probleem met php/mysql updaten. Ik wil gewoon het tabel bot update dat doe ik op de volgende manier (zie script).
Maar op deze manier werkt het niet wat doe ik fout?
De variabele $edit = ?edit=id

// Edit het werkende script

PHP:

<?
if ($HTTP_COOKIE_VARS["gebruikersnaam"] == "")
{
    echo "To see this page you must be Admin!";
    exit;
}
?><?php

include "config.php";

$query=mysql_query("SELECT * FROM bot ORDER BY id DESC");
while ($row =@ mysql_fetch_array($query)) {
 $table_body .="<table border=\"1\" cellpadding=\"0\" cellspacing=\"0\" style=\"border-collapse: collapse\" bordercolor=\"#111111\" width=\"100%\">
  <tr>
    <td width=\"100%\"><font face=\"verdana\" size=\"2\">Bot Name: $row[name] | <a href=\"?edit=$row[id]\">EDIT</a></font></td>
  </tr>
</table><br>";}

echo "$table_body";


If ($save)
{
$sql = mysql_connect ("localhost","root","");
mysql_select_db ("msnshit_user", $sql);
$update = "UPDATE bot SET name='$name', url='$url', section='$section', text='$text' WHERE id=$edit";
$query = mysql_query($update);
$edit = addslashes($edit);
echo "$update";

exit;
}

$query=mysql_query("SELECT * FROM bot WHERE id=$edit");
while ($row =@ mysql_fetch_array($query)) {
 $table .="<table border=\"1\" cellpadding=\"0\" cellspacing=\"0\" style=\"border-collapse: collapse\" bordercolor=\"#111111\" width=\"100%\">
  <tr>
    <td width=\"100%\"><form method=\"post\" action=\"$PHP_SELF?edit=$edit\">
        Name:<br><input type=\"text\" value=\"$row[name]\" name=\"name\"><br>
        URL:<br><input type=\"text\" value=\"$row[url]\" name=\"url\"><br>
        Section:<br><select size=\"1\" name=\"section\">
        <option value=\"NickName\">NickName</option>
        <option value=\"AddOns\">AddOns</option>
        <option value=\"Other\">Other</option>
        <option value=\"Hacker\">Hacker</option>
        </select><br>
        Extra:<br>
        </font><textarea rows=\"10\" name=\"text\" cols=\"28\">$row[text]</textarea></font><br>
        <br>
        <input type=\"submit\" value=\"submit\" name=\"save\"></td>
  </tr>
</table><br>";

}
echo "$table";
exit;
?>

[LegacyCode]

$update = "UPDATE bots SET name=

Daar staat UPDATE bots SET ,
moet dat niet bot zijn ???

[thomaske]

vrij simpel:

1. lees de quickstart
3. gebruik
4. [php] tags ipv [code] tags
5. verwijder al je @-tekens (die zijn vies)
6. zet error_reporting(E_ALL) bovenaan je script
7. zet na iedere mysql_* functie: or die (mysql_error());
8. succes!

[Verwijderd]

Beetje onoverzichtelijk. Ken je aub [php] tags gebruiken?

[Sybr_E-N]

$update = "UPDATE bots SET name='$name', url='$url', section='$section', text='$text' WHERE id=$edit";

Ben bang dat de tabel bot moet zijn ipv bots..

[Verwijderd]

ja klopt ik heb dat veranderd en dan geeft ie de volgende error :
You have an error in your SQL syntax near '' at line 1

[Sybr_E-N]

Weet je zeker of zie update query wel klopt. Zo te zien print je die niet, voor het debuggen.
Als je nou :
echo $update;
voor/boven de mysql_query plakt, kun je zien of die query wel deugt.

[Verwijderd]

ja ik zie nu dat er iets misgaat hij krijgt die id=$edit niet door

$update = "UPDATE bots SET name='$name', url='$url', section='$section', text='$text' WHERE id=$edit";

ideeen?

[smaij]

zet is quotes ' om $edit heen

[smaij]

kijk ook eens naar de veiligheid van je script.

De variabele $edit = ?edit=id zeg jij.. is het niet slim om $edit te checken dan?

PHP:

$edit = addslashes($edit);

kom je al een heel end.

[Verwijderd]

zet is quotes ' om $edit heen

nope dat heeft geen nut

[smaij]

nog iets..

je hebt in je script alleen het volgende staan

<form method=\"post\" action=\"$PHP_SELF\">

maar daar is snel iets anders van dan. omdat je nu $edit niet doorgeeft aan je formulier.

dat kan op 2 manieren

1. hidden fields met naam edit en value = id
2. $PHP_SELF?edit=$edit te gebruiken.. ik neem aan dat $edit op dat moment de ID bevat..

[Verwijderd]

hij werkt nu bedankt!