{"issue":{"created_at":"2011-11-29T15:57:31Z","state":"closed","updated_at":"2011-11-29T15:57:40Z","user":{"gravatar_id":"9994df352157ae1b778b792744bdbf05","id":1073273,"url":"https://api.github.com/users/aschempp","avatar_url":"https://secure.gravatar.com/avatar/9994df352157ae1b778b792744bdbf05?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"aschempp"},"comments":7,"body":"Während der Entwicklung einer Newsletter-Erweiterung ist mir eine Kleinigkeit aufgefallen, welche Probleme bereite könnte.\n\nIn system/modules/newsletter/Newsletter.php, Funktion createNewUser() wird ein neues Mitglied den entsprechenden Newslettern zugeordnet.\n\nDabei 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.\n\nIm zutreffenden Fall wäre das Mitglied dann wohl doppelt in der Tabelle, und würde Newsletter zweimal erhalten.\n\nAus meiner Sicht sollte zuvor die entsprechende E-Mail Adresse aus der tl_newsletter_recipients gelöscht werden.\n\n--- _Originally created on November 13th, 2008, at 04:47pm (ID 258)_","assignee":{"gravatar_id":"87d2e5aefadcb310f9155df28c2b3582","id":1192057,"url":"https://api.github.com/users/leofeyer","avatar_url":"https://secure.gravatar.com/avatar/87d2e5aefadcb310f9155df28c2b3582?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"leofeyer"},"id":2386186,"number":258,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"labels":[{"name":"backend","color":"e6e6e6","url":"https://api.github.com/repos/contao/import/labels/backend"},{"name":"Defect","color":"bd2c00","url":"https://api.github.com/repos/contao/import/labels/Defect"},{"name":"Invalid","color":"ffdd00","url":"https://api.github.com/repos/contao/import/labels/Invalid"}],"title":"Newsletter createNewUser() sollte auf bestehende Abonnemente achten","closed_at":"2011-11-29T15:57:40Z","html_url":"https://github.com/contao/import/issues/258","url":"https://api.github.com/repos/contao/import/issues/258","milestone":{"created_at":"2011-11-29T15:29:28Z","state":"open","due_on":null,"closed_issues":12,"creator":{"gravatar_id":"87d2e5aefadcb310f9155df28c2b3582","id":1192057,"url":"https://api.github.com/users/leofeyer","avatar_url":"https://secure.gravatar.com/avatar/87d2e5aefadcb310f9155df28c2b3582?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"leofeyer"},"number":5,"description":"TYPOlight 2.6.3 (maintenance release)","title":"2.6.3","url":"https://api.github.com/repos/contao/import/milestones/5","open_issues":1}},"action":"closed"}