[ABGESCHLOSSEN!]Kein einschreiben in Datenbank

[terminatorx120]

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!

[Jan500]

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

[terminatorx120]

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!

[mad-manne]

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.

[terminatorx120]

wo und wie muss ich es den einbauen??

[mad-manne]

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.

[terminatorx120]

ja, aber wie genau?






TerminatorX120

*Erster MOD! sprich Neuling*

[Jan500]

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

[terminatorx120]

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!

[Jan500]

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