I am using MIME Mail with Mime Mail CSS Compressor. The "Link images only" option has no affect. Images are still attached to the mails.

CommentFileSizeAuthor
#4 mimemail-1439918-1.patch575 bytessgabe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sgabe’s picture

Status: Active » Closed (cannot reproduce)

The "Link images only" option allows you to disable embedding of images site-wide, so the images will be inserted in the message with an absolute path. I have just tested it, and it seems to work.

warappa’s picture

Maybe you had the same problem I had: I left "Mail Mime" enabled (don't mix up "Mime Mail" with "Mail Mime")! After I disabled it images were added no more.

gagarine’s picture

Status: Closed (cannot reproduce) » Active

Same problem. I add some image with absolute path in mimemail-message.tpl.php.

Look like that:

<?php
//mimemail-message.tpl.php
$path = drupal_get_path('theme', 'my_theme');
?>
<img alt="" height="192px" width="640px" src="<?php print $path; ?>/img/newsletter_footer.png" />

Those images are attached in the message as files making the mail heavy...

Modules:
Simplenews (simplenews) 7.x-1.0-rc1
Mail System 7.x-2.34
Mime Mail (mimemail) 7.x-1.0-alpha1+1 1-dev

sgabe’s picture

Title: "Link images only" option not working » "Link images only" option not working if the file exists as-is
Status: Active » Postponed
FileSize
575 bytes

If the file exists on the server as-is, _mimemail_url will be skipped, which would check if the image can be embedded.

Though I am attaching a patch, this needs to be postponed because this part of the code will be modified soon.

gagarine’s picture

the patch #4 works for me. Thanks

gagarine’s picture

still postponed?

sgabe’s picture

Title: "Link images only" option not working if the file exists as-is » "Link images only" is not working if the file exists as-is
Status: Postponed » Fixed

Committed to the development snapshot.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.