Changeset 7086 for trunk/src/includes/core/actions.php

Timestamp:

05/28/2020 03:38:54 PM (5 years ago)

Author:

johnjamesjacoby

Message:

Signups: Ensure that the dynamic role exists before setting it.

This commit introduces several new helper functions for validating Forum roles before saving & assigning them to new user accounts.

It also adds relevant capability checks to prevent unauthorized users from performing role assignments.

In trunk, for 2.7.0.

See #3157.

File:

• trunk/src/includes/core/actions.php (modified) (1 diff)

trunk/src/includes/core/actions.php

@@ -408 +408 @@
 add_action( 'clean_post_cache', 'bbp_clean_post_cache', 10, 2 );
 
-// User Creation
+// User Registration
+add_action( 'added_existing_user', 'bbp_user_add_role_on_register', 10, 1 );
+add_action( 'bbp_user_register',   'bbp_user_add_role_on_register', 10, 1 );
+
+// Invite a New User
 add_action( 'invite_user',         'bbp_user_add_role_on_invite',   10, 3 );
+
+// Multisite Activation (does not work in wp-activate.php)
 add_action( 'wpmu_activate_user',  'bbp_user_add_role_on_activate', 10, 3 );
-add_action( 'bbp_user_register',   'bbp_user_add_role_on_register', 10, 1 );
-add_action( 'added_existing_user', 'bbp_user_add_role_on_register', 10, 1 );
-add_action( 'register_new_user',   'bbp_user_add_role_on_register', 10, 1 );
 
 /**