BP_Email::set_from() seems to set recipient details, not sender details

[henry.wright]

We can set the from address and name like this: BP_Email::set_from( $email_address, $name );

The problem is the first line of BP_Email::set_from() instantiates an object of type BP_Email_Recipient and passes arguments to that class constructor.

To me, "set from" should set sender info, not recipient info.

[thebrandonallen]

While a little confusing, BP_Email::set_from() does indeed set the from name and from email to the sender info, even though it uses BP_Email_Recipient. BP_Email_Recipient is recipent/sender agnostic, so it works with the info you pass to it's constructor. It's built to handle multiple types of data. This is especially helpful when passing multiple recipients, hence the name, and then formatting/validating the data into reliable objects. In the case of the from name/email, it's essentially just validating the passed from name/email, and returning the appropriate object.

Are you seeing a different behavior?

[r-a-y]

The BP_Email::set_from() method does set the sender's info. @thebrandonallen outlines how the class method works pretty darn well :)

Closing for the moment, but if you encounter any bugs, @henry.wright, do let us know.

[r-a-y]

Maybe you are referring to how BP sets the Reply-To email header with the site admin's email address:
​https://buddypress.trac.wordpress.org/browser/tags/2.5.2/src/bp-core/classes/class-bp-email.php?marks=177-178#L154

I do find this a little weird as well.

Reopening for feedback.

[DJPaul]

I chatted with Henry about this some time ago and I recall it was a misunderstanding with the function name.

@r-a-y The reply-to email address should be a valid address. Technically, the FROM address should also be valid, but since WordPress makes up a fake email address, setting the reply-to at least will route responses to the right place.

(During 2.5 dev, I tried experimenting with setting FROM in this way, and that quickly revealed a lot of compatibility problems, so I left that out.)