mysql_query wird nicht ausgeführt, keine Fehlermeldung

PHP-Code:

function sendanswer ($pmid,$uid,$text){
    $timestamp = time();
    $text = mysql_real_escape_string($text);
    $sql = mysql_query("
               INSERT INTO msg_answers (pmid, uid, text, date)
               Values ('".$pmid."', '".$uid."', '".$text."', '".$timestamp."')");
    if($sql){
        $update = mysql_query("
                               UPDATE msg SET opened = '".$uid."' 
                               AND last_answer = '".$timestamp."' 
                               WHERE id = '".$pmid."'");
        if($update){
            return true;
        }else{
            return false;
        }
    }else{
        return false;
    }                                        
} 


Ich habe diese Funktion geschieben um Antworten auf bereits bestehende Nachrichten einzutragen. Das eintragen der Antwort klappt auch, aber die Hauptnachricht wird nicht aktualisiert (also das zweiter mysql_query). Es kommt aber auch keine Fehlermeldung selbst mit report all errors.

Hab ich einen Fehler im Code oder weiß jemand woran es liegen könnte?

// Lösung:

Statt And muss bei einer Update Anfrage ein Komma stehen.

Zum x-tausendsten Mal: mysql_error nutzen!

mysql_error

Hab ich doch... Wie oben gesagt, es kommt keine Fehlermeldung

In dem Code ist nirgendwo ein Aufruf von mysql_error().

Wenn du es nicht verstehst, schlag es nach und hör auf mit "hab ich doch". Hast du eben nicht, du gibst lediglich false zurück, statt mysql_error() abzufragen!

Nicht immer gleich beleidigend werden. Ich hab eine Version ohne mysql_error gepostet, aber ich habs natürlich auch mit ausprobiert.

PHP-Code:

function sendanswer ($pmid,$uid,$text){
    $timestamp = time();
    $text = mysql_real_escape_string($text);
    $sql = mysql_query("INSERT INTO msg_answers (pmid, uid, text, date) 
                                     Values ('".$pmid."', '".$uid."', '".$text."', '".$timestamp."')");
    if($sql){
        $update = mysql_query("UPDATE msg SET opened = '".$uid."' 
                                                  AND last_answer = '".$timestamp."' 
                                                  WHERE id = '".$pmid."'");
        if($update){
            ?><div class="warning"><?echo mysql_error();?></div><?
            return false;
        }else{
            return false;
        }
    }else{
        return false;
    }                                        
}

Dann gebe ich übrigens false zurück, damit man die Fehlermeldung auch sieht (bei true wird man weitergeleitet).

Zitat von Spunk95

Nicht immer gleich beleidigend werden.

Gut - du dann aber bitte nicht immer mit ausgeschalteten Gehirn programmieren.

PHP-Code:

        $update = mysql_query("UPDATE msg SET opened = '".$uid."' AND last_answer = '".$timestamp."' WHERE id = '".$pmid."'");
        if($update){
            ?><div class="warning"><?echo mysql_error();?></div>

Wenn die Query erfolgreich verarbeitet werden konnte, gibt mysql_error natürlich nichts aus - also ist es witzlos, dass du es nur in genau dem Fall aufrufst.

Das ist ja das seltsame: Obwohl es nicht gemacht wird, scheint es zu kappen. Die Klasse "warning" wird ja angezeigt, nur halt ohne Text.

Setze mysql_error vors erste if!

Auch dann leider keine Meldung..

Dann bau ganz oben in die Funktion eine nicht bedingungsabhängige Kontrollausgabe ein - um erst mal zu schauen, ob die Funktion auch ausgeführt wird, oder ob du an der ganz falschen Stelle suchst ...

PHP-Code:

function sendanswer ($pmid,$uid,$text){
    die('Wir befinden uns in sendanswer()!');
    $timestamp = time();
    // ... 


Die Funktion wird ja auf jeden Fall ausgeführt, weil die Klasse warning erscheint und die Antwort wird ja auch eingetragen...
Aber danke für den Vorschlag

// habs ausprobiert, "Wir befinden uns in sendanswer()" wird angezeigt.

Vieleicht sendest du auch mal deinen Code wo du das hingeschrieben hast.

Schreibe hinter mysql_query("dein query string") or die mysql_error();

Und das bei beiden Querys, oder wirf mal nen Blick ins Manual wie und wo du das hin platzieren musst.

Schraube auch mal das PHP error_reporting() voll auf und lass dir mal die Variablen ausgeben die du ins Update schreibst.

Vielleicht ist dein UPDATE-Statement auch gar nicht syntaktisch fehlerhaft, sondern nur inhaltlich - und es gibt gar nichts upzudaten, weil es keinen der Selektion per WHERE-Klausel entsprechenden Datensatz gibt, oder der bestehende bereits die „neuen“ Werte hat.

Lass dir die generierte Query zur Kontrolle ausgeben, und teste sie per Copy&Paste in phpMyAdmin o.ä.