Hi all,

I found an issue with file email attachement when the file field is multiple.
I made a little patch.

Could you fix this in the next version ?
Thanks,

CommentFileSizeAuthor
#6 issue_with_email-2837644-6.patch1.09 KBjrockowitz
#5 issue_with_email-2837644-5.patch1.79 KBjrockowitz
Fix_Issue_Email_Attachment_when_multiple_files_.patch1.88 KBromualdbrisson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

romualdbrisson created an issue. See original summary.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, Fix_Issue_Email_Attachment_when_multiple_files_.patch, failed testing.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented

@romualdbrisson You might just have to generate the patch via CLI. I have never had any luck with an IDE generated patch working with D.O.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs work » Reviewed & tested by the community
FileSize
issue_with_email-2837644-5.patch1.79 KB

I rerolled the patch.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Reviewed & tested by the community » Needs review
FileSize
issue_with_email-2837644-6.patch1.09 KB

Here is a tidied patch.

  • jrockowitz committed ffa9537 on 8.x-5.x 
    Issue #2837644 by jrockowitz, romualdbrisson: Issue with email...
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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