Sounds good to me. But maybe we should consider marking the function as obsolete and create a "COM_mail" class to send email?<br><br>-Vinny<br><br><div class="gmail_quote">On Sat, Jul 25, 2009 at 11:47 AM, Dirk Haun <span dir="ltr"><<a href="mailto:dirk@haun-online.de">dirk@haun-online.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Coming back to this:<br>
<div class="im"><br>
>>details: <a href="http://project.geeklog.net/cgi-bin/hgweb.cgi/rev/3b8700815448" target="_blank">http://project.geeklog.net/cgi-bin/hgweb.cgi/rev/3b8700815448</a><br>
>>changeset: 7122:3b8700815448<br>
>>user: blaine Lang <<a href="mailto:blaine@portalparts.com">blaine@portalparts.com</a>><br>
>>date: Mon Jun 15 10:07:53 2009 -0400<br>
>>description:<br>
>>Added support for Bcc - Blind copy.<br>
><br>
>Blaine, any objections if we roll back that change and push it to 1.6.1<br>
>instead?<br>
<br>
</div>COM_mail currently looks like this:<br>
<br>
function COM_mail( $to, $subject, $message, $from = '', $html = false,<br>
$priority = 0, $cc = '' )<br>
<br>
My suggestion would be to turn $cc into an array of additional email<br>
headers. If it's an array, then it's supposed to consist of Header<br>
Name / Value pairs, e.g.<br>
<br>
array('Cc' => '<a href="mailto:john@example.com">john@example.com</a>', 'Bcc' => '<a href="mailto:jane@example.com">jane@example.com</a>', ...)<br>
<br>
For backward compatibility, we would of course also accept a string<br>
value, in which case it's simply the CC address.<br>
<br>
Comments?<br>
<br>
bye, Dirk<br>
<font color="#888888"><br>
<br>
--<br>
<a href="http://www.geeklog.net/" target="_blank">http://www.geeklog.net/</a><br>
</font><div><div></div><div class="h5"><a href="http://geeklog.info/" target="_blank">http://geeklog.info/</a><br>
<br>
_______________________________________________<br>
geeklog-devel mailing list<br>
<a href="mailto:geeklog-devel@lists.geeklog.net">geeklog-devel@lists.geeklog.net</a><br>
<a href="http://eight.pairlist.net/mailman/listinfo/geeklog-devel" target="_blank">http://eight.pairlist.net/mailman/listinfo/geeklog-devel</a><br>
</div></div></blockquote></div><br>