Symfony mailer supports embedding images and file attachements (https://symfony.com/doc/5.4/mailer.html#embedding-images / https://symfony.com/doc/5.4/mailer.html#file-attachments). At the moment this has not yet been implemented (as noted in https://www.drupal.org/docs/contributed-modules/symfony-mailer-0/feature → ...), and the related methods are commented in Drupal\symfony_mailer\BaseEmailTrait.
Interface:
attachFromPath(). Following Drupal conventions, probably the parameter should be called mime type instead of content type and if blank we should use the Drupal MimeTypeGuesser.embedFromPath() however we probably want the function to take care of assigning cid else every caller has to. Maybe it could be the same function as attachFromPath() - in the embed case the cid is used, in the attach case it is not (is there any other difference??)attachFile() from patch 15 which is a simple wrapper round the aboveattach() or embed() because is loses the source of the file, which we most likely want for security checkingattachPart() as it doesn't seem to relevant to DrupalgetAttachments() not sure of the use case, because there is no way to modify themHere are some ideas for how we might use attachments:
We need to consider security to control attaching of protected files. For Drupal managed files we could perhaps automatically check if the recipient has access to the file. However sometimes the admin might wish to override this - "yes the file is not public, but I am choosing to send it to this set of recipients".
We should check the webform module because it has similar features to some of these.
The OP requested to implement direct support for the symfony mailer functions below. As maintainer, I propose that we start from the requirement discussion above and see what API it leads to.
Active
1.0
Code
Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Update patch to work with the latest stable release.
Update patch to work with the latest stable release. Forgot the staged files.
This is a meta issue and its purpose is to group together related issues. Please create separate issues for each part of the work. Patches here will not be committed.
Well, in the mean-time, thanks for keeping the patches updated @Wouter Waeytens .