[buddypress-trac] [BuddyPress] #4689: bp_core_email_from_name_filter() is too aggressive

buddypress-trac noreply at wordpress.org
Wed Nov 28 20:52:49 UTC 2012


#4689: bp_core_email_from_name_filter() is too aggressive
--------------------------+--------------------
 Reporter:  r-a-y         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  1.7
Component:  Core          |     Version:  1.2.2
 Severity:  normal        |  Resolution:
 Keywords:  dev-feedback  |
--------------------------+--------------------

Comment (by r-a-y):

 I knew there was another ticket for this, but couldn't find it!  Thanks
 for finding it, Paul.

 I just looked into this briefly and it looks like we can't deprecate the
 function.

 If we don't set the 'From' name, it will default to "WordPress":
 https://core.trac.wordpress.org/browser/tags/3.4.2/wp-
 includes/pluggable.php#L317

 I think Boone's approach of creating a wrapper `bp_mail()` function is
 more flexible since we would only have to add the mail headers in one
 place instead of for each `wp_mail()` call like I originally suggested.

 If we're all in favor (favour for Paul!), I'll create a patch.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4689#comment:3>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list