![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Changes between Version 4 and Version 5 of Ticket #30128, comment 7
- Timestamp:
- 08/06/2025 10:18:08 PM (11 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #30128, comment 7
v4 v5 7 7 This was commented on here #56779, so I think we can kill two birds with one stone with a minimal extra work 8 8 9 I've been trying to work on this patch, but I've noticed there is another function that is on the way and I don't understand what it's doing in formatting.php: `wp_staticize_emoji_for_email` (this function was created one year after this report was made). This is almost a copy-paste of the original header manipulation function in `wp_mail`, definitely a WET function that we should consider refactoring for the health of a patch for this report.9 I've been trying to work on this patch, but I've noticed there is another function that is on the way, and I don't understand what it's doing in `formatting.php`: `wp_staticize_emoji_for_email` (this function was created one year after this report was made). This is almost a copy-paste of the original header manipulation function in `wp_mail`, definitely a WET function that we should consider refactoring for the health of a patch for this report. 10 10 11 11 Finally, I'm attaching some unit tests so anyone that would like to work on refining the current patch could have a testing reference (for both CC/BCC thing and the Custom Headers)
