[phpBB Debug] PHP Warning: in file [ROOT]/ext/tas2580/seourls/event/listener.php on line 213: Undefined array key "FORUM_NAME"
Multinewsletter Addon Feedback / Bugfix - Seite 11 - REDAXO Forum
Hallo,

Wir haben in letzter Zeit festgestellt, dass die Kommunikation via Slack viel schneller und zielführender ist als ein Beitrag im Forum. Aufgrund der neuen und besseren Möglichkeiten der Kommunikation haben wir uns entschlossen das Forum nur noch als Archiv zur Verfügung zu stellen. Somit bleibt es weiterhin möglich hier nach Lösungen zu suchen. Neue Beiträge können nicht mehr erstellt werden.

Wir empfehlen, für deine Fragen/Probleme Slack zu nutzen. Dort sind viele kompetente Benutzer aktiv und beantworten jegliche Fragen, gerne auch von REDAXO-Anfängern! Slack wird von uns sehr intensiv und meistens "rund um die Uhr" benutzt :-)
Selbst einladen kannst Du dich hier: https://redaxo.org/slack/
frood
Beiträge: 226
Registriert: 7. Okt 2008, 15:17

Re: Multinewsletter Addon Feedback / Bugfix

22. Jul 2013, 16:46

Hi Tobi,
Ich löse das so: im Redaxo Artikel setze ich einen Link auf das Abmelden Formular.
Naja, ist nicht wirklich eine Lösung, oder? :wink:
Der Grund liegt darin, dass es nicht möglich sein soll herauszufinden welche Adressen in der Datenbank vorhanden sind. Bis jetzt fanden das - nach der Erklärung - fast alle sinnvoll.
Verstehe ich nicht ganz: wenn man willkührlich mail-Adressen ausprobiert (warum auch immer) ist die Meldung doch egal ... bei einem Treffer ist die mail auf jeden Fall aus dem Verteiler gelöscht. Hattest Du hier mal einen konkreten Fall?
Ich denke hier eher an die Leute, die sich schonmal ausgetragen haben und dann bei einem zweiten (Kontroll-)austrag nochmal dieselbe Meldung bekommen. Die denken dann doch:"Der Hansel, der läßt mich nicht aus seinem Newsletter und müllt mich bis in alle Ewigkeit zu!"
Du meinst so ein richtig hübsches XForm mit Captcha.
Zum Beispiel ... aber fürs erste langt hier sicher auch eine Session-Lösung:

In der newsletter.inc.php ab Zeile 315:

Code: Alles auswählen

......

// if the user has to confirm the subscription, send an email
if(intval($REX['ADDON375']['config']['confirmmail'])==1)
{
     //  ##### NEU: nur, wenn noch nicht gesendet ##### 
     if($_SESSION['newsletter_gesendet']!='1')
     {
          // set correct title from config file

          ........								

          if(!$mail->send())
          {
               $return['error'][] = 'could_not_send';
          }
          else
          {
              //  ##### NEU: Session-Wert setzen, um reload zu verhindern ##### 
               $_SESSION['newsletter_gesendet']='1';
               $return['msg'][] = 'confirmation_sent';
          }
     }
     // ##### NEU: Meldung, wenn bereits versendet ##### 
     else
     {         
          $return['error'][] = 'already_confirmed';
          $return['msg'][] = 'confirmation_sent';	
     }
}

.........

Ausserdem gebe ich die Session wieder frei, wenn das Formular angezeigt wird, damit sich trotzdem mehrere Leute über einen Rechner für den Newsletter anmelden können:

In der redaxo_modules.inc.php ab Zeile 307:

Code: Alles auswählen

..........

