[Tickets #6897] Re: Fix import of multiple email addresses

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6897
-----------------------------------------------------------------------
 Ticket             | 6897
 Updated By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
 Summary            | Fix import of multiple email addresses
 Queue              | Turba
 Version            | HEAD
 Type               | Bug
 State              | Feedback
 Priority           | 1. Low
 Milestone          |
 Patch              |
 Owners             |
-----------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2008-06-12 03:30) wrote:

> If something doesn't validate, then we shouldn't be letting it into
> the email field - that's half the point of this function...

I got the impression the function was there to strip of the "personal"
part of an email address. Well, if it also was intended as validation, then
we should inform the user if something went wrong. IMHO it would be ok to
import the "bad" data and the user can correct it later. But I'm okay with
the change, if you want to you can modify the code to output a notification
including the failing address and skip it from $results.