First of all, thanks for the module. We heavily depend on swiftmailer on many of our projects and symfony_mailer is far behind when it is about attachmends and embedding images.

Unfortunatly there are 2 code issues, that prevent the embedding of images.
Should be easy and quick fixes.

One is a deprecated call:
https://www.drupal.org/node/3126004
The other is a type while someone converted the code from swiftmailer.

CommentFileSizeAuthor
#3 symfony_mailer_lite-embedded-images-3393324-3.patch1.39 KBzengenuity
symfony_mailer_lite--image-embed-drupal10.1-compatibility.patch985 bytesayalon

Comments

ayalon created an issue. See original summary.

didebru’s picture

didebru
Primary language German
commented
Status: Active » Reviewed & tested by the community

This works for me!
Thanks!

zengenuity’s picture

zengenuity commented
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new symfony_mailer_lite-embedded-images-3393324-3.patch1.39 KB

I've modified this patch to add the backward compatibility mentioned in the change record. Can you test with this version of the patch to make sure it still works with your emails that include embedded images?

didebru’s picture

didebru
Primary language German
commented
Status: Needs work » Reviewed & tested by the community

Still works.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Priority: Normal » Major

I'm bumping this to major since this prevents mails from being sent out after updating to Drupal 10.

  • zengenuity committed c6d98aa6 on 1.0.x 
    Issue #3393324 by ayalon, zengenuity: Image Embed + Drupal 10.1 fatal...
zengenuity’s picture

zengenuity commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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