Changeset 13442 for trunk/src/bp-messages/actions/compose.php

Timestamp:

03/27/2023 06:19:06 PM (21 months ago)

Author:

imath

Message:

Improve Components Member's single item navigation generation

• Edit the BP_Component Class so that it globalize navigation items before registering it.
• Introduce bp_get_component_navigations(), a new function that will be used to get Member's single navigation customizable slugs within the BuddyPress settings area.
• Perform all remaining bp_loggedin_user_domain() replacements (55) in favor of the bp_loggedin_user_url() function which uses BP Rewrites to build URLs.
• Improve bp_loggedin_user_link() by adding a new $chunks array of arguments to output escaped URL in templates.
• Adapt some PHPUnit testcases.

Props r-a-y, johnjamesjacoby, boonebgorges

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

File:

• trunk/src/bp-messages/actions/compose.php (modified) (3 diffs)

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

@@ -44 +44 @@
 
         // Setup the link to the logged-in user's messages.
-        $member_messages = trailingslashit( bp_loggedin_user_domain() . bp_get_messages_slug() );
+        $message_slug        = bp_get_messages_slug();
+        $custom_message_slug = bp_rewrites_get_slug( 'members', 'member_' . $message_slug, $message_slug );
+        $path_chunks         = array(
+            'single_item_component' => $custom_message_slug,
+        );
 
         // Site-wide notice.
@@ -51 +55 @@
             // Attempt to save the notice and redirect to notices.
             if ( messages_send_notice( $_POST['subject'], $_POST['content'] ) ) {
-                $success     = true;
-                $feedback    = __( 'Notice successfully created.', 'buddypress' );
-                $redirect_to = trailingslashit( $member_messages . 'notices' );
+                $success                           = true;
+                $feedback                          = __( 'Notice successfully created.', 'buddypress' );
+                $path_chunks['single_item_action'] = bp_rewrites_get_slug( 'members', 'member_' . $message_slug . '_notices', 'notices' );
+                $redirect_to = bp_loggedin_user_url( $path_chunks );
 
             // Notice could not be sent.
@@ -88 +93 @@
             // Send the message and redirect to it.
             if ( true === is_int( $send ) ) {
-                $success     = true;
-                $feedback    = __( 'Message successfully sent.', 'buddypress' );
-                $view        = trailingslashit( $member_messages . 'view' );
-                $redirect_to = trailingslashit( $view . $send );
+                $success                                     = true;
+                $feedback                                    = __( 'Message successfully sent.', 'buddypress' );
+                $path_chunks['single_item_action']           = bp_rewrites_get_slug( 'members', 'member_' . $message_slug . '_view', 'view' );
+                $path_chunks['single_item_action_variables'] = array( $send );
+                $redirect_to                                 = bp_loggedin_user_url( $path_chunks );
 
             // Message could not be sent.