wp_mail() does not allow"<firstname.lastname@example.org>", regression
|Reported by:||kitchin||Owned by:||westi|
This regression was caused by Ticket #17305: wp_mail() does not accommodate address format "Name <address@…>".
The format "<address@…>" no longer works, it has to be "address@…". In Ticket #17305 Comment 12 commenter Pavelevap says this is a regression.
I explained already in Ticket #17305 Comment 11 why the new regex is not optimal (greediness), and I provided a new regex:
patch was: '/(.+)\s?<(.+)>/'
should be: '/(.*)<(.+)>/'
I called it an enhancement, because the new regex would make "<address@…>" work without a "Name" in front. But it seems actually to fix a regression.
For belt-and-suspenders, trim() can be applied to the results, as it is now, though the call to AddAnAddress() actually does the trim() anyway.
Change History (10)
- Keywords westi-likes added
- Milestone changed from Awaiting Review to 3.3
- Owner set to westi
- Status changed from new to accepted
comment:5 SergeyBiryukov — 2 years ago
- Component changed from Mail to General
- Keywords needs-unit-tests added; needs-codex removed