[Vtigercrm-commits] [vtiger development] #5567: Fix send_mail API

[vtiger development]

#5567: Fix send_mail API
------------------------+---------------------------------------------------
  Reporter:  prasad     |       Owner:  developer 
      Type:  defect     |      Status:  new       
  Priority:  major      |   Milestone:  5.1.0     
 Component:  vtigercrm  |     Version:  5.1.0-val1
Resolution:             |    Keywords:            
------------------------+---------------------------------------------------
Old description:

> If outgoing server is not configured send_mail API should immediately
> return insteading of attempting to invoke $mail->send()

New description:

 If outgoing server is not configured send_mail API should immediately
 return instead of attempting to invoke $mail->send()

Comment (by prasad):

 File: modules/Emails/mail.php

 {{{
 function send_mail(...) {
     // ...
     $mail = new PHPMailer();
 setMailerProperties($mail,$subject,$contents,$from_email,$from_name,...
     setCCAddress($mail,'cc',$cc);
     setCCAddress($mail,'bcc',$bcc);

     // vtmailscanner customization: If Support Reply to is defined use it.
     global $HELPDESK_SUPPORT_EMAIL_REPLY_ID;
     if($HELPDESK_SUPPORT_EMAIL_REPLY_ID && ...
     // END

     // Suggested Fix: Return immediately if Outgoing server not configured
     if(empty($mail->Host)) {
         return 0;
     }
     // END

     $mail_status = MailSend($mail);
 }}}

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