Newsletter Anmeldung

[Anonym]

Ich weiß nicht wieso das noch keine geändert hat, denn es verwirrt Kunden nur und es erfolgt auch keine Überprüfung der Eingaben bis auf den Sicherheitscode.

In der aktuellen Version wird nur der Sicherheitscode überprüft, zumindest erfolgt nur diese Fehlerausgabe.

Ich habe jetzt die newsletter.php soweit erweitert, dass für den Sicherheitscode, ob "eintragen" & "austragen" und für eine nicht korrekte eMail Adresse eine Fehlermeldung ausgegeben wird.

Das funktioniert auch soweit bis auf die Ausgabe einer nicht korrekten eMail Adressen Eingabe.

Ich weiß nicht wo ich da die falsche else Anweisung gesetzt habe, sehe auch den Fehler nicht mehr nach so vielen Stunden. Vielleicht kann da bitte jemand einmal drübersehen.
Änderungen beginnen mit // WM...

In der german.php müssen folgene Fehlermeldungen noch rein:

Code: PHP  [Auswählen]

// BUGIFX WM Fehlende Newsletter Fehlermeldungen
define('TEXT_WRONG_EMAIL','<font color="#ff0000">Deine eingegebene eMail Adresse ist nicht g&uuml;ltig. Bitte pr&uuml;fe die Schreibweise.<br>Bei weiteren Problemem nehmen Sie bitte Kontakt mit uns auf.</font>');
define('TEXT_WRONG_EA_AND_CODE','<font color="#ff0000">Der eingegebener Sicherheitscode stimmte nicht mit dem angezeigten Code &uumlberein.<br>Ebenso muss "eintragen" oder "austragen" auswählen sein."<br>Bitte wiederholen Sie Ihre Eingaben und versuchen es erneut.</font>');
define('TEXT_WRONG_EA','<font color="#ff0000">Es muss "eintragen" oder "austragen" gew&auml;hlt werden."<br>Bitte wiederholen Sie Ihre Eingabe.</font>');
 

Linkback: https://www.modified-shop.org/forum/index.php?topic=1177.0

[Anonym]

Eine weitere Frage: Wofür wird denn noch das db Feld: newsletter_history verwendet. In den ganzen Shop Dateien wird diesen Tabelenfeld nicht abgefragt bzw. kann es in keiner Abfrage feststellen. Ist das noch ein Überbleibsel von alten Versionen und kann gelöscht werden?

[Anonym]

Und zu guter letzt die berichtigten Sprächschnipsel. Da waren auch noch Rechtschreibfehle drin.

Code: PHP  [Auswählen]

define('TEXT_WRONG_CODE','<font color="#ff0000">Ihr eingegebener Sicherheitscode stimmte nicht mit dem angezeigten Code &uuml;berein.<br />Bitte versuchen Sie es erneut.</font>');
define('TEXT_EMAIL_EXIST_NO_NEWSLETTER','<font color="#ff0000">Diese eMail Adresse existiert bereits in unserer Datenbank ist aber noch nicht f&uuml;r den Empfang des Newsletters freigeschaltet!</font>');
define('TEXT_EMAIL_EXIST_NEWSLETTER','<font color="#ff0000">Diese eMail Adresse existiert bereits in unserer Datenbank und ist f&uuml;r den Newsletterempfang bereits freigeschalten!</font>');
define('TEXT_EMAIL_NOT_EXIST','<font color="#ff0000">Diese eMail Adresse existiert nicht in unserer Datenbank!</font>');
define('TEXT_EMAIL_DEL','Ihre eMail Adresse wurde aus unserer Newsletterdatenbank gel&ouml;scht.');
define('TEXT_EMAIL_DEL_ERROR','<font color="#ff0000">Es ist ein Fehler aufgetreten, Ihre eMail Adresse wurde nicht gel&ouml;scht!</font>');
define('TEXT_EMAIL_ACTIVE','<font color="#ff0000">Ihre eMail Adresse wurde erfolgreich f&uuml;r den Newsletterempfang freigeschaltet!</font>');
define('TEXT_EMAIL_ACTIVE_ERROR','<font color="#ff0000">Es ist ein Fehler aufgetreten, Ihre eMail Adresse wurde nicht freigeschaltet!</font>');
define('TEXT_EMAIL_SUBJECT','Ihre Newsletteranmeldung');
 

[Anonym]

Noch ein Bug in dem Ding. Es arbeitet wohl keiner mit Newslettern hier, ansonsten hätte dies doch auffallen müssen. Hat schon jemand versucht sich per Link, der in einem Newsletter mitgesendet wird abzumelden? Geht nicht. Da steht sehr dubioses Zeug in der newsletter.php. Man versucht dort im Abmeldelink den gesendeten Key mit der gesendeten eMail Adresse zu vergleichen und das ganze dann per xtc_validate_password.inc.php. Was soll denn das. Aus der mitgesendeten eMail Adresse erst mal einen  md5 String zu basteln und diesen dann mit dem zufällig generierten Key zu vergleichen?

Wieviel Bölkstoff hat man denn da im Kopf gehabt. Das kann nie funktionieren.

Also in der newsletter.php im Bereich "// Accountdeaktivierung per Emaillink" dies suchen und schnell entfernen:

Code: PHP  [Auswählen]

if (!xtc_validate_password($check_mail['customers_email_address'], $_GET['key'])) {
 

durch dies ersetzen:

Code: PHP  [Auswählen]

if (!$check_mail['mail_key'] == $_GET['key']) {
 

dann klappt es auch mit der Abmeldung per Link

[Tomcraft]

Danke für den Fix... hast recht, wir sind nicht so die Newsletter Fans. Die meisten Kunden ärgert es mehr als es Nutzen für den Shop-Betreiber bringt.

Mal schauen, wann einer von uns mal Zeit findet das genauer anzuschauen und es evtl. zusammen mit deinem Coupon-Fix ins Trunk zu übernehmen.

[Tomcraft]

test