[sql] Onlineliste ausgeben

Fragen zu allen Themen rund ums Programmieren außerhalb von phpBB können hier gestellt werden - auch zu anderen Programmiersprachen oder Software wie Webservern und Editoren.
Antworten
Benutzeravatar
Ambience
Mitglied
Beiträge: 628
Registriert: 02.09.2006 11:28
Wohnort: daheim
Kontaktdaten:

[sql] Onlineliste ausgeben

Beitrag von Ambience »

Hallo, ich habe folgenden Code:

Code: Alles auswählen

$sql = 'SELECT `username`
      FROM ' . USERS_TABLE . '
        WHERE `active_room` = "' . $db->safe($_SESSION['active_room']) . '"
          AND `last_action` > ' . time()-1800;
Und bekomme einen mysql fehle:
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 '-1800' at line 1
Ich hatte eigentlich vor, alle User auszulesen, welche in einem raum sind und innerhalb der letzen 30 minuten online waren... aber ich weiß nicht was daran falsch sein sollte..

Ambience
Benutzeravatar
S2B
Ehemaliges Teammitglied
Beiträge: 3258
Registriert: 10.08.2004 22:48
Wohnort: Aachen
Kontaktdaten:

Beitrag von S2B »

Hmm, den Fehler kann ich ehrlich gesagt nicht nachvollziehen... :-?

Hast du mehrere Queries, in denen du " " zum Einfügen von Strings verwendest? Wenn nicht, versuche es doch mal mit ' ':

Code: Alles auswählen

"WHERE `active_room` = '" . $db->safe($_SESSION['active_room']) . "'
Gruß, S2B
Keinen Support per ICQ/PM!
Hier kann man meine PHP-Skripte und meine MODs für phpBB runterladen.
Benutzeravatar
Pyramide
Ehrenadmin
Beiträge: 12734
Registriert: 19.04.2001 02:00
Wohnort: Meschede

Beitrag von Pyramide »

Lass dir doch einfach mal $sql per echo ausgeben, in 95% der Fälle sticht einem der Fehler dann direkt ins Auge.
KB:knigge
Benutzeravatar
Ambience
Mitglied
Beiträge: 628
Registriert: 02.09.2006 11:28
Wohnort: daheim
Kontaktdaten:

Beitrag von Ambience »

der fehler war das ich den raum als string und nicht als zahl übergeben habe.... hmm da muss man auch erst draufkommen.... schon wieder ganzer tag auf fehlersuche draufgegangen...
Antworten

Zurück zu „Coding & Technik“