Skip to:
Content

BuddyPress.org


Ignore:
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:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/core/invitations.php

    r14026 r14070  
    113113        $u2 = self::factory()->user->create();
    114114        $u3 = self::factory()->user->create();
    115         self::set_current_user( $u1 );
     115        wp_set_current_user( $u1 );
    116116
    117117        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    135135        $this->assertEqualSets( array( $i1, $i2 ), $invites );
    136136
    137         self::set_current_user( $old_current_user );
     137        wp_set_current_user( $old_current_user );
    138138    }
    139139
    … …  
    143143        $u1 = self::factory()->user->create();
    144144        $u2 = self::factory()->user->create();
    145         self::set_current_user( $u1 );
     145        wp_set_current_user( $u1 );
    146146
    147147        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    159159        $this->assertEquals( $i1, $i2 );
    160160
    161         self::set_current_user( $old_current_user );
     161        wp_set_current_user( $old_current_user );
    162162    }
    163163
    … …  
    169169        $u3 = self::factory()->user->create();
    170170
    171         self::set_current_user( $u1 );
     171        wp_set_current_user( $u1 );
    172172
    173173        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    204204        $this->assertEqualSets( array( $i1 ), $invite_ids );
    205205
    206         self::set_current_user( $old_current_user );
     206        wp_set_current_user( $old_current_user );
    207207    }
    208208
    … …  
    213213        $u2 = self::factory()->user->create();
    214214        $u3 = self::factory()->user->create();
    215         self::set_current_user( $u1 );
     215        wp_set_current_user( $u1 );
    216216
    217217        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    240240        $this->assertEqualSets( array(), wp_list_pluck( $invites, 'id' ) );
    241241
    242         self::set_current_user( $old_current_user );
     242        wp_set_current_user( $old_current_user );
    243243    }
    244244
    … …  
    249249        $u2 = self::factory()->user->create();
    250250        $u3 = self::factory()->user->create();
    251         self::set_current_user( $u1 );
     251        wp_set_current_user( $u1 );
    252252
    253253        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    281281        $this->assertEqualSets( array( $i1, $r1 ), $invites );
    282282
    283         self::set_current_user( $old_current_user );
     283        wp_set_current_user( $old_current_user );
    284284    }
    285285
    … …  
    288288
    289289        $u1 = self::factory()->user->create();
    290         self::set_current_user( $u1 );
     290        wp_set_current_user( $u1 );
    291291
    292292        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    308308        $this->assertEqualSets( array( $r1, $r2 ), $requests );
    309309
    310         self::set_current_user( $old_current_user );
     310        wp_set_current_user( $old_current_user );
    311311    }
    312312
    … …  
    317317
    318318        $u1 = self::factory()->user->create();
    319         self::set_current_user( $u1 );
     319        wp_set_current_user( $u1 );
    320320
    321321        // Create a couple of requests.
    … …  
    328328        $this->assertFalse( $invites_class->add_request( $request_args ) );
    329329
    330         self::set_current_user( $old_current_user );
     330        wp_set_current_user( $old_current_user );
    331331    }
    332332
    … …  
    336336        $u1 = self::factory()->user->create();
    337337        $u2 = self::factory()->user->create();
    338         self::set_current_user( $u1 );
     338        wp_set_current_user( $u1 );
    339339
    340340        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    366366        $this->assertEqualSets( array( $r1, $i1 ), $invites );
    367367
    368         self::set_current_user( $old_current_user );
     368        wp_set_current_user( $old_current_user );
    369369    }
    370370
    … …  
    374374        $u1 = self::factory()->user->create();
    375375        $u2 = self::factory()->user->create();
    376         self::set_current_user( $u1 );
     376        wp_set_current_user( $u1 );
    377377
    378378        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    394394        $this->assertEquals( 1, $invite->invite_sent );
    395395
    396         self::set_current_user( $old_current_user );
     396        wp_set_current_user( $old_current_user );
    397397    }
    398398
    … …  
    402402        $u1 = self::factory()->user->create();
    403403        $u2 = self::factory()->user->create();
    404         self::set_current_user( $u1 );
     404        wp_set_current_user( $u1 );
    405405
    406406        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    439439        $this->assertEqualSets( array( $i2 ), $invites );
    440440
    441         self::set_current_user( $old_current_user );
     441        wp_set_current_user( $old_current_user );
    442442    }
    443443
    … …  
    446446
    447447        $u1 = self::factory()->user->create();
    448         self::set_current_user( $u1 );
     448        wp_set_current_user( $u1 );
    449449
    450450        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    461461        $this->assertEquals( $time, $req->date_modified );
    462462
    463         self::set_current_user( $old_current_user );
     463        wp_set_current_user( $old_current_user );
    464464    }
    465465
    … …  
    469469        $u1 = self::factory()->user->create();
    470470        $u2 = self::factory()->user->create();
    471         self::set_current_user( $u1 );
     471        wp_set_current_user( $u1 );
    472472
    473473        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    487487        $this->assertEquals( $time, $inv->date_modified );
    488488
    489         self::set_current_user( $old_current_user );
     489        wp_set_current_user( $old_current_user );
    490490    }
    491491
    … …  
    496496        $u2 = self::factory()->user->create();
    497497        $u3 = self::factory()->user->create();
    498         self::set_current_user( $u1 );
     498        wp_set_current_user( $u1 );
    499499
    500500        $invites_class = new BPTest_Invitation_Manager_Extension();
    … …  
    537537        $this->assertEquals( array( $i3, $i1, $i2 ), $invites );
    538538
    539         self::set_current_user( $old_current_user );
     539        wp_set_current_user( $old_current_user );
    540540    }
    541541}
Note: See TracChangeset for help on using the changeset viewer.