contao/test — Issue #1455: Import csv-Mailliste für Newsletter 
Author: importer
Created: 2011-11-29 07:57:07+00:00
State: open
Labels: Defect, Invalid

Hi Leo,

jetzt habe ich noch ein Problem gefunden, zu dem ich hier uns sonst wo nichts vergleichbares finde.

Wie schon in #1426 berichtet, akzeptiert die Funktion sowohl splitFriendlyName() '[' und ']' als auch '<' und '>' zum einschließen der Mail.
wird jetzt eine csv-Datei importiert mit folgenden Inhalt:
test1 [test1@example.com],test2 ,test3@example.com,test4

So werden vier neue Empfänger angelegt. Dabei stellen sich jedoch zwei Probleme:

1. Beim Empfänger test2
An diesen können Mails versendet werden, jedoch wird einerseits in der Liste nur test2 angezeigt, die '<' und '>' werden nicht zu &lt; und &gt; umgesetzt.
Andererseits kann der Eintrag dieses Empfängers nicht mehr direkt bearbeitet werden. (Beim Eintrag im Edit-Feld „test2 " müssen erst manuell die '<' und '>' in '[' und ']' geändert werden, sonst filtert deine stripTags() in den Post-Funktionen die Mail-Adresse heraus.)

2. Beim Empfänger test4
Hier handelt es sich ja nicht um eine gültige e-Mail, was jedoch beim Import nicht erkannt wird.
Damit wird beim Versuch ein Newsletter zu senden eine Fehlermeldung generiert (siehe unten).
In meinem Beispiel erscheint diese nach 3 versendeten Newslettern, nachfolgende Empfänger erhalten diesen Newsletter dann nicht mehr.

Vorschlag:
1. Prüfung der E-Mail-Adressen beim Import aus einer CSV-Datei.
2. Wenn die Nutzung von '<' und '>' bei der Mail nicht erlaubt sein soll, dann sollte die Funktion splitFriendlyName() diese auch nicht berücksichtigen. Zudem solltes du deine Kommentare in der email.php
* Friendly name portions (e.g. Leo ) are allowed.
Vielleicht überarbeiten.
Persönlich würde ich jedoch das Format "Leo " bevorzugen, was anscheinend mehr an Änderungen bedeutet.

Nachfolgend noch die debug-Ausgabe beim Newsletterversand.

Mfg
weke

```
Fatal error: Uncaught exception Swift_RfcComplianceException with message Address in mailbox given [test4] does not comply with RFC 2822, 3.6.2. thrown in /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/Headers/MailboxHeader.php on line 309

#0 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/Headers/MailboxHeader.php(239): Swift_Mime_Headers_MailboxHeader->_assertValidAddress('test4')
#1 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/Headers/MailboxHeader.php(97): Swift_Mime_Headers_MailboxHeader->normalizeMailboxes(Array)
#2 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/Headers/MailboxHeader.php(61): Swift_Mime_Headers_MailboxHeader->setNameAddresses(Array)
#3 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/SimpleHeaderFactory.php(64): Swift_Mime_Headers_MailboxHeader->setFieldBodyModel(Array)
#4 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/SimpleHeaderSet.php(77): Swift_Mime_SimpleHeaderFactory->createMailboxHeader('To', Array)
#5 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/plugins/swiftmailer/classes/Swift/Mime/SimpleMessage.php(301): Swift_Mime_SimpleHeaderSet->addMailboxHeader('To', Array)
#6 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/system/libraries/Email.php(357): Swift_Mime_SimpleMessage->setTo(Array)
#7 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/system/modules/newsletter/Newsletter.php(371): Email->sendTo('test4')
#8 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/system/modules/newsletter/Newsletter.php(179): Newsletter->sendNewsletter(Object(Email), Object(DB_Mysqli_Result), Array, '', 'Testnewsletter', '')
#9 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/system/modules/backend/Backend.php(169): Newsletter->send(Object(DC_Table), 'tl_newsletter', Array)
#10 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/typolight/main.php(99): Backend->getBackendModule('newsletter')
#11 /mnt/web2/13/32/5548232/htdocs/www.mw-keil.de/typolight/main.php(299): Main->run()
#12 {main}
```

Related issues: #1526, #2096

--- _Originally created by weke on January 25th, 2010, at 01:11pm (ID 1455)_