Email handler doesn't taking into account the "Included email values" while adding attachments to email.
Means, even if you uncheck some of the managed file field, it still will be send as an attachment.

CommentFileSizeAuthor
#4 include_files_as-2840954-4.patch4.66 KBjrockowitz

Comments

vanyamtv created an issue. See original summary.

jrockowitz’s picture

jrockowitz commented
Priority: Normal » Major

Changing status to 'Major' since this is an unexpected behavior.

  • 7d13008 committed on 2840954-email-file-exclude 
    Issue #2840954: "Include files as attachments" option doesn't taking...
jrockowitz’s picture

jrockowitz commented
Status: Active » Needs review
StatusFileSize
new include_files_as-2840954-4.patch4.66 KB

  • jrockowitz authored c9c2f5a on 8.x-5.x 
    Issue #2840954 by jrockowitz: "Include files as attachments" option...
jrockowitz’s picture

jrockowitz commented
Status: Needs review » Fixed

Since I was able to easily add a valid test and there are no regressions, I committed the patch.

Please download or checkout the latest dev release to review..

Status: Fixed » Closed (fixed)

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