Set $attachments in drupal_wrap_mail() [#750916]

In the current version of mime mail it's not possible to send attachments via drupal_mail()/hook_mail(). I don't understand why, because its only a little thing to change (see my attachment)...

Usage example:

function mymodule_mail($key, &$message, $params) {
   [...]
   $file = new stdClass();
   $file->filename = 'example.pdf';
   $file->filepath = file_directory_temp().'/example.pdf';
   $message["attachments"]=array($file);
   [...]
}

Comment	File	Size	Author
#6	mimemail.750916_01.patch	835 bytes	sgabe
#5	mimemail_750916.patch	789 bytes	tobiasb
#3	mimemail_hook_mail_attachments_1.patch	1.71 KB	javi-er
	mimemail_hook_mail_attachments.patch	783 bytes	crifi

Comments

Comment #1

sgabe commented 24 March 2010 at 15:14

Status:

Active

» Needs review

Changing status, since we have a patch to review.

Comment #3

javi-er commented 26 August 2010 at 19:37

Version:

6.x-1.0-alpha1

» 6.x-1.0-alpha5

Status	File	Size
new	mimemail_hook_mail_attachments_1.patch	1.71 KB

Hi, I'm using this change in a production site and it's working well, it now allows me to attach a file via hook_mail.
However, the attached patch is in a wrong format, I'm attaching a CVS patch with this change for the latest version (DRUPAL-6--1-0-ALPHA5)

Cheers!