Changeset 14070 for trunk/tests/phpunit/testcases/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.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/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.php (modified) (12 diffs)

trunk/tests/phpunit/testcases/core/nav/bpCoreMaybeHookNewSubnavScreenFunction.php

@@ -49 +49 @@
     public function test_user_has_access_false_user_logged_out() {
         $old_current_user = get_current_user_id();
-        self::set_current_user( 0 );
+        wp_set_current_user( 0 );
 
         $subnav_item = array(
@@ -62 +62 @@
         $this->assertSame( $expected, bp_core_maybe_hook_new_subnav_screen_function( $subnav_item ) );
 
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -68 +68 @@
         $u = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u );
+        wp_set_current_user( $u );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -82 +82 @@
         $this->assertSame( bp_members_get_user_url( $u ), $found['redirect_args']['root'] );
 
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -89 +89 @@
         $u2 = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -113 +113 @@
 
         // Clean up
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
         buddypress()->default_component = $old_default_component;
         buddypress()->bp_nav = $old_bp_nav;
@@ -125 +125 @@
         $u2 = self::factory()->user->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u1 );
+        wp_set_current_user( $u1 );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -150 +150 @@
 
         // Clean up
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
         buddypress()->default_component = $old_default_component;
         buddypress()->bp_nav = $old_bp_nav;
@@ -170 +170 @@
         $g = self::factory()->group->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u );
+        wp_set_current_user( $u );
         $this->set_permalink_structure( '/%postname%/' );
 
@@ -188 +188 @@
 
         // Clean up
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 
@@ -195 +195 @@
         $g = self::factory()->group->create();
         $old_current_user = get_current_user_id();
-        self::set_current_user( $u );
+        wp_set_current_user( $u );
 
         $group = groups_get_group( $g );
@@ -211 +211 @@
 
         // Clean up
-        self::set_current_user( $old_current_user );
+        wp_set_current_user( $old_current_user );
     }
 }