Changeset 34636 for trunk/src/wp-admin/users.php

Timestamp:

09/27/2015 07:10:08 PM (10 years ago)

Author:

wonderboymusic

Message:

Users List Table: show the "Change role to" dropdown on the top and bottom. Currently only shows on top.

See #27743.

File:

• trunk/src/wp-admin/users.php (modified) (3 diffs)

trunk/src/wp-admin/users.php

@@ -96 +96 @@
 
     $editable_roles = get_editable_roles();
-    if ( empty( $editable_roles[$_REQUEST['new_role']] ) )
-        wp_die(__('You can’t give users that role.'));
+    $role = false;
+    if ( ! empty( $_REQUEST['new_role2'] ) ) {
+        $role = $_REQUEST['new_role2'];
+    } elseif ( ! empty( $_REQUEST['new_role'] ) ) {
+        $role = $_REQUEST['new_role'];
+    }
+
+    if ( ! $role || empty( $editable_roles[ $role ] ) ) {
+        wp_die( __( 'You can’t give users that role.' ) );
+    }
 
     $userids = $_REQUEST['users'];
@@ -107 +115 @@
             wp_die(__('You can’t edit that user.'));
         // The new role of the current user must also have the promote_users cap or be a multisite super admin
-        if ( $id == $current_user->ID && ! $wp_roles->role_objects[ $_REQUEST['new_role'] ]->has_cap('promote_users')
+        if ( $id == $current_user->ID && ! $wp_roles->role_objects[ $role ]->has_cap('promote_users')
             && ! ( is_multisite() && is_super_admin() ) ) {
                 $update = 'err_admin_role';
@@ -123 +131 @@
 
         $user = get_userdata( $id );
-        $user->set_role($_REQUEST['new_role']);
+        $user->set_role( $role );
     }