wp_mail() does not allow"<address@tld.com>", regression

[kitchin]

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.

[pavelevap]

Thank you for creating this ticket...

[westi]

We need to add some test cases for this here too - ​http://unit-tests.trac.wordpress.org/browser/wp-testcase/test_includes_pluggable.php#L144

[westi]

Replying to westi:

We need to add some test cases for this here too - ​http://unit-tests.trac.wordpress.org/browser/wp-testcase/test_includes_pluggable.php#L144

​[UT439]

[westi]

I think we can probably remove the trims rather than adding more.

[westi]

In [18717]:

Improve the parsing of email addresses in wp_mail to re-support RFC2822 nameless "<address@…>" style. Fixes #18463 props kitchin and SergeyBiryukov.