Newseintrag in Datenbank bearbeiten

[falscherprophet]

Heyho,
ich bin gerade dabei mir selbst ein Script für meine Seite zu schreiben mit ACP etc und habe daher mal mit Datenbanken angefangen. Ich habe die Tabelle "News" (beinhaltet Titel, Datum, Inhalt, Timestamp und ID), worein per Formular (welches bestens funktioniert) Newseinträge eingetragen werden, welche dann auf der Mainpage angezeigt werden. Soweit alles gut, auch das Datenbanken löschen klappt soweit, nur wenn ich auf den Bearbeiten Button klicke und dann halt eine textarea mit dem Inhalt des Newsfeldes erscheint und ich den Text ändern will und auf abschicken klicke, bekomme ich folgenden Error:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 3

Der Code lautet wie folgt , der schreiben soll:

<?php 
    include('dbconnect.php'); 
    $date = date('d.m.Y'); 
    if($_GET['action'] == 'edit'){

    $sql = mysql_query("SELECT * FROM News WHERE ID = ".$_GET['id']) or die(mysql_error());
    $data = mysql_fetch_array($sql);
?>
<form name="news" method="post" action="<?php echo $_SERVER['SCRIPT_NAME']; ?>?action=<?php echo $_GET['action']; ?>&id=<?php echo $_GET['id']; ?>">
<textarea name="content" rows="5" cols="35" ><? echo $data['Inhalt'];  ?></textarea> 
<input type="submit" value="Abschicken" accesskey="s">
</form>   
<?php
        if ($_POST) {
    $sql2 = mysql_query("UPDATE
                         News
                        SET
                         Inhalt = '".$_POST['content']."' 
                        WHERE ID = ".$_GET['id']) 
          or die(mysql_error());
          echo 'UPDATE 
                 News 
                SET 
                 Inhalt = '.$_POST['content'].' 
                WHERE ID = '.$_GET['id'];
        }
    }
    if($_GET['action'] == 'delete'){
    $sql3 = mysql_query("DELETE FROM
                      News
                     WHERE ID = ".$_GET['id'])
          or die(mysql_error());
        echo 'Eintrag mit der ID '.$_GET['id'].' erfolgreich gelöscht!';
    }

            if ($_POST) {
            $ID = $_POST['identify'];
            
            
            $add = "DELETE FROM 
                     News
                    WHERE ID = $ID";
            
            $newnews = mysql_query($add) or die(mysql_error());

            
            }
            ?>

wäre echt nett, wenn ihr mir helfen könntet =)

Cheers

[Lenki]

Naja, die Meldung sagt ja schon: im Query ist ein Fehler. Lass dir im die() nach mysql_query() neben mysql_error() mal noch das Query ausgeben. Dazu musst du natürlich das Query vorher in einer Variablen speichern.

[falscherprophet]

hier ists:
UPDATE News SET Inhalt = wwwww WHERE ID = 29 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 3

[Lenki]

wwwww ist doch ein String, den musst du auch so im Query kenntlich machen. Wenn du nicht weißt, wie: Google ist dein Freund

[falscherprophet]

Inhalt = '" .$_POST['content']. "' 

so stehts doch drin also

Inhalt = ' STRING ' eigentlich müsstes so doch klappen ...

[Gast]

btw: Such mal anch SQL-Injections

[falscherprophet]

So besser ?

<?php 
    include('dbconnect.php'); 
    $date = date('d.m.Y'); 
    if($_GET['action'] == 'edit'){
    $ident=mysql_real_escape_string($_GET['id']);

    $sql="SELECT * FROM News WHERE ID = ".$ident;
    $sqlqry = mysql_query($sql) or die(mysql_error());
    
    $data = mysql_fetch_array($sqlqry);
?>
<form name="news" method="post" action="<?php echo $_SERVER['SCRIPT_NAME']; ?>?action=<?php echo $_GET['action']; ?>&id=<?php echo $_GET['id']; ?>">
<textarea name="content" rows="5" cols="35" ><? echo $data['Inhalt'];  ?></textarea> 
<input type="submit" value="Abschicken" accesskey="s">
</form>   
<?php
        if ($_POST) {
    $contd=mysql_real_escape_string($_POST['content']);
    $ident=mysql_real_escape_string($_GET['id']);
    $sql2="UPDATE 
                 News 
                SET 
                 Inhalt = \'.$contd.\' 
                WHERE ID = ".$ident;
           
    $sql2qry=mysql_query($sql2) or die(mysql_error());
    
          echo 'UPDATE 
                 News 
                SET 
                 Inhalt = "'.$contd.'" 
                WHERE ID = ".$ident'; 
                
        }
    }
    if($_GET['action'] == 'delete'){
    $ident=mysql_real_escape_string($_GET['id']);
    $sql3="DELETE FROM
            News
           WHERE ID = ".$ident;
           
    $sql3qry=mysql_query($sql3) or die(mysql_error());
          
        echo 'Eintrag mit der ID '.$ident.' erfolgreich gelöscht!';
    }

            if ($_POST) {
            $ID = mysql_real_escape_string($_POST['identify']);
            
            
            $add = "DELETE FROM 
                     News
                    WHERE ID = $ID";
            
            $newnews = mysql_query($add) or die(mysql_error());

            
            }
            ?>

[Gast]

Auf den ersten Blick - Ja. Allerdings sollte man der Konseqenz halber Zahlen auch als solche benutzen, also:

$ident=(int)$_GET['id'];
//oder
$ident=intval($_GET['id']); 

[falscherprophet]

ok hab das geändert aber nun mal zurück zum Problem:

Warum spackt das bei zB der Abfrage ?

UPDATE
 News 
SET
 Inhalt = 'hhh' 
WHERE ID = '29'; 

bekomm immernoch den Error

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 3

[nikosch]

Sorry, aber Du schaust die falsche Query an. Massgeblich ist das:

$sql2="UPDATE 
                 News 
                SET 
                 Inhalt = \'.$contd.\' 
                WHERE ID = ".$ident; 

, nicht Dein echo. Und nun mal genau hinsehen.

[falscherprophet]

ich vergaß, der aktuelle code lautet wiefolgt ...

<?php 
    include('dbconnect.php'); 
    $date = date('d.m.Y'); 
    if($_GET['action'] == 'edit'){
    $ident=mysql_real_escape_string(intval($_GET['id']));

    $sql="SELECT * FROM News WHERE ID = ".$ident;
    $sqlqry = mysql_query($sql) or die(mysql_error());
    
    $data = mysql_fetch_array($sqlqry);
?>
<form name="news" method="post" action="<?php echo $_SERVER['SCRIPT_NAME']; ?>?action=<?php echo $_GET['action']; ?>&id=<?php echo $_GET['id']; ?>">
<textarea name="content" rows="5" cols="35" ><? echo $data['Inhalt'];  ?></textarea> 
<input type="submit" value="Abschicken" accesskey="s">
</form>   
<?php
        if ($_POST) {
    $contd=mysql_real_escape_string($_POST['content']);
    $ident=mysql_real_escape_string(intval($_GET['id']));
    $sql2="UPDATE
                 News
                SET
                 Inhalt = '" . $contd . "'
                WHERE ID = '". $ident . "';";
           
    $sql2qry=mysql_query($sql2) or die(mysql_error());
    
          echo($sql2); 
                
        }
    }
    if($_GET['action'] == 'delete'){
    $ident=mysql_real_escape_string(intval($_GET['id']));
    $sql3="DELETE FROM
            News
           WHERE ID = ".$ident;
           
    $sql3qry=mysql_query($sql3) or die(mysql_error());
          
        echo 'Eintrag mit der ID '.$ident.' erfolgreich gelöscht!';
    }

            if ($_POST) {
            $ID = mysql_real_escape_string($_POST['identify']);
            
            
            $add = "DELETE FROM 
                     News
                    WHERE ID = $ID";
            
            $newnews = mysql_query($add) or die(mysql_error());

            
            }
            ?>

also ist die maßgebende stelle

if ($_POST) {
    $contd=mysql_real_escape_string($_POST['content']);
    $ident=mysql_real_escape_string(intval($_GET['id']));
    $sql2="UPDATE
                 News
                SET
                 Inhalt = '" . $contd . "'
                WHERE ID = '". $ident . "';";
           
    $sql2qry=mysql_query($sql2) or die(mysql_error());
    
          echo($sql2); 
                
        }
    } 

[comlar]

Mach mal das Semikolon in der SQL-Anweisung weg.

[falscherprophet]

immernoch die Selbe Fehlermeldung ...

[comlar]

Schon mal die SQL-Abfrage z.B. in phpMyAdmin laufen lassen? Funktioniert sie da?

[nikosch]

$sql2="UPDATE
                 News
                SET
                 Inhalt = '" . $contd . "'
                WHERE ID = '". $ident . "';"; 

            $add = "DELETE FROM 
                     News
                    WHERE ID = $ID"; 

    $sql3="DELETE FROM
            News
           WHERE ID = ".$ident; 

Vielleicht auch einfach mal auf eine Methode „einigen“, die ID in die Query zu bekommen.