[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 15 - 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/
Benutzeravatar
eaCe
Beiträge: 21
Registriert: 11. Jul 2015, 20:56

Re: Multinewsletter Addon Feedback / Bugfix

3. Mai 2016, 16:30

Hi,

wollte mich an dieser Stelle nur mal bedanken das du das Addon auf R5 portiert hast :)

Super Sache!


Viele Grüße

eaCe

wolfgang
Beiträge: 126
Registriert: 20. Okt 2005, 20:52
Wohnort: 23966 Tressow
Kontaktdaten: Website

Re: Multinewsletter Addon Feedback / Bugfix

11. Jul 2016, 21:57

Hallo Tobias,

die Versionierung von 2.2.3 auf 2.2.4 über den Installer funktioniert nicht. Wenn ich das Update über den Installer mache, erscheint immer noch 2.2.3.

Der Bestätigungslink über die Voreinstellungen funktioniert nicht zuverlässig.

Dort müsste statt

Code: Alles auswählen

<p>bitte bestätigen Sie aus Sicherheitsgründen Ihre Anmeldung zum Newsletter. Klicken Sie dazu einfach auf den folgenden Link:  <a href=\"+++AKTIVIERUNGSLINK+++\">+++AKTIVIERUNGSLINK+++</a></p>

Code: Alles auswählen

<p>bitte bestätigen Sie aus Sicherheitsgründen Ihre Anmeldung zum Newsletter. Klicken Sie dazu einfach auf den folgenden Link:  <a href="+++AKTIVIERUNGSLINK+++">+++AKTIVIERUNGSLINK+++</a></p>
Also ohne Backslash vor den Anführungszeichen stehen.

Wenn im Bestand eine E-Mail Adresse mit falscher Syntax steht, also beispielsweise example@-examplede, dann hängt sich der Versand auf.

Abhilfe schafft folgender Code in class.multinewsletter_newsletter.inc.php (Zeile 476 ff):

Code: Alles auswählen

if($newsletter->sendNewsletter($recipient) == false) {
     echo rex_warning('Fehler beim Versand von User-Id '.$recipient->user_id.' E-Mail: '.$recipient->email);
}
anstatt

Code: Alles auswählen

if($newsletter->sendNewsletter($recipient) == false) {
     return false;
}
vg Wolfgang

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

Re: Multinewsletter Addon Feedback / Bugfix

12. Jul 2016, 14:17

Hi Wolfgang,

danke für deinen super Post! Zum Versionsstring: ich habe vergessen die Versionsnummer zu ändern. Das Update lief aber korrekt durch. Sorry!

Zu der anderen Frage: wäre es nicht interessant, die defekte E-Mailadresse gleich zu löschen?

Liebe Grüße,

Tobias

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

Re: Multinewsletter Addon Feedback / Bugfix

5. Sep 2016, 10:00

Hi Wolfgang,

da ich nichts mehr von dir gehört habe, habe ich deine Vorschläge übernommen. In Version 2.2.5 sind beide Fehler behoben.

Liebe Grüße,

Tobi

Benutzeravatar
gseilheimer
Beiträge: 78
Registriert: 3. Jan 2008, 01:31

Re: Multinewsletter Addon Feedback / Bugfix

19. Okt 2016, 10:51

Moin zusammen. Habe ein Frage zum An- / Abmeldeformular von MultiNewsletter in REX4.
In den Vorversionen konnte mit einfachem PHP die Eintragung in die Liste erfolgen. Das hat sich wohl mit der neueren Version geändert.

z.B.:
<?php
//------------------------------> Funktionen
if ( !function_exists( 'add_newsletter_user')):

function add_newsletter_user( $name, $vorname, $email) {
if( $vorname == '')
{
return 'Bitte f&uuml;llen Sie das Feld "Vorname" aus!';
}

if( $name == '') {
return 'Bitte f&uuml;llen Sie das Feld "Nachname" aus!';
}

if( $email == '')
{
return 'Bitte f&uuml;llen Sie das Feld "Email Adresse" aus!';
}

$sql = new sql();

$qry = 'SELECT id FROM rex_375_user WHERE email = "'. $email .'"';
$sql->setQuery($qry);

if ( $sql->getRows() > 0)
{
$qry = 'UPDATE rex_375_user SET status=1 WHERE email = "'. $email .'"';
$sql->setQuery($qry);
}
else
{
$qry = 'INSERT INTO rex_375_user SET firstname ="'. $vorname . '", lastname="'. $name .'", email = "'. $email .'", title=1, status=1, subscriptiontype="frontend"';
$sql->setQuery($qry);

$qry = 'SELECT id FROM rex_375_user WHERE email = "'. $email .'"';
$sql->setQuery($qry);
if ($sql->getRows() > 0)
{
$id = $sql->getValue('id');
$qry = 'INSERT INTO rex_375_user2group SET uid='.$id.' ,gid=1';
$sql->setQuery($qry);
}
}

return $sql->getError() == '';
}

