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. Built from https://develop.svn.wordpress.org/trunk@54673 git-svn-id: http://core.svn.wordpress.org/trunk@54225 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
89e665383f
commit
0f9b449466
|
@ -1030,30 +1030,6 @@ function add_role( $role, $display_name, $capabilities = array() ) {
|
||||||
return wp_roles()->add_role( $role, $display_name, $capabilities );
|
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.
|
* Removes a role, if it exists.
|
||||||
*
|
*
|
||||||
|
|
|
@ -172,76 +172,6 @@ class WP_Roles {
|
||||||
return $this->role_objects[ $role ];
|
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.
|
* Removes a role by name.
|
||||||
*
|
*
|
||||||
|
|
|
@ -16,7 +16,7 @@
|
||||||
*
|
*
|
||||||
* @global string $wp_version
|
* @global string $wp_version
|
||||||
*/
|
*/
|
||||||
$wp_version = '6.2-alpha-54672';
|
$wp_version = '6.2-alpha-54673';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.
|
||||||
|
|
Loading…
Reference in New Issue