bug #59836 [Mailer][Postmark] Set CID for attachments when it exists (IssamRaouf)

fabpot · fabpot · commit c1082f255e96 · 2025-02-26T07:56:23.000+01:00
This PR was squashed before being merged into the 6.4 branch. Discussion ---------- [Mailer][Postmark] Set CID for attachments when it exists | Q | A | ------------- | --- | Branch? | 6.4  | Bug fix? | yes | New feature? | no  | Deprecations? | no  | Issues | Fix #59819  | License | MIT  Commits ------- 83b0491 [Mailer][Postmark] Set CID for attachments when it exists
diff --git a/src/Symfony/Component/Mailer/Bridge/Postmark/Transport/PostmarkApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Postmark/Transport/PostmarkApiTransport.php
@@ -147,7 +147,7 @@ private function getAttachments(Email $email): array
             ];
 
             if ('inline' === $disposition) {
-                $att['ContentID'] = 'cid:'.$filename;
+                $att['ContentID'] = 'cid:'.($attachment->hasContentId() ? $attachment->getContentId() : $filename);
             }
 
             $attachments[] = $att;

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ private function getAttachments(Email $email): array`
`147`	`147`	`];`
`148`	`148`
`149`	`149`	`if ('inline' === $disposition) {`
`150`		`- $att['ContentID'] = 'cid:'.$filename;`
	`150`	`+ $att['ContentID'] = 'cid:'.($attachment->hasContentId() ? $attachment->getContentId() : $filename);`
`151`	`151`	`}`
`152`	`152`
`153`	`153`	`$attachments[] = $att;`