// print out the form
if($showform)
{
    //  ##### NEU: Session wieder zum Senden freigeben ##### 
    $_SESSION['newsletter_gesendet']='';
  
    .......... 

Wie geagt, halte ich für sehr wichtig gegen Reload-Spam, z.B.: "Kaltenbach ist doof, denen schicke ich jetzt erstmal 100 Newsletteranmeldungen mit deren eigener mail-Adresse!"
Zur ToDo Liste: Die Arbeite ich immer dann ab, wenn ich mal Zeit habe.
Kann ich gut nachvollziehen. Aber das Ding läuft ja insgesamt sehr schön, und die paar Kleinigkeiten sollte jeder selbst hinbekommen.

Was mir noch aufgefallen ist: Beim Versand:
1. Newsartikel auswählen
2. Testmail schicken => gewählter Newsletterartikel wird als Link angezeigt, ist aber immer auf article_id=5 verlinkt!

Werde mich mal auf die Suche machen, woran's liegt ....
EDIT: Da hammer's ja schon: in der /addons/multinewsletter/pages/newsletter.inc.php, Zeile 626 ist die URL hardcoded, müsste aber heissen:

Code: Alles auswählen

... <a href="../index.php?article_id=<?php print $REX['ADDON375']['postget']['newsletter']['article_id']?>&clang=0" target="_blank"> ...
Gruss, Frood

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

22. Jul 2013, 18:38

Hi Frood,

danke für den Bugfix! Den nehme ich gleich in die nächste Version mit rein. Hier mal ein Testpaket:
https://www.dropbox.com/s/6wpkve4lbp3i2 ... letter.zip

Funktioniert das bei dir? Oder fehlt noch was?

Liebe Grüße,

Tobi

frood
Beiträge: 226
Registriert: 7. Okt 2008, 15:17

Re: Multinewsletter Addon Feedback / Bugfix

23. Jul 2013, 21:33

Hallo Tobi,

ich hatte bisher leider nicht die Zeit, das mit meiner Installation zu vergleichen und zu testen ....
Was hast du denn nun geändert? (Wie jdlx schon sagte: eine Versionierung und git wäre echt nicht schlecht!)

Ich kann dir bisher nur sagen, dass der letzte release plus die von mir beschriebenen Änderungen bei mir auf rex4.5 sehr gut funktionieren ....

Gruss, frood

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

23. Jul 2013, 22:33

Hi Frood,

ich habe nur die Änderungen von dir rein genommen. Aber nicht auf Fehler o.ä. getestet.

Liebe Grüße,

Tobi

frood
Beiträge: 226
Registriert: 7. Okt 2008, 15:17

Re: Multinewsletter Addon Feedback / Bugfix

24. Jul 2013, 16:03

So, nu hab ich mal geschaut:

functions/newsletter.inc.php, Zeile 352:
vor dem Kommentar die neue if-Klammer auch noch schliessen, dann passt es:

Code: Alles auswählen

   }
   // ##### NEU: Meldung, wenn bereits versendet ##### 
   .......
In der redaxo_modules.inc.php hat die td für email-label eine feste Breite ... nervt etwas beim Formatieren ;)

Ansonsten passt es (die An- und Abmeldemodule hast Du ja anscheinend noch nicht angepasst).

Gruss, Frood

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

24. Jul 2013, 16:16

Hi Frood,

danke für dein Testen! Ich habe die neue Version 1.4.2 online gestellt. Da habe ich auch die Breitenangabe komplett entfernt. Macht ja auch keinen Sinn, wenn alles andere in % angegeben wird.

Wenn noch was ist kannst du dich gerne melden!

Liebe Grüße,

Tobi

frood
Beiträge: 226
Registriert: 7. Okt 2008, 15:17

Re: Multinewsletter Addon Feedback / Bugfix

1. Aug 2013, 16:37

Hallo Tobi, hab noch was gefunden:

Im Archiv werden ja die de und en-Newsletter getrennt aufgelistet ... bei Klick auf das "E" für die englischen Empfänger werden die Empfänger nicht angezeigt (Empfängerzahl immmer 0 !)

Gruss, Frood

EDIT: Lag nicht an der Sprachversion, sondern daran, dass es nur ein Empfänger gab ... Lösung:

In der archiveout.inc.php ab Zeile 16:

Code: Alles auswählen

if(strpos($newsletter['recipients'],','))
  $newsletter['recipients'] = explode(',',$newsletter['recipients']);
else
  $newsletter['recipients'] = array();
}
ändern in:

Code: Alles auswählen

if(strpos($newsletter['recipients'],',')){
  $newsletter['recipients'] = explode(',',$newsletter['recipients']);
}else{
  $only_mail = $newsletter['recipients'];
  $newsletter['recipients'] = array();
  $newsletter['recipients'][] = $only_mail;
}

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

1. Aug 2013, 20:50

Hi Frood,

danke! Kommt in die nächste Version 1.4.3 rein.

Vielen Dank!

Liebe Grüße,

Tobi

frood
Beiträge: 226
Registriert: 7. Okt 2008, 15:17

Re: Multinewsletter Addon Feedback / Bugfix

2. Aug 2013, 10:10

Hi Tobi,
ich hab noch einen kleinen Verbesserungsvorschlag für die nächste Version:

