[Vtigercrm-commits] [vtiger development] #4116: In emails module : cannot assign emails within <> in bcc

[vtiger development]

#4116: In emails module : cannot assign emails within <> in bcc
-----------------------------+----------------------------------------------
  Reporter:  etienne.jodoin  |       Owner:  etienne.jodoin
      Type:  defect          |      Status:  closed        
  Priority:  unassigned      |   Milestone:  5.0.4         
 Component:  vtigercrm       |     Version:  5.0.3         
Resolution:  fixed           |    Keywords:                
-----------------------------+----------------------------------------------
Changes (by etienne.jodoin):

  * status:  new => closed
  * resolution:  => fixed

Comment:

 This patch works in Firefox but not in IE. Need more investigation :

 File Smarty/Templates/ComposeEmail.tpl, line 204, replace :


 {{{
 for(var i=0; i<=arr.length-1; i++)
   {
    if(trim(arr[i]) != "" &&
 !/^[a-z0-9]([a-z0-9_\-\.]*)@([a-z0-9_\-\.]*)(\.[a-z]{2,3}(\.[a-z]{2}){0,2})$/.test(trim(arr[i])))
    {
     alert(bcc_err_msg+": "+arr[i]);
     return false;
    }
   }

 }}}

 By :


 {{{
 for(var i=0; i<=arr.length-1; i++)
   {

    tmp = arr[i];
    if(tmp.match('<') && tmp.match('>')) {
        if(!findAngleBracket(arr[i])) {
                alert(bcc_err_msg+": "+arr[i]);
                return false;
        }
    }
    else if(trim(arr[i]) != "" &&
 !/^[a-z0-9]([a-z0-9_\-\.]*)@([a-z0-9_\-\.]*)(\.[a-z]{2,3}(\.[a-z]{2}){0,2})$/.test(trim(arr[i])))
    {
        alert(bcc_err_msg+": "+arr[i]);
        return false;
    }


   }





 }}}

-- 
Ticket URL: <http://trac.vtiger.com/cgi-bin/trac.cgi/ticket/4116#comment:2>
vtiger development <http://trac.vtiger.com/>
vtigerCRM