<div dir="ltr">This is the function that VTEmailTask uses, which we use for workflow-driven transactional emails.  It hasn't given us any trouble.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 19, 2013 at 3:14 AM, Alan Lord <span dir="ltr"><<a href="mailto:alanslists@gmail.com" target="_blank">alanslists@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just noticed some odd stuff in the above file.<br>
<br>
in function send_email around line 93:<br>
<br>
        $mail_error = getMailError($mail,$mail_<u></u>status,$mailto);<br>
<br>
$mailto is not defined anywhere.<br>
<br>
In the function getMailError($mail,$mail_<u></u>status,$to)<br>
<br>
$to is never examined!<br>
<br>
Also $error_msg is not set in the final else block (It's commented out) so the return $error_msg will return NULL I guess.<br>
<br>
This all looks very dodgy and old code from what I can tell.<br>
<br>
Should I be using a different mailer tool for sending emails from custom cron scripts?<br>
<br>
Cheers<br>
<br>
Al<br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
<a href="http://www.vtiger.com/" target="_blank">http://www.vtiger.com/</a><br>
</blockquote></div><br></div>