Nach dem Versenden der Testmail kann man den Newsletter nur anzeigen und nicht editieren.
Zum Editieren muss wieder durch die ganze Struktur toben ....
Ich habe hier einen entsprechenden Shortcut-Link ergänzt.

newsletter.inc.php, Zeile 626:

Code: Alles auswählen

<a href="../index.php?article_id=<?php print $REX['ADDON375']['postget']['newsletter']['article_id']?>&clang=0" target="_blank"><?php print $REX['ADDON375']['postget']['newsletter']['article_name']?></a>
ändern in

Code: Alles auswählen

<strong><?php print $REX['ADDON375']['postget']['newsletter']['article_name']?></strong>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<a href="../index.php?article_id=<?php print $REX['ADDON375']['postget']['newsletter']['article_id']?>&clang=0" target="_blank">ANZEIGEN</a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<a href="./index.php?page=content&article_id=<?php print $REX['ADDON375']['postget']['newsletter']['article_id']?>&mode=edit&clang=0" target="_blank">EDITIEREN</a>
(Sprache ist noch hardcoded)

Gruss, Frood

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

5. Aug 2013, 09:16

Hi Frood,

das ist eine sehr gute Idee. Kommt in die nächste Version rein. Danke!

Liebe Grüße,

Tobi

Benutzeravatar
Oliver.Kreischer
Beiträge: 2508
Registriert: 17. Dez 2004, 00:03
Wohnort: Velbert - LA
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

3. Sep 2013, 22:16

Halllo Zusammen,

ich habe gerade eine Webseite mit dem Addon von einem richtigen Hoster auf einen 1und1 Server umziehen dürfen. Zum Glück funktioniert fast alles :-).

Leider wird nach dem versenden an alle Newsletterempfänger die Seite "?page=multinewsletter&subpage=newsletter" nicht angezeigt (neu geladen wird die Seite anscheinend).

Die Newsletter werden richtig verschickt (an 10 Testuser).
Ich weiß, dass das Problem hier schonmal besprochen wurde. Habe aber keine Lösung finden können.

Auf dem Entwicklungsserver funktioniert es einwandfrei.

Habt Ihr einen Tipp?

Testserver: REX: 4.5b4.0, PHP: 5.4.9, MySQL: 5.5.28
1und1: REX: 4.5.0, PHP: 5.4.19, MySQL: 5.1.71

Keine weiteren Sprachen im Backend aktiv.


LG
Oliver
> Friends Of REDAXO Gemeinsame REDAXO-Entwicklung!
> REDAXO Agenturen. Eintragen erwünscht!
> Komm in den Slack Channel Es lohnt sich!

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

4. Sep 2013, 16:34

Hi Oliver,
Oliver.Kreischer hat geschrieben:Leider wird nach dem versenden an alle Newsletterempfänger die Seite "?page=multinewsletter&subpage=newsletter" nicht angezeigt (neu geladen wird die Seite anscheinend).
kommt irgend eine Fehlermeldung? Gibt dir ein Browser einen Timeout? Falls nein tönt es nach einen Fehler im PHP Code. Was passiert, wenn du für das Backend dein PHP Error Reporting auf -1 stellst?

Liebe Grüße,

Tobi

Benutzeravatar
Oliver.Kreischer
Beiträge: 2508
Registriert: 17. Dez 2004, 00:03
Wohnort: Velbert - LA
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

4. Sep 2013, 23:17

Hi Tobias,

danke für deine Antwort. Auf der Subpage "newsletter" kommen leider oder zum Gllück keine Fehlermeldungen. Auch nicht nach dem Absenden. Es wird einfach die Seite nicht mehr geladen...

Auf den anderen Subpages hagelt es "Notice" FEhlermeldungen (meist "Undefined index").

--

Wenn ich mir den Quelltext von der leeren Seite anzeigen lassen möchte liefert mir der Browser die Info "Dokument erloschen. Dieses Dokument ist nicht mehr verfügbar...."

Hast Du vielleicht noch eine Idee?

LG
Oliver
> Friends Of REDAXO Gemeinsame REDAXO-Entwicklung!
> REDAXO Agenturen. Eintragen erwünscht!
> Komm in den Slack Channel Es lohnt sich!

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

5. Sep 2013, 08:42

