Changeset 11829

Timestamp:

01/27/2018 04:39:38 PM (9 years ago)

Author:

r-a-y

Message:

Activation: Add default WordPress role for new signups.

Props r-a-y, needle, espellcaste, DJPaul, boonebgorges.

Fixes #7565.

Location:

trunk

Files:

• src/bp-members/bp-members-functions.php (modified) (2 diffs)
• tests/phpunit/testcases/members/functions.php (modified) (1 diff)

trunk/src/bp-members/bp-members-functions.php

@@ -1858 +1858 @@
                         bp_delete_user_meta( $user_id, 'activation_key' );
 
-                        $member = get_userdata( $user_id );
-                        $member->set_role( get_option('default_role') );
-
                         $user_already_created = true;
 
@@ -1983 +1980 @@
 
 /**
+ * Add default WordPress role for new signups on the BP root blog.
+ *
+ * @since 3.0.0
+ *
+ * @param int $user_id The user ID to add the default role for.
+ */
+function bp_members_add_role_after_activation( $user_id ) {
+        // Get default role to add.
+        $role = bp_get_option( 'default_role' );
+
+        // Multisite.
+        if ( is_multisite() && ! is_user_member_of_blog( $user_id, bp_get_root_blog_id() ) ) {
+                add_user_to_blog( bp_get_root_blog_id(), $user_id, $role );
+
+        // Single-site.
+        } elseif ( ! is_multisite() ) {
+                $member = get_userdata( $user_id );
+                $member->set_role( $role );
+        }
+}
+add_action( 'bp_core_activated_user', 'bp_members_add_role_after_activation', 1 );
+
+/**
  * Migrate signups from pre-2.0 configuration to wp_signups.
  *

trunk/tests/phpunit/testcases/members/functions.php

@@ -576 +576 @@
                 );
         }
+
+        /**
+         * @group bp_core_activate_signup
+         */
+        public function test_bp_core_activate_signup_should_add_user_role() {
+                $key = 'test';
+
+                // Create the signup.
+                $this->factory->signup->create( array(
+                        'user_login'     => 'test',
+                        'user_email'     => 'test@example.com',
+                        'activation_key' => $key,
+                        'meta' => array(
+                                'field_1' => 'Foo Bar',
+                                'password' => 'foobar',
+                        ),
+                ) );
+
+                // Activate user.
+                $user_id = bp_core_activate_signup( $key );
+
+                // Assert that user has a role.
+                $user = get_userdata( $user_id );
+                $this->assertNotEmpty( $user->roles );
+        }
 }