Hvordan får jeg en update til at virke i php

mysql_query("UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]' WHERE id = ".$_GET['id'].";") or die(mysql_error());

Giver følgene fejl You have an error in your SQL syntax near '' at line 1

ret:

while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
        echo $id;
}?>

til:

while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
        echo $id;
}?>

(Har sat ID ind som et hidden felt)

Så er din $_GET['id'] tom. Du skal have den med over, fx.

    echo "<form action=\"ret1.php?id={$_GET['id]}\" method=\"post\">";

echo "<form action=\"ret1.php?id={$_GET['id']}\" method=\"post\">";

Ah ja, hvis du bruger min skal du rette $_GET['id'] til $_POST['id'] i updaten ;)

You have an error in your SQL syntax near '' at line 1
samme fejl

Hvad har du gjort? :P

ja nu retter den men giver stadig samme fejl :)

Ah, fjern ; i din SQL statement ;)

Der er en del forslag ovenfor. Hvad har du gjort?

Hvis du brugte min:

mysql_query("UPDATE nyheder SET navn ='$_POST[navn]', SET overskrift ='$_POST[overskrift]' WHERE id = ".$_POST['id']) or die(mysql_error());

jeg har ændret get til post samt indsat ændringen
while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
        echo $id;
}?>

coder dk den du lige indsattte giver denne fejl

You have an error in your SQL syntax near 'SET overskrift ='skod' WHERE id = 329' at line 1

Du skal fjerne SET som jeg skrev først.

Prøv lige at rette den til:

echo $sql = "UPDATE nyheder SET navn ='$_POST[navn]', SET overskrift ='$_POST[overskrift]' WHERE id = ".$_POST['id'];
mysql_query($sql) or die(mysql_error());

Og skriv hvad den giver dig ;) (du kan fjerne echo når du har fået det til at virke - det er generelt pænt at lave statement først og så køre det bagefter)

doh! ja, erik har da ret, den overså jeg lige :)

så skriver den
UPDATE nyheder SET navn ='peter', SET overskrift ='skod' WHERE id = 329You have an error in your SQL syntax near 'SET overskrift ='skod' WHERE id = 329' at line 1

Denne skulle så virke ;)

$sql = "UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]' WHERE id = ".$_POST['id'];
mysql_query($sql) or die(mysql_error());

så skriver den
You have an error in your SQL syntax near '' at line 1
men hvis jeg bruger denne her så skriver den query was emty
mysql_query("UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]', tekst ='$_POST[tekst]' WHERE id = ".$_GET['id'].";") or die(mysql_error());
mysql_query($sql) or die(mysql_error());

Hvad laver du?!

$sql = "UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]' WHERE id = ".$_POST['id'];
mysql_query($sql) or die(mysql_error());

ja du har ret det var lige mig men hvis jeg indsætter
$sql = "UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]', tekst='$_POST[tekst]' WHERE id = ".$_POST['id'];
mysql_query($sql) or die(mysql_error());
får jeg stadig You have an error in your SQL syntax near '' at line 1

Har du ændret din <form> så der står noget i $_POST['id'] - vi kan ikke hjælpe dig
uden du viser hvad du har lavet.

ok det jeg har lavet på selve form siden ser således ud det skal dog siges scriptet virker men jeg får stadig fejlen :)

<?
$db = mysql_connect("database", "bruger","kode");
mysql_select_db("rdnwo_dk",$db);
$fore = mysql_query("SELECT * FROM nyheder WHERE id=".$_GET['id'].";") or die(mysql_error());
while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php?id={$_GET['id']}\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
     
}?>


scriptet der retter i databasen ser sådan ud



<?
$db = mysql_connect("database","bruger","kode");
mysql_select_db("rdnwo_dk",$db);
$sql = "UPDATE nyheder SET navn ='$_POST[navn]', overskrift ='$_POST[overskrift]',tekst='$_POST[tekst]' WHERE id = ".$_POST['id'];
mysql_query($sql) or die(mysql_error());
ECHO "Indlægget er nu rettet <a href=\"news.php\">Tilbage</a>";
her vil jeg gerne sende brugeren tilbage med en header("location:news.php") men hvis jeg gør det retter den ikke i indlægget
?>


Mvh
Niels Christiansen

Som jeg skrev, ændr:

$db = mysql_connect("database", "bruger","kode");
mysql_select_db("rdnwo_dk",$db);
$fore = mysql_query("SELECT * FROM nyheder WHERE id=".$_GET['id'].";") or die(mysql_error());
while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php?id={$_GET['id']}\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
   
}?>

til:

$db = mysql_connect("database", "bruger","kode");
mysql_select_db("rdnwo_dk",$db);
$fore = mysql_query("SELECT * FROM nyheder WHERE id=".$_GET['id'].";") or die(mysql_error());
while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php?id={$_GET['id']}\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"{$_GET['id']}\">";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
   
}?>

Eller bedre:

$db = mysql_connect("database", "bruger","kode");
mysql_select_db("rdnwo_dk",$db);
$fore = mysql_query("SELECT * FROM nyheder WHERE id=".$_GET['id'].";") or die(mysql_error());
while($data = mysql_fetch_array($fore)){
    echo "<form action=\"ret1.php\" method=\"post\">";
        echo "Overskrift<br><input type=\"text\" name=\"overskrift\" value=\"$data[2]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"{$_GET['id']}\">";
        echo "Navn:<br><input type=\"text\" name=\"navn\" value=\"$data[5]\"><br>";
        echo "<input type=\"hidden\" name=\"id\" value=\"$data[0]\">";
        echo "<textarea name=\"tekst\" rows=\"10\">$data[3]</textarea><br>";
      echo"<input type=\"submit\" name=\"et\" value=\"indsæt\">";
        echo "</form>";
        $id = $data[0];
 
}?>

hej jeg har fået det til at virke nogenlunde så det er brugbart men da jeg har brugt noget fra jer begge synes jeg det er rimeligt at i begge får point så hvis i gider give mig et svar så skal i få jeres point

:)
mvh
Niels

:)

du fik alle point coderdk men hvis du hører fra erikjacobsen må du give ham halvdelen syntes bare at nu er der næsten gået en uge og har ikke fået et svar fra ham endnu

GOD JUL

MVH
Niels Christiansen

Jeg samler slet ikke på point, så det er fint nok for mig.