Ticket #8123: 8123.patch

src/bp-members/bp-members-functions.php

@@ -609 +609 @@
         return apply_filters( 'bp_core_get_active_member_count', $count );
 }
 
+/**
+ * Update the spam status of the member on multisite configs.
+ *
+ * @since 5.0.0
+ *
+ * @param int   $user_id The user ID to spam or ham.
+ * @param int   $value   0 to mark the user as `ham`, 1 to mark as `spam`.
+ * @return bool          True if the spam status of the member changed.
+ *                       False otherwise.
+ */
+function bp_core_update_member_status( $user_id = 0, $value = 0 ) {
+        if ( ! is_multisite() || ! $user_id ) {
+                return false;
+        }
+
+        if ( ! function_exists( 'wp_get_registered_image_subsizes' ) ) {
+                return update_user_status( $user_id, 'spam', $value );
+        }
+
+        $user = wp_update_user( array(
+                'ID'   => $user_id,
+                'spam' => $value,
+        ) );
+
+        if ( is_wp_error( $user ) ) {
+                return false;
+        }
+
+        return true;
+}
+
 /**
  * Process a spammed or unspammed user.
  *
@@ -671 +702 @@
                 }
 
                 // Finally, mark this user as a spammer.
-                if ( is_multisite() ) {
-                        update_user_status( $user_id, 'spam', $is_spam );
-                }
+                bp_core_update_member_status( $user_id, $is_spam );
         }
 
         // Update the user status.

tests/phpunit/testcases/blogs/functions.php

@@ -291 +291 @@
                         $this->setExpectedDeprecated( 'wpmu_new_blog' );
                 }
 
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 // Create a regular member
                 $u = self::factory()->user->create();
 

tests/phpunit/testcases/core/class-bp-user-query.php

@@ -389 +389 @@
          * @group spam
          */
         public function test_bp_user_query_type_alphabetical_spam_xprofileoff() {
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 $u1 = self::factory()->user->create();
                 $u2 = self::factory()->user->create();
 

tests/phpunit/testcases/members/functions.php

@@ -378 +378 @@
                         $this->markTestSkipped();
                 }
 
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 $bp = buddypress();
                 $displayed_user = $bp->displayed_user;
 
@@ -389 +385 @@
                 $bp->displayed_user->id = $u1;
 
                 // Bulk spam in network admin uses update_user_status
-                update_user_status( $u1, 'spam', '1' );
+                bp_core_update_member_status( $u1, '1' );
 
                 $this->assertTrue( bp_is_user_spammer( $u1 ) );
 
@@ -410 +406 @@
                         $this->markTestSkipped();
                 }
 
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 $bp = buddypress();
                 $displayed_user = $bp->displayed_user;
 
@@ -426 +418 @@
                 $this->assertTrue( bp_is_user_spammer( $u1 ) );
 
                 // Bulk unspam in network admin uses update_user_status
-                update_user_status( $u1, 'spam', '0' );
+                bp_core_update_member_status( $u1, '0' );
 
                 $this->assertFalse( bp_is_user_spammer( $u1 ) );
 
@@ -438 +430 @@
          * @group bp_core_process_spammer_status
          */
         public function test_bp_core_process_spammer_status_make_spam_user_filter() {
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 add_filter( 'make_spam_user', array( $this, 'notification_filter_callback' ) );
 
                 $u1 = self::factory()->user->create();
@@ -454 +442 @@
         }
 
         public function test_bp_core_process_spammer_status_make_ham_user_filter() {
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
+                $u1 = self::factory()->user->create();
+                $s  = bp_core_process_spammer_status( $u1, 'spam' );
 
                 add_filter( 'make_ham_user', array( $this, 'notification_filter_callback' ) );
 
-                $u1 = self::factory()->user->create();
-                $n = bp_core_process_spammer_status( $u1, 'ham' );
+                $h = bp_core_process_spammer_status( $u1, 'ham' );
 
                 remove_filter( 'make_ham_user', array( $this, 'notification_filter_callback' ) );
 
@@ -470 +456 @@
         }
 
         public function test_bp_core_process_spammer_status_bp_make_spam_user_filter() {
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 add_filter( 'bp_make_spam_user', array( $this, 'notification_filter_callback' ) );
 
                 $u1 = self::factory()->user->create();
@@ -486 +468 @@
         }
 
         public function test_bp_core_process_spammer_status_bp_make_ham_user_filter() {
-                if ( is_multisite() && function_exists( 'wp_get_registered_image_subsizes' ) ) {
-                        $this->setExpectedDeprecated( 'update_user_status' );
-                }
-
                 add_filter( 'bp_make_ham_user', array( $this, 'notification_filter_callback' ) );
 
                 $u1 = self::factory()->user->create();