Revision: 12719 http://harbour-project.svn.sourceforge.net/harbour-project/?rev=12719&view=rev Author: vszakats Date: 2009-10-16 11:59:18 +0000 (Fri, 16 Oct 2009)
Log Message: ----------- 2009-10-16 13:54 UTC+0200 Viktor Szakats (harbour.01 syenar.hu) * contrib/hbtip/sendmail.prg * contrib/hbtip/mail.prg ! Fixed encoding of subject, from, to, cc, bcc address lists, in some places it was missing (cc, bcc, from), in some other places multiple address support was missing (to, cc, bcc) from recent addition of encoding support. + Added/Fixed support for human readable name component in e-mail addresses. + Using TIPMAIL:SETHEADER() in HB_SENDMAIL() to set above header components. ! Several minor cleanups in TIPMAIL:SETHEADER(): - handling of empty values - using space instead of tab when passing multiple addresses in headers. Haven't checked the standard, but I used to see space there. - Minor formatting, optimizations. ! Minor tweaks to space and _ char in Q encoding. Now they are both simply encoded. ; Overall probably a rewrite would be the best in case of hbtip mailing, and maybe not just for mailing. Code is full of strange tweaks, redundancy and the class layout/communication is rather strange, f.e. SMTP protocol data is passed via TURL:cFile variable from HB_SENDMAIL to SMTP client class. Also charset support is just an aftertought so user code needs close syncronisation with HB_SENDMAIL() and replicating the same parameter parsing logic on multiple layers of code. And I've probably just scratched the surface, f.e. I didn't test attachments of HTML mails and probably several other options as well. ; Anyway please test. Modified Paths: -------------- trunk/harbour/ChangeLog trunk/harbour/contrib/hbtip/mail.prg trunk/harbour/contrib/hbtip/sendmail.prg This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. _______________________________________________ Harbour mailing list Harbour@harbour-project.org http://lists.harbour-project.org/mailman/listinfo/harbour