* fixed #1554;

* updated changelog.txt;
* updated readme.txt;
* clean comments;

changelog.txt

@@ -1,6 +1,6 @@
 == Changelog ==
 
-= 2.8.7 September 30, 2024 =
+= 2.8.7 October 01, 2024 =
 
 * Enhancements:
 
@@ -48,6 +48,7 @@
   - Hook: Action hook `um_admin_user_action_hook`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
   - Hook: Action hook `um_admin_user_action_{$bulk_action}_hook`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
   - Hook: Action hook `um_admin_custom_hook_{$action}`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
+  - Hook: Filter hook `um_admin_views_users`. Use 'um_user_statuses_admin_filter_options' hook instead.
   - Function: `UM()->user()->set_status( $status )`. Use function `UM()->common()->users()->set_status( $status, $user_id )` instead.
   - Function: `UM()->user()->assign_secretkey()`. Use function `UM()->common()->users()->assign_secretkey( $user_id )` instead.
   - Function: `UM()->user()->approve( $repeat )`. Use function `UM()->common()->users()->approve( $user_id, $force )` instead.

includes/admin/class-secure.php

@@ -150,7 +150,6 @@ if ( ! class_exists( 'um\admin\Secure' ) ) {
 				}
 				// Restore Account Status.
 				if ( isset( $metadata['account_status'] ) ) {
-					// UM()->user()->set_status( $metadata['account_status'] );
 					UM()->common()->users()->set_status( $user_id, $metadata['account_status'] );
 				}
 

includes/class-init.php

@@ -1451,7 +1451,6 @@ if ( ! class_exists( 'UM' ) ) {
 			require_once 'core/um-actions-form.php';
 			require_once 'core/um-actions-access.php';
 			require_once 'core/um-actions-wpadmin.php';
-//			require_once 'core/um-actions-core.php';
 			require_once 'core/um-actions-ajax.php';
 			require_once 'core/um-actions-login.php';
 			require_once 'core/um-actions-register.php';

includes/common/class-secure.php

@@ -229,10 +229,8 @@ if ( ! class_exists( 'um\common\Secure' ) ) {
 			$user->update_user_level_from_caps();
 
 			if ( is_user_logged_in() ) {
-				//UM()->user()->set_status( 'inactive' );
 				UM()->common()->users()->set_status( $user->ID, 'inactive' );
 			} else {
-				//UM()->user()->set_status( 'rejected' );
 				UM()->common()->users()->set_status( $user->ID, 'rejected' );
 			}
 			um_reset_user();

includes/core/rest/class-api-v1.php

@@ -239,7 +239,6 @@ if ( ! class_exists( 'um\core\rest\API_v1' ) ) {
 
 			switch ( $data ) {
 				case 'status':
-					//UM()->user()->set_status( $value );
 					UM()->common()->users()->set_status( $id, $value );
 					$response['success'] = __( 'User status has been changed.', 'ultimate-member' );
 					break;

includes/core/rest/class-api-v2.php

@@ -221,7 +221,6 @@ if ( ! class_exists( 'um\core\rest\API_v2' ) ) {
 
 			switch ( $data ) {
 				case 'status':
-					// UM()->user()->set_status( $value );
 					UM()->common()->users()->set_status( $id, $value );
 					$response['success'] = __( 'User status has been changed.', 'ultimate-member' );
 					break;

includes/core/um-actions-register.php

@@ -64,7 +64,7 @@ function um_after_insert_user( $user_id, $args, $form_data = null ) {
 		um_fetch_user( $user_id );
 		$status = um_user( 'status' );
 	}
-	// UM()->user()->set_status( $status );
+
 	UM()->common()->users()->set_status( $user_id, $status );
 
 	// Create user uploads directory.

includes/um-deprecated-hooks.php

@@ -95,3 +95,17 @@ if ( ! defined( 'ABSPATH' ) ) {
  *
  * @param {int} $user_id User ID.
  */
+
+/**
+ * Filters the WP Users list table views.
+ *
+ * Fully deprecated. Please use 'um_user_statuses_admin_filter_options' hook instead since 2.8.7.
+ *
+ * @param {array} $views List table filter views.
+ *
+ * @return {array} List table filter views.
+ *
+ * @since 1.3.x
+ * @depecated 2.8.7 Fully deprecated because there is used dropdown with statuses instead of list table views.
+ * @hook um_admin_views_users
+ */

readme.txt

@@ -166,7 +166,7 @@ No specific extensions are needed. But we highly recommended keep active these P
 
 IMPORTANT: PLEASE UPDATE THE PLUGIN TO AT LEAST VERSION 2.6.7 IMMEDIATELY. VERSION 2.6.7 PATCHES SECURITY PRIVILEGE ESCALATION VULNERABILITY. PLEASE SEE [THIS ARTICLE](https://docs.ultimatemember.com/article/1866-security-incident-update-and-recommended-actions) FOR MORE INFORMATION
 
-= 2.8.7 2024-09-30 =
+= 2.8.7 2024-10-01 =
 
 **Enhancements**
 
@@ -214,6 +214,7 @@ IMPORTANT: PLEASE UPDATE THE PLUGIN TO AT LEAST VERSION 2.6.7 IMMEDIATELY. VERSI
 * Hook: Action hook `um_admin_user_action_hook`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
 * Hook: Action hook `um_admin_user_action_{$bulk_action}_hook`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
 * Hook: Action hook `um_admin_custom_hook_{$action}`. Use filter hook `um_handle_bulk_actions-users-{$current_action}` for custom user bulk actions instead.
+* Hook: Filter hook `um_admin_views_users`. Use 'um_user_statuses_admin_filter_options' hook instead.
 * Function: `UM()->user()->set_status( $status )`. Use function `UM()->common()->users()->set_status( $status, $user_id )` instead.
 * Function: `UM()->user()->assign_secretkey()`. Use function `UM()->common()->users()->assign_secretkey( $user_id )` instead.
 * Function: `UM()->user()->approve( $repeat )`. Use function `UM()->common()->users()->approve( $user_id, $force )` instead.