Böser Bug in Newsletteranmeldung

[info@…]

Wenn man seine erste Newsletteranmeldung nicht bestätigt, hat man nie wieder die Chance, sich für den Newsletter einzutragen, weil der versendete Bestätigungscode ein neuer ist, der Eintrag in der Datenbank aber nicht geändert wird und man folglich einen falschen Bestätigungscode zugesendet bekommt, mit dem man seine Anmeldung dann natürlich nicht bestätigen kann...

Reproduktion:

• Beim Newsletter anmelden
• Bestätigungsmail löschen/nicht beachten
• Noch einmal anmelden
• Link in der zweiten Bestätigungsmail anklicken

Meiner Meinung sollte es an der Stelle bei der zweiten Anmeldung auch nicht die Meldung geben "Diese E-Mail-Adresse existiert bereits in unserer Datenbank, ist aber noch nicht für den Empfang des Newsletters freigeschaltet!"

Das dürfte einen, der sich neu für den Newsletter anmelden möchte, herzlich wenig interessieren, da er sich nun anmelden möchte und offensichtlich die erste Bestätigungsmail nicht bekommen oder sich doch dagegen entschieden hat. Es würde also der Hinweis genügen, dass die Bestätigungsmail raus ist - ganz so wie bei der ersten Anmeldung...

Wenn gewünscht, kann ich auch kurz die Korrektur ausarbeiten...

[kaisa]

Ich bin kein Profi, aber ich glaube, ich habe es hinbekommen:
/newsletter.php
ab Zeile 106:

if ($check_mailmail_status == '0') {

BOF *KS* update mail_key
$info_message = TEXT_EMAIL_EXIST_NO_NEWSLETTER;

$info_message = TEXT_EMAIL_INPUT;

xtc_db_query("update ".TABLE_NEWSLETTER_RECIPIENTS." set mail_key = '".$vlcode."' where customers_email_address = '".$check_mailcustomers_email_address."'");

EOF *KS* Muss man da xtc_db_input() benutzen?

ggf. korrigieren.
ES wird einfach der neue vlcode in der Datenbank eingetragen und der normale Hinweistext ausgegeben, als wenn es keinen alten Eintrag gäbe. So ist es am benutzerfreundlichsten.

[kaisa]

gleich noch hinzugefügt bei Zeile 165:
BOF *KS* Date gets updated when subscriptions is activated

xtc_db_query("update ".TABLE_NEWSLETTER_RECIPIENTS." set mail_status = '1' where customers_email_address = '".xtc_db_input($_GETemail)."', date_added='".now()."'");

EOF *KS*

So ist genauer nachvollziehbar, wann wirklich die Zustimmung zum Newsletterversand gegeben wurde und Änderungen sind auch nachvollziehbar.

[Gerhard Waldemair]

Newsletterklasse wurde komplett überarbeitet