[buddypress-trac] [BuddyPress] #4218: Disabling activation email with bp_core_signup_send_activation_key breaks the signup

[buddypress-trac at lists.automattic.com]

#4218: Disabling activation email with bp_core_signup_send_activation_key breaks
the signup
--------------------------+------------------------------
 Reporter:  boonebgorges  |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Core          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by DJPaul):

 At that point, user_status = 2 has already been set, and
 bp_core_signup_disable_inactive() will prevent such users to log in. As
 far as I see, those users *will* need a key to activate their account (I
 have no idea how they'd find out what their key is), so I think moving
 generation of the activation key outside of the conditional is safe.

 This seems a bit of a weird filter, and we probably want to add true 'skip
 user account activation' in a future version.

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