Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
05/29/2023 05:51:12 AM (11 months ago)
Author:
imath
Message:

Make sure BP_Component::parse_query() always run

Only running it when BP Rewrites are on is not the right way to go as some plugins might have been using the WP Rewrites API for a while and might have extended this method from their component's class.

Instead of only running this method if BP Rewrites are on, we're adding a URL parser check inside each component having a directory to figure out whether it's needed to set BP URI globals.

See #4954
Fixes #8908
Closes https://github.com/buddypress/buddypress/pull/109

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-members/classes/class-bp-members-component.php

    r13481 r13492  
    777777     */
    778778    public function parse_query( $query ) {
     779        /*
     780         * If BP Rewrites are not in use, no need to parse BP URI globals another time.
     781         * Legacy Parser should have already set these.
     782         */
     783        if ( 'rewrites' !== bp_core_get_query_parser() ) {
     784            return parent::parse_query( $query );
     785        }
     786
    779787        // Init the current member and member type.
    780788        $member      = false;
Note: See TracChangeset for help on using the changeset viewer.