Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
06/18/2023 04:11:04 AM (3 years ago)
Author:
imath
Message:

BP Rewrites: optimize the code used to build URLs

The bp_members_get_path_chunks() & bp_groups_get_path_chunks() functions are making sure URL chunks are customized according to the slugs settings. Instead of redoing at many places all or some of the operations performed by these functions, update the code to build URLs so that it uses these functions.

See #4954
Fixes #8923
Closes ​https://github.com/buddypress/buddypress/pull/117

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-friends/bp-friends-template.php

    r13493 r13503  
    222222
    223223                $friendship_status = bp_is_friend( $potential_friend_id );
    224                 $friends_slug        = bp_get_friends_slug();
    225                 $custom_friends_slug = bp_rewrites_get_slug( 'members', 'member_' . $friends_slug, $friends_slug );
     224                $friends_slug      = bp_get_friends_slug();
    226225
    227226                if ( empty( $friendship_status ) ) {
    … …  
    239238                                        'wrapper_id'        => 'friendship-button-' . $potential_friend_id,
    240239                                        'link_href'         => wp_nonce_url(
    241                                                 bp_loggedin_user_url(
    242                                                         array(
    243                                                                 'single_item_component'        => $custom_friends_slug,
    244                                                                 'single_item_action'           => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests', 'requests' ),
    245                                                                 'single_item_action_variables' => array( bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests_cancel', 'cancel' ), $potential_friend_id ),
    246                                                         )
    247                                                 ),
     240                                                bp_loggedin_user_url( bp_members_get_path_chunks( array( $friends_slug, 'requests', array( 'cancel', $potential_friend_id ) ) ) ),
    248241                                                'friends_withdraw_friendship'
    249242                                        ),
    … …  
    264257                                        'wrapper_class'     => 'friendship-button awaiting_response_friend',
    265258                                        'wrapper_id'        => 'friendship-button-' . $potential_friend_id,
    266                                         'link_href'         => bp_loggedin_user_url(
    267                                                 array(
    268                                                         'single_item_component' => $custom_friends_slug,
    269                                                         'single_item_action'    => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests', 'requests' ),
    270                                                 )
    271                                         ),
     259                                        'link_href'         => bp_loggedin_user_url( bp_members_get_path_chunks( array( $friends_slug, 'requests' ) ) ),
    272260                                        'link_text'         => __( 'Friendship Requested', 'buddypress' ),
    273261                                        'link_title'        => __( 'Friendship Requested', 'buddypress' ),
    … …  
    287275                                        'wrapper_id'        => 'friendship-button-' . $potential_friend_id,
    288276                                        'link_href'         => wp_nonce_url(
    289                                                 bp_loggedin_user_url(
    290                                                         array(
    291                                                                 'single_item_component'        => $custom_friends_slug,
    292                                                                 'single_item_action'           => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_remove_friend', 'remove-friend' ),
    293                                                                 'single_item_action_variables' => array( $potential_friend_id ),
    294                                                         )
    295                                                 ),
     277                                                bp_loggedin_user_url( bp_members_get_path_chunks( array( $friends_slug, 'remove-friend', array( $potential_friend_id ) ) ) ),
    296278                                                'friends_remove_friend'
    297279                                        ),
    … …  
    313295                                        'wrapper_id'        => 'friendship-button-' . $potential_friend_id,
    314296                                        'link_href'         => wp_nonce_url(
    315                                                 bp_loggedin_user_url(
    316                                                         array(
    317                                                                 'single_item_component'        => $custom_friends_slug,
    318                                                                 'single_item_action'           => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_add_friend', 'add-friend' ),
    319                                                                 'single_item_action_variables' => array( $potential_friend_id ),
    320                                                         )
    321                                                 ),
     297                                                bp_loggedin_user_url( bp_members_get_path_chunks( array( $friends_slug, 'add-friend', array( $potential_friend_id ) ) ) ),
    322298                                                'friends_add_friend'
    323299                                        ),
    … …  
    488464                }
    489465
    490                 $friends_slug = bp_get_friends_slug();
    491                 $url          = wp_nonce_url(
    492                         bp_loggedin_user_url(
    493                                 array(
    494                                         'single_item_component'        => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug, $friends_slug ),
    495                                         'single_item_action'           => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests', 'requests' ),
    496                                         'single_item_action_variables' => array( bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests_accept', 'accept' ), $friendship_id ),
    497                                 )
    498                         ),
     466                $url = wp_nonce_url(
     467                        bp_loggedin_user_url( bp_members_get_path_chunks( array( bp_get_friends_slug(), 'requests', array( 'accept', $friendship_id ) ) ) ),
    499468                        'friends_accept_friendship'
    500469                );
    … …  
    537506                }
    538507
    539                 $friends_slug = bp_get_friends_slug();
    540                 $url          = wp_nonce_url(
    541                         bp_loggedin_user_url(
    542                                 array(
    543                                         'single_item_component'        => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug, $friends_slug ),
    544                                         'single_item_action'           => bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests', 'requests' ),
    545                                         'single_item_action_variables' => array( bp_rewrites_get_slug( 'members', 'member_' . $friends_slug . '_requests_reject', 'reject' ), $friendship_id ),
    546                                 )
    547                         ),
     508                $url = wp_nonce_url(
     509                        bp_loggedin_user_url( bp_members_get_path_chunks( array( bp_get_friends_slug(), 'requests', array( 'reject', $friendship_id ) ) ) ),
    548510                        'friends_reject_friendship'
    549511                );
Note: See TracChangeset for help on using the changeset viewer.