With the newest release of Webform (4.17) - SMTP Authentication support no longer sends attachments with webform. I found the code in the webform module that was removed that prevented this:

In the component/file.inc file between lines 502 and 503, the following was removed (since the current version of mimemail no longer needs it):

// This is necessary until the next release of mimemail is out, see #1388786: mimemail_html_body fails to retrieve file's URI.
$file['filepath'] = $file['uri'];$file['filepath'] = $file['uri'];

SMTP Authentication uses the 'filepath' information to check on attaching a file (line 489 of smtp.mail.inc). This will need to be rehandled in SMTP Authentication Support to fix the issue.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dcamburn created an issue. See original summary.

dcamburn’s picture

Issue summary: View changes
dcamburn’s picture

Issue summary: View changes
Liam Morland’s picture

Version: 7.x-1.7 » 7.x-1.x-dev
Issue summary: View changes
Status: Active » Needs review
Related issues: +#2966816: Attachments don't work
FileSize
812 bytes
retiredpro’s picture

Thanks for the patch, Liam. This patch works for me. I'm using the latest releases of the following modules:

Webform 7.x-4.18
SMTP 7.x-1.7
MIME Mail 7.x-1.1
Mail System 7.x-2.35

dzepol’s picture

Thanks. Confirming this patch applies correctly and fixes the problem for me too.

Chris Matthews’s picture

Status: Needs review » Reviewed & tested by the community

The patch to smtp.module in #4 applied cleanly to the latest 7.x-1.x-dev snapshot and also fixes this issue for me so with three reviews/tests I'm changing the status to RTBC.

aubjr_drupal’s picture

Also tested this patch. It works for our sites.

aubjr_drupal’s picture

Also tested this patch. It works for our sites.

jroberts’s picture

This patch applied cleanly to our sites using:

SMTP 7.x-1.7
Webform 7.x-4.18
MIME Mail 7.x-1.1
Mail System 7.x-2.34

and resolved the attachment issue.

sheshsharma’s picture

tested this patch. it work for our site

aubjr_drupal’s picture

If there are any maintainers for this module left around, can you please apply this patch for us and at least get it into the dev branch?

Liam Morland’s picture

There are 18 RTBC issues and no commits for 11 months. We probably need someone new to step-up to help with maintenance.

Chris Matthews’s picture

wundo’s picture

wundo’s picture

Status: Reviewed & tested by the community » Fixed

  • wundo committed 1b04d4b on 7.x-1.x authored by Liam Morland
    Issue #2979682 by Liam Morland, dcamburn, aubjr_drupal, Chris Matthews,...
wundo’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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