Hi Oliver,
Oliver.Kreischer hat geschrieben:Wenn ich mir den Quelltext von der leeren Seite anzeigen lassen möchte liefert mir der Browser die Info "Dokument erloschen. Dieses Dokument ist nicht mehr verfügbar...."
"Dokument erloschen" hat oft was mit der Session zu tun. Aber im MultiNewsletter gibt es keine Session. Gibt es sonst irgend eine Ausgabe von Redaxo selbst? Was passiert, wenn du auf der leeren Seite F5 drückst?

Sehr seltsam. Welche MultiNewsletter Version hast du? Dann versuche ich das mal zu testen.

Liebe Grüße,

Tobi

Benutzeravatar
Oliver.Kreischer
Beiträge: 2508
Registriert: 17. Dez 2004, 00:03
Wohnort: Velbert - LA
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

5. Sep 2013, 21:46

Hi Tobi,
Tobias Krais hat geschrieben: "Dokument erloschen" hat oft was mit der Session zu tun. Aber im MultiNewsletter gibt es keine Session. Gibt es sonst irgend eine Ausgabe von Redaxo selbst? Was passiert, wenn du auf der leeren Seite F5 drückst?
Das gleiche Ergebnis.
Tobias Krais hat geschrieben: Sehr seltsam. Welche MultiNewsletter Version hast du? Dann versuche ich das mal zu testen.
Wo schreibst Du die Version hin?? Laut help.inc.php müsste es die 1.4.1 sein.

Auf einem anderen Webspace mit REX 4.5b4.0 läuft es einwandfrei.
Die Probleme habe ich erst mit 1und1 + REX 4.5


LG
Oliver
> Friends Of REDAXO Gemeinsame REDAXO-Entwicklung!
> REDAXO Agenturen. Eintragen erwünscht!
> Komm in den Slack Channel Es lohnt sich!

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

5. Sep 2013, 22:40

Hi Oliver,
Oliver.Kreischer hat geschrieben:Wo schreibst Du die Version hin?? Laut help.inc.php müsste es die 1.4.1 sein.
Die Version steht in hellem Grau direkt neben der großen Überschrift des Addons. 1.4.1 ist die neueste.
Oliver.Kreischer hat geschrieben:Auf einem anderen Webspace mit REX 4.5b4.0 läuft es einwandfrei.
Die Probleme habe ich erst mit 1und1 + REX 4.5
Sehr seltsam. Ich habe ein System von 1&1 mit genau den selben Daten wie du (Redaxo 4.5, gleiche PHP und MySQL. Der einzige Unterschied: die Seite hat 10 Sprachen. Und bei mir funktioniert es einwandfrei. Was passiert, wenn du eine weitere Sprache hinzufügst?

Liebe Grüße,

Tobias

Benutzeravatar
Peter_01
Beiträge: 8
Registriert: 19. Jan 2008, 17:49
Wohnort: Berlin

Re: Multinewsletter Addon Feedback / Bugfix

4. Okt 2013, 15:14

Ich denke ich habe einen Bug gefunden (v. 1.4.1), es sei denn ich habe irgendwas nicht verstanden ... betrifft das manuelle Bearbeiten von usern im Backend:
In pages/user.inc.php wird in der Selectbox für aktiv/inaktiv 1 und 2 als value verwendet, überall anders scheint es 1 und 0 zu sein. Speichert man einen Nutzer, wird bei inaktiven wird daher immer der Status auf 2 gesetzt, was später nicht wirklich ausgewertet werden kann und in der Liste erscheinen diese dann "orange" aber aktiv ...

also richtig scheint mir zu sein (außer bei der 'showstatus' selectbox !!)

Code: Alles auswählen

$status->addOption($REX['ADDON375']['I18N']->msg('status_offline'),'0');
Viele Grüße,
Peter

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

4. Okt 2013, 16:43

Hallo Peter,

das kann sein. Ich bin leider mit dem Code im Moment nicht mehr firm und schaffe es im Moment nicht wegen Überlastung. Hast du die Änderung getestet? Wenn es funktioniert, dann gib mir bitte Bescheid.

Ich weiß nur noch, dass es tatsächlich drei verschiedene Stati gibt: Aktiv, Inaktiv und Nicht bestätigt.

Liebe Grüße,

Tobi

Benutzeravatar
RexDude
Beiträge: 2543
Registriert: 22. Apr 2010, 11:24

Re: Multinewsletter Addon Feedback / Bugfix

4. Nov 2013, 14:22

Hallo zusammen,

ist das Multinewsletter Addon eigentlich der Nachfolger vom GroupLetter? Ich hab hier nämlich ein GroupLetter im Einsatz was ich auf R4.5 hochgezogen habe. Nun funzt es aber dummerweise nich mehr (kommt leider auch keine Fehlermeldung aussser halt paar Notice etc.) ;) Weiss jemand vielleicht wie ich hier sauber updaten kann?

