Role/Capability: Revert the newly added `update_role` function for 6.1.
Based on feedback, this enhancement isn't quite ready. Reverting [54213] for now to continue the work in the next cycle. Follow-up to [54213]. Props manfcarlo, peterwilsoncc, SergeyBiryukov. Reviewed by SergeyBiryukov. Reverts [54213] in the 6.1 branch. See #54572. Built from https://develop.svn.wordpress.org/branches/6.1@54683 git-svn-id: http://core.svn.wordpress.org/branches/6.1@54235 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
a505398b95
commit
0e34d61860
|
@ -1030,30 +1030,6 @@ function add_role( $role, $display_name, $capabilities = array() ) {
|
|||
return wp_roles()->add_role( $role, $display_name, $capabilities );
|
||||
}
|
||||
|
||||
/**
|
||||
* Updates an existing role. Creates a new role if it doesn't exist.
|
||||
*
|
||||
* Modifies the display name and/or capabilities for an existing role.
|
||||
* If the role does not exist then a new role is created.
|
||||
*
|
||||
* The capabilities are defined in the following format: `array( 'read' => true )`.
|
||||
* To explicitly deny the role a capability, set the value for that capability to false.
|
||||
*
|
||||
* @since 6.1.0
|
||||
*
|
||||
* @param string $role Role name.
|
||||
* @param string|null $display_name Optional. Role display name. If null, the display name
|
||||
* is not modified. Default null.
|
||||
* @param bool[]|null $capabilities Optional. List of capabilities keyed by the capability name,
|
||||
* e.g. `array( 'edit_posts' => true, 'delete_posts' => false )`.
|
||||
* If null, don't alter capabilities for the existing role and make
|
||||
* empty capabilities for the new one. Default null.
|
||||
* @return WP_Role|void WP_Role object, if the role is updated.
|
||||
*/
|
||||
function update_role( $role, $display_name = null, $capabilities = null ) {
|
||||
return wp_roles()->update_role( $role, $display_name, $capabilities );
|
||||
}
|
||||
|
||||
/**
|
||||
* Removes a role, if it exists.
|
||||
*
|
||||
|
|
|
@ -172,76 +172,6 @@ class WP_Roles {
|
|||
return $this->role_objects[ $role ];
|
||||
}
|
||||
|
||||
/**
|
||||
* Updates an existing role. Creates a new role if it doesn't exist.
|
||||
*
|
||||
* Modifies the display name and/or capabilities for an existing role.
|
||||
* If the role does not exist then a new role is created.
|
||||
*
|
||||
* The capabilities are defined in the following format: `array( 'read' => true )`.
|
||||
* To explicitly deny the role a capability, set the value for that capability to false.
|
||||
*
|
||||
* @since 6.1.0
|
||||
*
|
||||
* @param string $role Role name.
|
||||
* @param string|null $display_name Optional. Role display name. If null, the display name
|
||||
* is not modified. Default null.
|
||||
* @param bool[]|null $capabilities Optional. List of capabilities keyed by the capability name,
|
||||
* e.g. `array( 'edit_posts' => true, 'delete_posts' => false )`.
|
||||
* If null, don't alter capabilities for the existing role and make
|
||||
* empty capabilities for the new one. Default null.
|
||||
* @return WP_Role|void WP_Role object, if the role is updated.
|
||||
*/
|
||||
public function update_role( $role, $display_name = null, $capabilities = null ) {
|
||||
if ( ! is_string( $role ) || '' === trim( $role ) ) {
|
||||
return;
|
||||
}
|
||||
|
||||
if ( null !== $display_name && ( ! is_string( $display_name ) || '' === trim( $display_name ) ) ) {
|
||||
return;
|
||||
}
|
||||
|
||||
if ( null !== $capabilities && ! is_array( $capabilities ) ) {
|
||||
return;
|
||||
}
|
||||
|
||||
if ( null === $display_name && null === $capabilities ) {
|
||||
if ( isset( $this->role_objects[ $role ] ) ) {
|
||||
return $this->role_objects[ $role ];
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
if ( null === $display_name ) {
|
||||
if ( ! isset( $this->role_objects[ $role ] ) ) {
|
||||
return;
|
||||
}
|
||||
|
||||
$display_name = $this->roles[ $role ]['name'];
|
||||
}
|
||||
|
||||
if ( null === $capabilities ) {
|
||||
if ( isset( $this->role_objects[ $role ] ) ) {
|
||||
$capabilities = $this->role_objects[ $role ]->capabilities;
|
||||
} else {
|
||||
$capabilities = array();
|
||||
}
|
||||
}
|
||||
|
||||
if ( isset( $this->roles[ $role ] ) ) {
|
||||
if ( null === $capabilities ) {
|
||||
$capabilities = $this->role_objects[ $role ]->capabilities;
|
||||
}
|
||||
|
||||
unset( $this->role_objects[ $role ] );
|
||||
unset( $this->role_names[ $role ] );
|
||||
unset( $this->roles[ $role ] );
|
||||
}
|
||||
|
||||
// The roles database option will be updated in ::add_role().
|
||||
return $this->add_role( $role, $display_name, $capabilities );
|
||||
}
|
||||
|
||||
/**
|
||||
* Removes a role by name.
|
||||
*
|
||||
|
|
|
@ -16,7 +16,7 @@
|
|||
*
|
||||
* @global string $wp_version
|
||||
*/
|
||||
$wp_version = '6.1-RC2-54682';
|
||||
$wp_version = '6.1-RC2-54683';
|
||||
|
||||
/**
|
||||
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
||||
|
|
Loading…
Reference in New Issue