endif;

if (!function_exists( 'remove_newsletter_user')):

function remove_newsletter_user($email) {
if( $email == '')
{
return 'Bitte f&uuml;llen Sie das Feld "Email Adresse" aus!';
}
$qry = 'UPDATE rex_375_user SET status=0 WHERE email = "'. $email .'"';

$sql = new sql();
$sql->setQuery($qry);

return $sql->getError() == '';
}

endif;

//------------------------------> Formularauswerten
$message = '';
if ( !empty($_POST['newsletter_anmelden']))
{
$result = add_newsletter_user( $_POST['newsletter_name'], $_POST['newsletter_firstname'], $_POST['newsletter_email']);
if ( $result === true)
{
$message .= 'Benutzer wurde erfolgreich für den Newsletter angemeldet!';
$message .= $id;
}
else
{
$message .= $result . '<br/>Anmeldung konnte <u>nicht</u> durchgef&uuml;rt werden!';
}
}
elseif ( !empty($_POST['newsletter_abmelden']))
{
$result = remove_newsletter_user( $_POST['newsletter_email']);
if ($result === true)
{
$message .= 'Benutzer wurde erfolgreich für den Newsletter abgemeldet!';
$message .= $id;
}
else
{
$message .= $result . '<br/>Abmeldung konnte <u>nicht</u> durchgef&uuml;rt werden!';;
}
}

//------------------------------> Formular

if ( $message != '')
{
echo '<h4>'. $message .'</h4><br /><br />';
}
?>
Gern würde ich bei der KISS-Methode für die An- /Abmeldung bleiben. (An- und Abmeldeformular per PHP übergibt die Daten an das AddOn direkt ohne die Verwendung der im Backend hinterlegten Klassen)
=> Kennt sich damit jemand besser aus und könnte mir weiterhelfen?

Vielen Dank.

Gruß, GS

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

Re: Multinewsletter Addon Feedback / Bugfix

19. Okt 2016, 10:56

Hi gs,

ich habe nicht genau verstanden, was dir im aktuellen Addon Modul fehlt. Fehlt eine Funktion oder geht es nur im die Verwendung der Klassen.

Wenn es um die Verwendung der Klassen geht: die nehmen noch Prüfungen vor und fangen eine denkbare SQL Injection Attacke ab.

Hilft dir das weiter?

Liebe Grüße,

Tobi

Benutzeravatar
gseilheimer
Beiträge: 78
Registriert: 3. Jan 2008, 01:31

Re: Multinewsletter Addon Feedback / Bugfix

19. Okt 2016, 11:09

Tobias Krais hat geschrieben:Hi gs,

ich habe nicht genau verstanden, was dir im aktuellen Addon Modul fehlt. Fehlt eine Funktion oder geht es nur im die Verwendung der Klassen.

Wenn es um die Verwendung der Klassen geht: die nehmen noch Prüfungen vor und fangen eine denkbare SQL Injection Attacke ab.

Hilft dir das weiter?

Liebe Grüße,

Tobi
Hallo, Tobias,

danke für dein Feedback. Ich möchte einen minimalen Weg ohne Überprüfungen.
Die Seite wird neu überarbeitet und soll einfach noch bis zum Re-Launch fehlerfrei funktionieren.

Wunsch: Der User füllt das Formular aus und sendet es an einen Redaxo-Artikel mit obigem PHP. Das PHP prüft die Eingaben und deaktiviert den User.

Beispiel eines Formulares für die Austragung am Newsletter:
notextile. <div id="tabs" class="tabs" title="Abmeldung">

h2. Abmeldung

<div id="rex-xform" class="xform">
<form action="URL/PFAD/DATEI.html" method="post">
<input type="hidden" name="article_id" value="99"/>
<fieldset>
<p class="formtext">
<label class="text" for="newsletter_email2">Email:*</label>
<input class="text" type="text" name="newsletter_email" id="newsletter_email2" />
</p>
<p class="formsubmit">
<input class="submit" type="submit" name="newsletter_abmelden" value="Abmelden" />
</p>
</fieldset>
</form>
</div>

notextile. </div>

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

Re: Multinewsletter Addon Feedback / Bugfix

19. Okt 2016, 11:45

Hi gs,

wie du willst. Dann mach einfach eine SQL Query mit einem:

Code: Alles auswählen

DELETE FROM rex_375_user WHERE email = 'xyz@domain.de'
Die Formularauswertung bekommst du ja hin, oder?

Liebe Grüße,

Tobi

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 09:10