Danke und LG,
RexDude

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

4. Nov 2013, 20:55

Hi RexDude,

ja, MultiNewsletter ist der Nachfolger vom Groupletter. So stand es mal in der ursprünglichen Beschreibung.

Gute Frage, wie man das am besten hochziehen kann. Ich denke am besten ist die Tabellenstruktur von Hand anpassen und dann sollte es im MultiNewsletter nutzbar sein. Ein Skript gibt es leider nicht.

Hilft dir das?

Liebe Grüße,

Tobi

P.S.: Dein SEO42 wird jetzt wohl doch mein SEO Addon der Wahl. Ich meld mich bei dir wahrscheinlich bald man im Bugtracker ;-). Muss ich aber noch testen.

Benutzeravatar
RexDude
Beiträge: 2543
Registriert: 22. Apr 2010, 11:24

Re: Multinewsletter Addon Feedback / Bugfix

4. Nov 2013, 22:01

Hi Tobi,

ja, das hilft mir erstmal weiter. Vielen Dank :)

Cool, freut mich :D Dann bin ich ja mal gespannt... ;)

LG,
RexDude

Benutzeravatar
nuxx
Beiträge: 164
Registriert: 16. Dez 2005, 18:45
Wohnort: Kaputtgart 21

Re: Multinewsletter Addon Feedback / Bugfix

16. Feb 2014, 22:06

Hallo Tobias,
erst einmal Danke für dieses großartige Addon.
Ich würde gerne die Standart Texte neu definieren "Möchten Sie regelmäßig informiert werden? Dann abonnieren Sie unseren interessanten Newsletter..." usw.
Leider habe ich die Stellen nicht gefunden, wo ich eingreifen muß.

in den lang files wurde ich nicht fündig.

Kann ich auch irgendwie in einem Link die User ID des Empfängers anhängen ?
zb. http://www.domain.com/userbereich/super ... userid=123

Schonmal herzlichen Dank für Deine Hilfe!

l.g. nuxx
Halbwissen ist schonmal besser als Vollpfosten!

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

16. Feb 2014, 22:38

Hi nuxx,
nuxx hat geschrieben:Ich würde gerne die Standart Texte neu definieren "Möchten Sie regelmäßig informiert werden? Dann abonnieren Sie unseren interessanten Newsletter..." usw.
Leider habe ich die Stellen nicht gefunden, wo ich eingreifen muß.
Schau mal unter files/clang0.lang nach.
nuxx hat geschrieben:Kann ich auch irgendwie in einem Link die User ID des Empfängers anhängen ?
zb. http://www.domain.com/userbereich/super ... userid=123
Bis jetzt nicht. Das müsste man noch programmieren.

Liebe Grüße und viel Erfolg!

Tobias

Benutzeravatar
nuxx
Beiträge: 164
Registriert: 16. Dez 2005, 18:45
Wohnort: Kaputtgart 21

Re: Multinewsletter Addon Feedback / Bugfix

17. Feb 2014, 18:50

Hallo Tobias,

herzlichen Dank für die flotte Hilfe!

ich habe nun das Abmelden via Link ausprobiert.
http://www.domain.tld/newsletter.html?u ... domain.tld aufgerufen

es kommt die Meldung "Sie erhalten ab sofort keinen Newsletter mehr von uns, können sich aber jederzeit wieder anmelden."

im Adminbereich ist der Benutzer aber noch vorhanden und wird als "aktiv" angezeigt.
Soll das so sein ?

Multinewsl Version 1.4.2
Rex 4.5.0
PHP: 5.4.24
MySQL: 5.1.72

mfg nuxx
Halbwissen ist schonmal besser als Vollpfosten!

Tobias Krais
Beiträge: 500
Registriert: 2. Okt 2006, 23:16
Wohnort: Lörrach
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

17. Feb 2014, 21:09

Hi nuxx,

bei mir kommt die Meldung nicht!?

Da sie aber bei dir kommt, muss es sich um einen Bug handeln. Kannst du ihn rausfinden und fixen?

Liebe Grüße,

Tobi

Zurück zu „Allgemeines [R4]“