E-Mail Attachments no more work in 7.x-1.x-beta1 and 7.x-1.x-dev but were working before in alpha2

Seconded -- rolling back to alpha2 fixed the issue.

Noticed the issue through integration with mass_contact module.

Yes, I am able to reproduce this issue! Using Rules with Mime Mail 7.x-1.x-beta1 no longer sends out emails with attachments.

Logfile:
The reaction rule signup (rules_ signup_e_mail_pdf) fails the integrity check and cannot be executed. Error: Missing configuration for parameter key.

It’s working with 7.x-1.0-alpha2!

@howdytom: You should update your rules settings as stated in the release note:

Since now it is possible to use custom keys and set the language for messages, be sure to set a unique key for your Rules and system actions and test your custom theme implementations!

@sgabe: You're absolutely right! I've missed the release note! Sorry.

Once I've set the unique key for our Rules action Mime Mail 7.x-1.x-beta1 is working again.

Check the location of your attachments. The files cannot be outside the public files directory just if you enable the "Send arbitrary files" permission.

The temporary directory can be set and usually defaults to be inside the public files directory.

@Anybody check "Site-wide default MailSystemInterface class" here admin/config/system/mailsystem, you should have MimeMailSystem class selected, because default Drupal class doesn't know how to handle attachments at all.

I don't know if this helps, but it seems to have to do with spaces in the filenames. I rolled back go Alpha2 and Mime Mail successfully sends attachments with spaces in the file names. Beta1 does not.

Steps to reproduce:

1) Create form with file field component
2) Configure form and check "send files as attachments"
3) Test form by uploading file name with spaces

Result - if file name has spaces, it will not be sent as an attachment. Using alpha2 version does not exhibit this behavior.

i will note that I am using "private files" setting

@Anybody: Currently this is mentioned in the documentation of mimemail_prepare_message(). Please make your suggestions and comments to improve this.

/**
 * Prepares the message for sending.
 *
 * @param array $message
 *   An array containing the message data. The optional parameters are:
 *   - plain: Whether to send the message as plaintext only or HTML. If evaluates to TRUE,
 *     then the message will be sent as plaintext.
 *   - plaintext: Optional plaintext portion of a multipart email.
 *   - attachments: An array of arrays which describe one or more attachments.
 *     Existing files can be added by path, dinamically-generated files
 *     can be added by content. The internal array contains the following elements:
 *      - filepath: Relative Drupal path to an existing file (filecontent is NULL).
 *      - filecontent: The actual content of the file (filepath is NULL).
 *      - filename: The filename of the file.
 *      - filemime: The MIME type of the file.
 *      The array of arrays looks something like this:
 *      Array
 *      (
 *        [0] => Array
 *        (
 *         [filepath] => '/sites/default/files/attachment.txt'
 *         [filecontent] => 'My attachment.'
 *         [filename] => 'attachment.txt'
 *         [filemime] => 'text/plain'
 *        )
 *      )
 *
 * @return array
 *   All details of the message.
 */
function mimemail_prepare_message($message) {
  [...]

7.x-1.0-beta3 Attachments do not work because of the following codes in mimemail.module the hook:
...

      function mimemail_mail($key, &$message, $params) {
  $context = $params['context'];
  $options = array('clear' => TRUE);

  // Prepare the array of the attachments.
   $attachments = array();
 $attachments_string = trim($params['attachments']);
  if (!empty($attachments_string)) {
    $attachment_lines = array_filter(explode("\n", trim($attachments_string)));
    foreach ($attachment_lines as $filepath) {
      $attachments[] = array(
        'filepath' => trim($filepath),
      );
    }
  }

This code expects attachments to be a long string of paths delimited by \n . it conflicts directly with the function

function mimemail_prepare_message($message)

which expects the attachment to be in form of an array of arrays :

* The array of arrays looks something like this:
* Array
* (
* [0] => Array
* (
* [filepath] => '/sites/default/files/attachment.txt'
* [filecontent] => 'My attachment.'
* [filename] => 'attachment.txt'
* [filemime] => 'text/plain'
* )
* )

The updated code also has no chance of working with different mime types (which I need!), I guess there can be some code to declare the mime type based on the file suffix, but it will make the module less flexible. The current format of attachments array of arrays is ok and is working!

I modified the function as follows and it now works perfectly (for my purposes anyway!)

function mimemail_mail($key, &$message, $params) {
  $context = $params['context'];
  $options = array('clear' => TRUE);

  // Prepare the array of the attachments.
  $attachments = $params['attachments'];

/* $attachments = array();
$attachments_string = trim($params['attachments']);
if (!empty($attachments_string)) {
$attachment_lines = array_filter(explode("\n", trim($attachments_string)));
foreach ($attachment_lines as $filepath) {
$attachments[] = array(
'filepath' => trim($filepath),
);
}
}*/

Something is terribly wrong there. As I mentioned this in a previous comment.

[...] the hook_mail() implementation in the module considers file attachments as a multiline string because it handles messages sent via Rules actions provided by MimeMail itself. For more information see drupal_mail().

You should never ever call drupal_mail('mimemail', [..])!

Same problem as #17 and #18 here.

Here's a patch that fixes the issue.