Hallo, jetzt ist mir eine neue dumme Kleinigkeit untergekommen, wo ich doch den Multinewsletter schon bei vielen Kunden und seit vielen Jahren im Einsatz habe.
Diesmal hat ein Kunde im Namen des Versandartikels ein Hochkomma drin. Und dieses hat den Archivierungsbefehl abgeschossen, allerdings kommentarlos. Jetzt dachte der Kunde natürlich, dass der Versand nicht geklappt hat.
Bei der anschließenden Suche bin ich erst auf den Fehler gekommen, als ich mir den Befehl ausgeben lassen und im PHPmyAdmin ausprobiert habe.
Meine Frage: Kann man das eventuell verhindern bzw. umschreiben? Ich leider nicht ...
Kommt sicher selten vor, aber zur Sicherheit wäre das schon toll!

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

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 09:50

Hi sueshi,

das ist wirklich ein sehr spezieller Fehler. Kannst du mal bitte folgende neue Version der Datei ausprobieren und mir Bescheid geben, ob das Abhilfe schafft: https://github.com/raum2projekt/multine ... er.inc.php

Danke im Voraus!

Liebe Grüße,

Tobi

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 11:43

Bin mir jetzt nicht sicher, ob wir nicht aneinander vorbei reden :?:

Mein Problem liegt in der newsletter.inc.php, die sql-Anweisung für insert bzw. update für das Eintragen ins die Archivtabelle. Der Artikel hieß in diesem Fall "Etienne's...". Nachdem ich das in "Etienne´s..." umgewandelt habe, wurde die Anweisung ausgeführt.

Aber danke für die rasche Begutachtung!
lg,su

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

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 11:56

Hi,

ja, das ist richtig. Der Punkt ist der. In der User Tabelle steht drin, dass der Versand für diesen User noch aussteht. Wenn der Versand erfolgt ist, muss die Änderung, dass kein Versand mehr aussteht in der Usertabelle gespeichert werden. Und das schlägt fehl und der Versand hängt sich auf oder geht in die Schleife. Deshalb die Änderung an der User Klasse.

Könntest du das für mich bitte noch testen?

Liebe Grüße,

Tobi

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 13:18

Jetzt bin ich bis zum "Newsletter vorbereiten" gekommen, dann passiert gar nichts mehr und ich kann nur mehr auf "alles zurücksetzen" ... Wenn ich das Hochkomma aus dem Artikelnamen rausnehme, funktioniert es wieder.

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

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 14:20

Hi,

vielen Dank fürs Testen. Ich denke jetzt hab ichs: https://github.com/raum2projekt/multine ... er.inc.php

Würdest du bitte nochmals testen?

Liebe Grüße,

Tobi

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 15:10

Nach der Gruppenauswahl kommt noch immer nicht das Feld mit "x Newsletter zu versenden", bleibt einfach alles geschlossen ... :(

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

Re: Multinewsletter Addon Feedback / Bugfix

25. Okt 2016, 23:02

Hi sueshi,

deine Gruppe ist leer. Der Fehler wird hier behoben: https://github.com/raum2projekt/multine ... er.inc.php

Wegen dem Fehler mit dem ' habe ich endlich einen Server gefunden, auf dem ich den Fehler nachvollziehen kann: 1&1. Bei meinen allen anderen Servern konnte ich den Fehler mit dem ' nicht nachvollziehen. Hier die Fehler bereinigte Version:
https://github.com/raum2projekt/multine ... er.inc.php

Außerdem ist im Anmeldemodul (Ausgabe) ein Fehler:
https://github.com/raum2projekt/multine ... ut.inc.php

Könntest du mir bitte folgendes nach hochladen der Änderungen testen:
- Anmeldung mit einem Namen mit ', z.B. O'Reilley. Erscheint der Name nach Anmeldung korrekt in der User Liste?
- Kannst du einen Newsletter mit dem gerade angelegten User vollständig versenden?
- Kannst du einen Newsletter an eine Gruppe ohne Nutzer versenden?

Vielen Dank im Voraus!

Liebe Grüße,

Tobi

missmissr
Beiträge: 175
Registriert: 9. Mär 2008, 18:36

Wonach User sortiert: Multinewsletter Addon Feedback / Bugfi

18. Nov 2016, 14:38

Hi Multinewsletter User,

bisher hat Multinewsletter super gearbeitet und heute hat sich irgendwie der Versand aufgehängt. Es wären noch 75 Newsletters zu versenden gewesen.

Meine Frage: Nach welchem Kriterium werden beim Versand die User sortiert? Damit ich herausfinden kann, welche 75 Adressen gefehlt haben. Im Archiv sehe ich, dass an 931 verschickt wurde.

Danke für Hilfe,
missmissr.

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

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 16:53

Hi missmissr,

den Fehler hat mir vor kurzem ein anderer User gemeldet. Welche MultiNewsletter Version verwendest du? Bei dem anderen User war beim Namen des Users ein ' drin (O'Reilley). Das hat dann den Versand aufgehängt. Wie ist das bei dir?

