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.

Thank you for creating this ticket...

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

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]

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

In [18717]:

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