Changeset 13443 for trunk/src/bp-messages/actions/delete.php

Timestamp:

03/31/2023 05:26:39 PM (2 years 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-messages/actions/delete.php (modified) (2 diffs)

trunk/src/bp-messages/actions/delete.php

@@ -19 +19 @@
     }
 
-    $thread_id = bp_action_variable( 1 );
+    $thread_id   = bp_action_variable( 1 );
+    $path_chunks = bp_members_get_path_chunks( array( bp_get_messages_slug(), bp_current_action() ) );
+    $redirect    = bp_displayed_user_url( $path_chunks );
 
     if ( ! $thread_id || ! is_numeric( $thread_id ) || ! messages_check_thread_access( $thread_id ) ) {
-        bp_core_redirect( trailingslashit( bp_displayed_user_domain() . bp_get_messages_slug() . '/' . bp_current_action() ) );
+        bp_core_redirect( $redirect );
     } else {
         if ( ! check_admin_referer( 'messages_delete_thread' ) ) {
@@ -34 +36 @@
             bp_core_add_message( __('Message deleted.', 'buddypress') );
         }
-        bp_core_redirect( trailingslashit( bp_displayed_user_domain() . bp_get_messages_slug() . '/' . bp_current_action() ) );
+        bp_core_redirect( $redirect );
     }
 }