Was ist falsch?

[Mort]

füge hinter

if (Error = check_message(forentext))
{
    alert(Error);
    return false
}

mal einem else-Zweig mit "return true;" ein.
wenn die funktion in einem formular verwendet werden soll muss ein bool- wert zurückgegeben werden sonst meint das formular es gibt false zurück und schickt die daten nicht ab.

Und ist die Zuweisung in der if-klausel überhaupt erwünscht? wenn es wirklich eine zuweisung ist, mach sie lieber vorher. und prüf dann erst "Error"

und wenn du dir das alert sparen kannst schreib doch einfach

return !check_message(forentext);