contao/test — Issue #258: Newsletter createNewUser() sollte auf bestehende Abonnemente achten
Author: aschempp
Created: 2011-11-28 16:29:48+00:00
State: open
Labels: backend, Defect, Invalid

Während der Entwicklung einer Newsletter-Erweiterung ist mir eine Kleinigkeit aufgefallen, welche Probleme bereite könnte.

In system/modules/newsletter/Newsletter.php, Funktion createNewUser() wird ein neues Mitglied den entsprechenden Newslettern zugeordnet.

Dabei wird nicht beachtet ob dieselbe E-Mail Adresse bereits angemeldet ist. Dies könnte z.B. passieren wenn sich ein Mitglied über ein Newsletter-Anmelden-Modul zuvor schon angemeldet hat, sich später (oder auf einer anderen TL-Seite derselben Installation) auch als Mitglied anmeldet.

Im zutreffenden Fall wäre das Mitglied dann wohl doppelt in der Tabelle, und würde Newsletter zweimal erhalten.

Aus meiner Sicht sollte zuvor die entsprechende E-Mail Adresse aus der tl_newsletter_recipients gelöscht werden.

--- _Originally created on November 13th, 2008, at 04:47pm (ID 258)_