Commit 90140b4
committed
minor #19946 [Mailer] Fixed and clarified custom Content-ID feature documentation (dakujem)
This PR was merged into the 6.4 branch.
Discussion
----------
[Mailer] Fixed and clarified custom Content-ID feature documentation
> Addresses [symfony/symfony #54162](symfony/symfony#54162)
The documentation is incorrect, as it shows calls to `DataPart::setContentId` with strings that are rejected by [the method's implementation](https://github.com/symfony/symfony/blob/7.2/src/Symfony/Component/Mime/Part/DataPart.php#L66), because they do not contain a required `@` character.

I also added a sentence clarifying that the actual Content-ID value will be ramdom-generated by Symfony when not using a custom Content-ID.
This feature has been added in `v6.3` (according to the documentation) and the documentation is incorrect throughout versions `6.3`, `6.4`, `7.0`, `7.1` and `7.2`.
I'm not sure how to distribute this PR to all other branches - should I create separate PRs for each?
Commits
-------
468a9ac [Mailer] Fixed and clarified custom Content-ID feature documentation1 file changed
+4
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
682 | 682 | | |
683 | 683 | | |
684 | 684 | | |
| 685 | + | |
| 686 | + | |
685 | 687 | | |
686 | 688 | | |
687 | 689 | | |
688 | 690 | | |
689 | | - | |
| 691 | + | |
690 | 692 | | |
691 | 693 | | |
692 | 694 | | |
693 | 695 | | |
694 | | - | |
| 696 | + | |
695 | 697 | | |
696 | 698 | | |
697 | 699 | | |
| |||
0 commit comments