Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
03/31/2023 05:26:39 PM (13 months ago)
Author:
imath
Message:

Make sure all displayed user URLs are built using the BP Rewrites API

Replace all remaining bp_displayed_user_domain() usage in favor of
bp_displayed_user_url().
Introduce the bp_members_get_path_chunks() function to quickly build BP
Rewrites argument for member's URL using an array of slugs.
Deprecate bp_activities_member_rss_link(), bp_blogs_blog_tabs() &
bp_groups_header_tabs().
Improve bp_displayed_user_link() so that it's possible to pass an array
of slugs to output an escaped BP Rewrites ready URL.

Props r-a-y, johnjamesjacoby, boonebgorges

Closes https://github.com/buddypress/buddypress/pull/79
See #4954

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-settings/actions/notifications.php

    r13090 r13443  
    5454    do_action( 'bp_core_notification_settings_after_save' );
    5555
    56     bp_core_redirect( bp_displayed_user_domain() . bp_get_settings_slug() . '/notifications/' );
     56    $settings_slug = bp_get_settings_slug();
     57    $path_chunks   = array(
     58        'single_item_component' => bp_rewrites_get_slug( 'members', 'member_' . $settings_slug, $settings_slug ),
     59        'single_item_action'    => bp_rewrites_get_slug( 'members', 'member_' . $settings_slug . '_notifications', 'notifications' ),
     60    );
     61
     62    bp_core_redirect( bp_displayed_user_url( $path_chunks ) );
    5763}
    5864add_action( 'bp_actions', 'bp_settings_action_notifications' );
Note: See TracChangeset for help on using the changeset viewer.