mysql_query wird nicht ausgeführt, keine Fehlermeldung

[Spunk95]

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:

[COLOR="Red"]Statt And muss bei einer Update Anfrage ein Komma stehen. [/COLOR]

[wahsaga]

Zum x-tausendsten Mal: mysql_error nutzen!

[onemorenerd]

mysql_error

[Spunk95]

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

[h3ll]

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

[AmicaNoctis]

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!

[Spunk95]

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

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).

[wahsaga]

Nicht immer gleich beleidigend werden.

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

        $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.

[Spunk95]

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

[onemorenerd]

Setze mysql_error vors erste if!

[Spunk95]

Auch dann leider keine Meldung..

[wahsaga]

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 ...

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

[Spunk95]

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.

[litterauspirna]

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.

[wahsaga]

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.ä.