Changeset 14070 for trunk/tests/phpunit/testcases/activity/template.php

Timestamp:

11/03/2024 06:19:06 PM (5 months ago)

Author:

espellcaste

Message:

A user is no longer de-authenticated when making REST API requests.

We are introducing a new BP_LoggedIn_User class to fetch data about a BuddyPress logged-in user. This new addition fixes an issue where a user could be de-authenticated when making REST API requests.

Props dcavins, DJPaul, johnjamesjacoby, and imath.

Closes ​https://github.com/buddypress/buddypress/pull/395
See #9229 and #9145
Fixes #7658

File:

• trunk/tests/phpunit/testcases/activity/template.php (modified) (10 diffs)

trunk/tests/phpunit/testcases/activity/template.php

@@ -12 +12 @@
         $u             = self::factory()->user->create();
         $original_user = bp_loggedin_user_id();
-        self::set_current_user( $u );
+        wp_set_current_user( $u );
 
         $a = self::factory()->activity->create(
@@ -26 +26 @@
 
         // Logged-out user can't delete
-        self::set_current_user( 0 );
+        wp_set_current_user( 0 );
         $this->assertFalse( bp_activity_user_can_delete( $activity ) );
 
         // Miscellaneous user can't delete
         $misc_user = self::factory()->user->create();
-        self::set_current_user( $misc_user );
+        wp_set_current_user( $misc_user );
         $this->assertFalse( bp_activity_user_can_delete( $activity ) );
 
@@ -45 +45 @@
         $bp->is_single_item = $is_single_item;
         $bp->is_item_admin  = $is_item_admin;
-        self::set_current_user( $original_user );
+        wp_set_current_user( $original_user );
     }
 
@@ -55 +55 @@
         $old_user = get_current_user_id();
         $u        = self::factory()->user->create();
-        self::set_current_user( $u );
+        wp_set_current_user( $u );
 
         // create an activity update for the user
@@ -75 +75 @@
 
         // reset
-        self::set_current_user( $old_user );
+        wp_set_current_user( $old_user );
     }
 
@@ -105 +105 @@
 
         $current_user = bp_loggedin_user_id();
-        self::set_current_user( $user_id );
+        wp_set_current_user( $user_id );
 
         bp_activity_add_user_favorite( $a1, $user_id );
         bp_activity_add_user_favorite( $a2, $user_id );
 
-        self::set_current_user( $current_user );
+        wp_set_current_user( $current_user );
 
         // groan. It sucks that you have to invoke the global
@@ -159 +159 @@
         // save the current user and override logged-in user
         $old_user = get_current_user_id();
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         $now = time();
@@ -209 +209 @@
         // clean up!
         $activities_template = null;
-        self::set_current_user( $old_user );
+        wp_set_current_user( $old_user );
     }
 
@@ -1152 +1152 @@
         $u3 = self::factory()->user->create();
 
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         $g = self::factory()->group->create(
@@ -1212 +1212 @@
         $u3 = self::factory()->user->create();
 
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
 
         $g = self::factory()->group->create(