mm_forum 0.1.6 Emailbenachrichtigung klappt nicht

thomas_vienna · 2008-05-05T14:35:56+00:00

Liebe Typo3 Community, ich kämpfe schon seit einigen Studen mit mm_forum auf der Seite www.teatro.at und habe folgendes Problem. Die User werden nicht benachrichtigt, wenn ein neuer Beitrag zu ihrem Post gemacht wird. Nach der ersten Installation (damals noch Version 0.1.5) wurde zwar ein Email versendet aber erstens von einer seltsamen Emailadresse (teatro@www62.world4you.com) und zweites wurde der Link nicht richtig dargestellt: http%3A%2F%2Fwww.teatro.at%2Fcms%2Fmitmachen%2Fforum%2Fdas-teatro-forum%2Fopen_topic.html%3Ftx_mmforum_pi1%255Bid%255D%3D5 Nach einem Update auf mm_forum 0.1.6. wurden dann überhaupt keine Emails versendet. Interessanter Weise werden sonst alle Emails richtig versendet. Hat jemand einen Hinweis? Danke im voraus und liebe Grüße, Thomas

thomas_vienna · 6. Mai 2008 13:54

Noch eine Frage:
weißt du ob user_id richtig übergeben wird.
Davon hängt nämlich ab, b in der DB etwas gefunden wird.

Benjamecho · 6. Mai 2008 14:12

Ich habe das kleine PHP-Snippet für die Textdatei direkt vor die if-Abfrage eingebaut und das Textdokument ist dann in meinem /html/ Ordner gelandet. Wofür die Funktion genau gut ist habe ich auch noch nicht verstanden. Jedenfalls ist sie in dieser Version neu hinzugekommen.

Benjamecho · 6. Mai 2008 14:13

Die user_id scheint richtig übergeben worden zu sein.

m.helmich · 6. Mai 2008 15:52

thomas_vienna schrieb

Sollte diese Zeile nicht eher so aussehen:

list($to_username, $to_usermail) = $GLOBALS['TYPO3_DB']->sql_fetch_row($GLOBALS['TYPO3_DB']->exec_SELECTquery($this->getUserNameField().'username','email','fe_users','uid="'.$row['user_id'].'"'));

Nein. Wie in der Quelltextdokumentation zu entnehmen, liefert getUserNameField() den Namen des Datenbankfeldes, in welchem der Benutzername steht, also in aller Regel "username". Daher muss auch das Komma vor "email" stehen (sonst würde ja schließlich das Feld "usernameemail" abgefragt) und daher ist auch das hinzufügen des "username"-Feldes überflüssig (noch dazu ohne Komma und als eigener String).
Im Bugtracker wurde das Problem bereits korrekt erkannt; ein Patch steht im SVN zur Verfügung.

Martin

thomas_vienna · 6. Mai 2008 16:06

aha danke!
Würde mir gerne die bereits gepachte Entwicklerversion runterladen.
http://svn.typo3.net/

Kriege aber immer eine Fehlermeldung...

Benjamecho · 6. Mai 2008 16:15

@Thomas: Ich habe nur die kleine Änderung vorgenommen, daher brauchgte ich nicht die ganze Datei.

@Martin: Besten Dank, jetzt funktioniert es! Eine Frage: warum hast Du die Änderung nicht entsprechend dem Vorschlag im Bugtracker gemacht, sondern die !empty($to_usermail)-Abfrage rausgeworfen? Wenn beim user keine Adresse hinterlegt ist, braucht das Forum doch erst keinen Mailversuch zu starten, oder wird das an anderer Stelle breits überprüft?

thomas_vienna · 6. Mai 2008 16:38

Hi!
Es klappt danke für den Hinweis...
LG Thomas

m.helmich · 6. Mai 2008 16:42

Benjamecho schrieb
@Martin: Besten Dank, jetzt funktioniert es! Eine Frage: warum hast Du die Änderung nicht entsprechend dem Vorschlag im Bugtracker gemacht, sondern die !empty($to_usermail)-Abfrage rausgeworfen? Wenn beim user keine Adresse hinterlegt ist, braucht das Forum doch erst keinen Mailversuch zu starten, oder wird das an anderer Stelle breits überprüft?

Hallo,

die Abfrage ist doch noch drin...?

Martin

thomas_vienna · 6. Mai 2008 16:48

Ja bei mir schon.
Ich habe nur die variable ausgetauscht...

Danke noch mal für die rasche Hilfe.
Thomas

Benjamecho · 6. Mai 2008 17:05

m.helmich schrieb
die Abfrage ist doch noch drin...?

Jetzt sehe ich es auch, sorry, habe offensichtlich in der falschen Version geschaut.

Muss die Änderung auch bei der darauf folgenden Funktion send_newpost_mail_forum durchgeführt werden? Da fehlt jedenfalls in der if-Abfrage die $to_usermail-Variable.

Ben