[buddypress-trac] [BuddyPress Trac] #8081: BuddyPress breaks disabling notification for new users

[buddypress-trac]

#8081: BuddyPress breaks disabling notification for new users
-----------------------------------------+---------------------
 Reporter:  zodiac1978                   |       Owner:  (none)
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  5.0.0
Component:  Members                      |     Version:  1.2.2
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch reporter-feedback  |
-----------------------------------------+---------------------
Changes (by imath):

 * keywords:  good-first-bug => has-patch reporter-feedback
 * version:   => 1.2.2
 * milestone:  Awaiting Contributions => 5.0.0


Comment:

 Hi @zodiac1978

 My understanding is that you want BuddyPress to take in account the 3rd
 parameter of `wp_new_user_notification()`: the one WordPress 4.6
 introduced.

 This means that by default, the notification should be sent to both and
 that the filter can now be used to define who should receive the
 notification (user, admin, both or no one - using `false`).

 Is this correct ? See attached patch.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8081#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac