Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I am running drupal 7.22
webform 7.x-4.0-alpha6
Mail system 7.x-2.34
with the latest dev version of mimemail
I am trying to get mimemail to send files uploaded to my webforms as attachments.
This is working just fine when I send files that do not have spaces in them.
But if the filenames have spaces in them, the files do not get attached to the email.
Is this a mimemail issue?
Thanks
alan
Comment | File | Size | Author |
---|---|---|---|
#12 | mimemail-2087913-12.patch | 3.9 KB | sgabe |
#9 | mimemail-2087913-9.patch | 2.81 KB | sgabe |
#6 | mimemail-2087913-6.patch | 980 bytes | sgabe |
Comments
Comment #1
trentl CreditAttribution: trentl commentedUsing version beta1 of mime mail I am seeing the same thing. Files with spaces in their names do not get attached, and files without do.
Comment #2
trentl CreditAttribution: trentl commentedI can confirm that this is a bug in Mime Mail. 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.
Since this is a support request against the dev version, I'm going to open a bug in the beta2 queue.
Comment #3
sgabe CreditAttribution: sgabe commented@geeklygroup: Please do not open a new issue, but change this one to a bug report.
Comment #4
belio CreditAttribution: belio commentedI'm having the same problem. Rolling back to Alfa2 for now.
Comment #5
splitsplitsplit CreditAttribution: splitsplitsplit commentedAlso having the same problem in the beta. Rolled back to the alpha.
Also in the alpha2 version I can't seem to send text files. Word files and pdf's work fine.
Comment #6
sgabe CreditAttribution: sgabe commentedPlease test the attached patch and report back.
Comment #8
jesss CreditAttribution: jesss commentedI applied the patch manually and it's working great for me. Thanks!
Comment #9
sgabe CreditAttribution: sgabe commentedUpdated patch attached with refactored _mimemail_url() function and a tiny test for this specific bug.
Comment #11
sgabe CreditAttribution: sgabe commentedVersion should be set to the branch to allow tests.
Comment #12
sgabe CreditAttribution: sgabe commentedThe attached patch with refactored _mimemail_url() function and some tests has been committed.