Changeset 13443 for trunk/src/bp-settings/actions/notifications.php

Timestamp:

03/31/2023 05:26:39 PM (18 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:

• trunk/src/bp-settings/actions/notifications.php (modified) (1 diff)

trunk/src/bp-settings/actions/notifications.php

@@ -54 +54 @@
     do_action( 'bp_core_notification_settings_after_save' );
 
-    bp_core_redirect( bp_displayed_user_domain() . bp_get_settings_slug() . '/notifications/' );
+    $settings_slug = bp_get_settings_slug();
+    $path_chunks   = array(
+        'single_item_component' => bp_rewrites_get_slug( 'members', 'member_' . $settings_slug, $settings_slug ),
+        'single_item_action'    => bp_rewrites_get_slug( 'members', 'member_' . $settings_slug . '_notifications', 'notifications' ),
+    );
+
+    bp_core_redirect( bp_displayed_user_url( $path_chunks ) );
 }
 add_action( 'bp_actions', 'bp_settings_action_notifications' );