Liebe Grüße,

Tobias

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 17:37

Warum kommen reihenweise Fehlermeldungen mit "Fehler beim Versand an.... bitte prüfen Sie die Benutzerdaten ..."
Es sind keine unerlaubten Zeichen in den Mailadressen, hundertmal nachgeprüft. Wo werden die Mailadressen genau gecheckt?

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 17:39

Noch eine kleine Anregung hätte ich: Früher stand im Archiv immer, an wie viele Adressen der Newsletter gesandt wurde. Könnte man das wieder einfügen? Das war für die Kontrolle immer sehr praktisch ...

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

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 18:25

sueshi hat geschrieben:Warum kommen reihenweise Fehlermeldungen mit "Fehler beim Versand an.... bitte prüfen Sie die Benutzerdaten ..."
Es sind keine unerlaubten Zeichen in den Mailadressen, hundertmal nachgeprüft. Wo werden die Mailadressen genau gecheckt?
Hi Sueshi,

kannst du mir per PM ein paar der Adressen mailen? Oder darf ich mal auf die DB schauen? So Pauschal kann ich nicht sagen wie das Zustande kommt.

Liebe Grüße,

Tobi

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

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 18:31

sueshi hat geschrieben:Noch eine kleine Anregung hätte ich: Früher stand im Archiv immer, an wie viele Adressen der Newsletter gesandt wurde. Könnte man das wieder einfügen? Das war für die Kontrolle immer sehr praktisch ...
Hi sueshi,

Kannst du mal schauen ob die neue Version der Datei https://github.com/raum2projekt/multine ... ve.inc.php deine Wünsche erfüllt? Bitte gib mir Bescheid.

Liebe Grüße,

Tobi

missmissr
Beiträge: 175
Registriert: 9. Mär 2008, 18:36

Antwort auf Mailabbruch: Multinewsletter Addon Feedback / Bu

18. Nov 2016, 19:03

Hi Tobias,

ich habe die MultiNewsletter 2.1.1. Bisher hat das echt brav gearbeitet. Das mit den Sonderzeichen und Fehlern in den Mailadressen ist mir auch schon einmal passiert, das weiß ich normal. Damals waren Leerstellen in den Mailadressen weil die in der Datenbank des Unternehmens drin waren und ich die Adressen kopiert hatte. Seit dem kontrolliere ich alle Mailsaddis immer, ich schaue noch einmal, ob mir was auffällt. Aber ich habe nicht herausgefunden, wonach der Versand die Reihenfolge der Mailadressen sortiert hat weil nur so könnte ich doch die fehlenden 75 herausfinden. Kann es was damit zu tun haben, dass es diesmal das erstemal über 1000 sind? Ne, oder?

Danke für Hilfe,
missmissr.

Tobias Krais hat geschrieben:Hi missmissr,

den Fehler hat mir vor kurzem ein anderer User gemeldet. Welche MultiNewsletter Version verwendest du? Bei dem anderen User war beim Namen des Users ein ' drin (O'Reilley). Das hat dann den Versand aufgehängt. Wie ist das bei dir?

Liebe Grüße,

Tobias

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

Re: Antwort auf Mailabbruch: Multinewsletter Addon Feedback

18. Nov 2016, 19:18

missmissr hat geschrieben:Hi Tobias,

ich habe die MultiNewsletter 2.1.1. Bisher hat das echt brav gearbeitet. Das mit den Sonderzeichen und Fehlern in den Mailadressen ist mir auch schon einmal passiert, das weiß ich normal. Damals waren Leerstellen in den Mailadressen weil die in der Datenbank des Unternehmens drin waren und ich die Adressen kopiert hatte. Seit dem kontrolliere ich alle Mailsaddis immer, ich schaue noch einmal, ob mir was auffällt. Aber ich habe nicht herausgefunden, wonach der Versand die Reihenfolge der Mailadressen sortiert hat weil nur so könnte ich doch die fehlenden 75 herausfinden. Kann es was damit zu tun haben, dass es diesmal das erstemal über 1000 sind? Ne, oder?

Danke für Hilfe,
missmissr.
Hi missmissr,

nein, mit den 1000 hat es nichts zu tun. Bitte mach ein Update auf die aktuelle Version (einfach per Installer Addon). Dann sollte dein Fehler weg sein.

Liebe Grüße,

Tobias

sueshi
Beiträge: 96
Registriert: 6. Feb 2007, 13:37
Wohnort: Österreich

Re: Multinewsletter Addon Feedback / Bugfix

18. Nov 2016, 19:25

Funktioniert, nur die Übersetzung fehlte noch, habe "multinewsletter_archive_recipients_count = Anzahl" der lang-Datei hinzugefügt.
Danke!!!

Zurück zu „Allgemeines [R4]“