phpBB.de

Verfasst: **04.06.2007 22:01**

Hi,
ich entwickle gerade einen neuen MOD, ein Video-Player fürs Forum.
Unter dem Video soll man auch Kommentare schreiben können.
Leider sendet das Script die Angaben nicht korrekt an die Datenbank!
Was habe ich falsch gemacht?
Ich habe es getestet und bemerkt das kein Eintrag-vorgenommen wurde!

Code:

Code: Alles auswählen

http://www.keepgaming.de/player.php?video=2

zum Player

Vielen Dank

TerminatorX120

Edit: Ich entschuldige mich für mein benehmen bezüglich des "keine Zeit" und für nur die Stichworte, hatte wirklich überhaupt keine Zeit

Neuer Quellcode!

Verfasst: **04.06.2007 22:33**

hi
nicht viel Zeit. Stichworte: RTFM, lesendenkenposten, genaue Fehlerbeschreibung

Jan

ich bin mal nett und lass das hier mal offen... aber in zukunft gilt: wenn keine zeit um richtig etwas zuschreiben, lass es gleich sein!

und wie soll das funktionieren? du hast ne variable mit dem sql query und nun?

schonmal was von $db->sql_query($sql) gehört?
und wieso fügst du id mit '' ein? ich gehe davon aus das das ein auto_increment ist also kannst du das ganz rauslassen im insert befehl

Verfasst: **05.06.2007 10:00**

OK, OK, es tut mir leid,
aber jetzt ist es verändert!

Ich bin mal nett und lass das hier mal offen... aber in zukunft gilt: wenn keine zeit um richtig etwas zuschreiben, lass es gleich sein!

und wie soll das funktionieren? du hast ne variable mit dem sql query und nun?

schonmal was von $db->sql_query($sql) gehört?
und wieso fügst du id mit '' ein? ich gehe davon aus das das ein auto_increment ist also kannst du das ganz rauslassen im insert befehl

Vielen Dank Jan 500, aber ich bin nicht auf den Kopf gefallen.

Und das "$db->sql_query($sql)" ist weiter oben, das ist nur nicht im Code mit drinnen!

Verfasst: **05.06.2007 10:32**

terminatorx120 hat geschrieben:Und das "$db->sql_query($sql)" ist weiter oben, das ist nur nicht im Code mit drinnen!

Nunja ... wenn dein query im code vor der Definiton des Querystrings liegt, kann dieser ja auch kaum ausgeführt werden

Gruss,
Manne.

Verfasst: **05.06.2007 11:25**

wo und wie muss ich es den einbauen??

Verfasst: **05.06.2007 11:35**

Nunja:

zuerst musst du den Query-String zusammenbauen(dabei die Hinweise von Jan beachten)
dann erst kanst du das Query an den Datenbankserver schicken

Gruss,
Manne.

Verfasst: **05.06.2007 12:29**

ja, aber wie genau?

TerminatorX120

*Erster MOD! sprich Neuling*

Verfasst: **05.06.2007 13:58**

das steht doch alles oben :-S

aber wenn du schon an solchen einfachen sachen scheiterst, ist die frage ob du das mit dem mod erstmal lassen solltest und vorher etwas "üben"...
dann passiert es auch nicht das du evtl sicherheitslücken einbaust (was dein code oben beinhaltet !)

Jan

Verfasst: **05.06.2007 14:10**

Nur wie soll ich's "üben", wenn ich mir keiner sagt, wie's geht, und nein, ich kapier's nicht, was du oben geschrieben hast.

Neuer Code:

Code: Alles auswählen

elseif ($_POST && $userdata['session_logged_in']) {
  $sql= "INSERT INTO phpbb2_video_comments (
video_id ,
comment_time ,
comment_from ,
comment_text
)
VALUES ( $video_id, $time, ".$userdata['user_name'].", ".$_GET[comment_text].")";
if (!($result = $db->sql_query($sql))){
 message_die(CRITICAL_ERROR, "Konnte Querrie auf Allgemeine Video-Kommentar-tabelle nicht ausf&uuml;hren!<br><br>Could not querrie video-comment information!", "", __LINE__, __FILE__, $sql);
 }
 $db->sql_query($sql);
 
  $comment_area = '<span style="color: green;"><br />Dein Kommentar wurde aufgenommen!<br />';
  }

Ich wäre sehr froh, wenn mir jemand direkt sagt, was ich ändern muss.

Denn außer dem Insert Into befehl, funktioniert das meiste schon sehr gut!!

Selber anschauen!

Verfasst: **05.06.2007 14:29**

kannst du mal das gesamte script geben?

wird die abfrage überhaupt true?
also hast du überhaupt $_POST (bei phpbb2 sollte man noch mit $HTTP_POST_VARS arbeiten)
im insert hast du jetzt plötzlich get (was für längere texte überhaupt nicht geeignet sind, und auch sollte man das nicht ungeprüft einfach inm einen swql query packen)

du führst gerad auch zweimal $db->sql_query($sql); aus...
es muss auch $userdata['username'] heißen
es kommt keine fehlermeldung?

Jan

phpBB.de

[ABGESCHLOSSEN!]Kein einschreiben in Datenbank

[ABGESCHLOSSEN!]Kein einschreiben in Datenbank