diff --git a/admin/core/um-admin-users.php b/admin/core/um-admin-users.php index 47535993..229aa8cf 100644 --- a/admin/core/um-admin-users.php +++ b/admin/core/um-admin-users.php @@ -159,9 +159,9 @@ class UM_Admin_Users { global $ultimatemember; $admin_err = 0; - + if (isset($_REQUEST) && !empty ($_REQUEST) ){ - + // bulk change role if (isset($_REQUEST['users']) && is_array($_REQUEST['users']) && isset($_REQUEST['um_changeit']) && $_REQUEST['um_changeit'] != '' && isset($_REQUEST['um_change_role']) && !empty($_REQUEST['um_change_role']) ){ @@ -171,7 +171,7 @@ class UM_Admin_Users { check_admin_referer('bulk-users'); $users = $_REQUEST['users']; - $new_role = $_REQUEST['um_change_role']; + $new_role = current( array_filter( $_REQUEST['um_change_role'] ) ); foreach($users as $user_id){ $ultimatemember->user->set( $user_id ); @@ -207,9 +207,9 @@ class UM_Admin_Users { check_admin_referer('bulk-users'); $users = $_REQUEST['users']; - $bulk_action = $_REQUEST['um_bulk_action']; - - if ( $bulk_action == 'um_delete' ) { // this needs confirmation + $bulk_action = current( array_filter( $_REQUEST['um_bulk_action']) ); + + if ( in_array('um_delete', $bulk_action ) > -1 ) { // this needs confirmation $uri = admin_url('users.php'); $userids = array_map( 'intval', (array) $_REQUEST['users'] ); @@ -254,8 +254,9 @@ class UM_Admin_Users { } // filter by user role - if ( isset($_REQUEST['um_filter_role']) && ! isset( $_REQUEST['new_role'] ) && $_REQUEST['um_filter_role'] ) { - exit( wp_redirect( admin_url('users.php?um_role=' . $_REQUEST['um_filter_role'] ) ) ); + if ( isset($_REQUEST['um_filter_role']) && ( ! isset( $_REQUEST['new_role'] ) || empty( $_REQUEST['new_role'] ) ) && $_REQUEST['um_filter_role'] ) { + $filter_role = current( array_filter( $_REQUEST['um_filter_role'] ) ); + exit( wp_redirect( admin_url('users.php?um_role=' .$filter_role ) ) ); } } @@ -271,7 +272,7 @@ class UM_Admin_Users {
- query->get_roles(); @@ -287,7 +288,7 @@ class UM_Admin_Users {
- user->get_bulk_admin_actions(); ?> @@ -299,7 +300,7 @@ class UM_Admin_Users {
- query->get_roles( $add_default = 'Community role…' ) as $key => $value) { ?>