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/groups/class-bp-group-extension.php

    r13980 r14070  
    241241        $this->go_to( bp_get_group_url( $g_obj ) );
    242242
    243         self::set_current_user( 0 );
     243        wp_set_current_user( 0 );
    244244
    245245        $e = new BPTest_Group_Extension_Inferred_Access_Settings_EnableNavItem_True();
    … …  
    251251        $this->assertFalse( $e2->user_can_visit() );
    252252
    253         self::set_current_user( $old_current_user );
     253        wp_set_current_user( $old_current_user );
    254254    }
    255255
    … …  
    260260        $this->set_permalink_structure( '/%postname%/' );
    261261        $old_current_user = get_current_user_id();
    262         self::set_current_user( 0 );
     262        wp_set_current_user( 0 );
    263263
    264264        $g = self::factory()->group->create( array(
    … …  
    292292        $this->assertFalse( $e6->user_can_visit() );
    293293
    294         self::set_current_user( $old_current_user );
     294        wp_set_current_user( $old_current_user );
    295295    }
    296296
    … …  
    307307        $u = self::factory()->user->create();
    308308        $old_current_user = get_current_user_id();
    309         self::set_current_user( $u );
     309        wp_set_current_user( $u );
    310310
    311311        $this->go_to( bp_get_group_url( $g_obj ) );
    … …  
    335335        $this->assertFalse( $e6->user_can_visit() );
    336336
    337         self::set_current_user( $old_current_user );
     337        wp_set_current_user( $old_current_user );
    338338    }
    339339
    … …  
    350350        $u = self::factory()->user->create();
    351351        $old_current_user = get_current_user_id();
    352         self::set_current_user( $u );
     352        wp_set_current_user( $u );
    353353
    354354        $this->add_user_to_group( $u, $g );
    … …  
    380380        $this->assertFalse( $e6->user_can_visit() );
    381381
    382         self::set_current_user( $old_current_user );
     382        wp_set_current_user( $old_current_user );
    383383    }
    384384
    … …  
    395395        $u = self::factory()->user->create();
    396396        $old_current_user = get_current_user_id();
    397         self::set_current_user( $u );
     397        wp_set_current_user( $u );
    398398
    399399        $m = $this->add_user_to_group( $u, $g );
    … …  
    427427        $this->assertFalse( $e6->user_can_visit() );
    428428
    429         self::set_current_user( $old_current_user );
     429        wp_set_current_user( $old_current_user );
    430430    }
    431431
    … …  
    442442        $u = self::factory()->user->create();
    443443        $old_current_user = get_current_user_id();
    444         self::set_current_user( $u );
     444        wp_set_current_user( $u );
    445445
    446446        $m = $this->add_user_to_group( $u, $g );
    … …  
    474474        $this->assertFalse( $e6->user_can_visit() );
    475475
    476         self::set_current_user( $old_current_user );
     476        wp_set_current_user( $old_current_user );
    477477    }
    478478
    … …  
    488488
    489489        $old_current_user = get_current_user_id();
    490         self::set_current_user( 0 );
     490        wp_set_current_user( 0 );
    491491
    492492        $this->go_to( bp_get_group_url( $g_obj ) );
    … …  
    516516        $this->assertFalse( $e6->user_can_visit() );
    517517
    518         self::set_current_user( $old_current_user );
     518        wp_set_current_user( $old_current_user );
    519519    }
    520520
    … …  
    530530
    531531        $old_current_user = get_current_user_id();
    532         self::set_current_user( 0 );
     532        wp_set_current_user( 0 );
    533533
    534534        $this->go_to( bp_get_group_url( $g_obj ) );
    … …  
    558558        $this->assertFalse( $e6->user_can_see_nav_item() );
    559559
    560         self::set_current_user( $old_current_user );
     560        wp_set_current_user( $old_current_user );
    561561    }
    562562
    … …  
    573573        $u = self::factory()->user->create();
    574574        $old_current_user = get_current_user_id();
    575         self::set_current_user( $u );
     575        wp_set_current_user( $u );
    576576
    577577        $this->go_to( bp_get_group_url( $g_obj ) );
    … …  
    601601        $this->assertFalse( $e6->user_can_see_nav_item() );
    602602
    603         self::set_current_user( $old_current_user );
     603        wp_set_current_user( $old_current_user );
    604604    }
    605605
    … …  
    616616        $u = self::factory()->user->create();
    617617        $old_current_user = get_current_user_id();
    618         self::set_current_user( $u );
     618        wp_set_current_user( $u );
    619619
    620620        $this->add_user_to_group( $u, $g );
    … …  
    646646        $this->assertFalse( $e6->user_can_see_nav_item() );
    647647
    648         self::set_current_user( $old_current_user );
     648        wp_set_current_user( $old_current_user );
    649649    }
    650650
    … …  
    661661        $u = self::factory()->user->create();
    662662        $old_current_user = get_current_user_id();
    663         self::set_current_user( $u );
     663        wp_set_current_user( $u );
    664664
    665665        $this->add_user_to_group( $u, $g );
    … …  
    693693        $this->assertFalse( $e6->user_can_see_nav_item() );
    694694
    695         self::set_current_user( $old_current_user );
     695        wp_set_current_user( $old_current_user );
    696696    }
    697697
    … …  
    708708        $u = self::factory()->user->create();
    709709        $old_current_user = get_current_user_id();
    710         self::set_current_user( $u );
     710        wp_set_current_user( $u );
    711711
    712712        $this->add_user_to_group( $u, $g );
    … …  
    740740        $this->assertFalse( $e6->user_can_see_nav_item() );
    741741
    742         self::set_current_user( $old_current_user );
     742        wp_set_current_user( $old_current_user );
    743743    }
    744744
Note: See TracChangeset for help on using the changeset viewer.