[Tickets #7249] Re: "My email adresses" and "Addresses to not send responses to" fields are not checked for valid email address

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/7249
------------------------------------------------------------------------------
  Ticket             | 7249
  Updated By         | yann at pleiades.fr.eu.org
  Summary            | "My email adresses" and "Addresses to not send
                     | responses to" fields are not checked for valid email
                     | address
  Queue              | Ingo
  Version            | 1.2
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


yann at pleiades.fr.eu.org (2008-08-28 10:23) wrote:

I propose a possible solution for that bug with the attached patch.

I created a Horde_Form_Type_longemail class extended from  
Horde_Form_Type_longtext, which in addition checks the validity of the  
given emails.

What do you think about this solution ?


You will notice that the longemail type also checks if the email  
address have a @domain part, I am not sure if this should be done here.
sieve doesn't like address without domain, it complains with the  
following error:
    script errors: line 8: address 'test': parse error, unexpected $,  
expecting '@'

but that might be solved differently by adding the default domain to  
unqualified address before sending them to the sieve driver.