Changeset 14026 for trunk/src/bp-messages/classes/class-bp-messages-component.php

Timestamp:

09/27/2024 09:11:27 PM (2 years ago)

Author:

espellcaste

Message:

Include the V2 of the BP REST API in BuddyPress core.

We are officially deprecating the V1 of the BP REST API. And bundling the new, default, V2 of the BP REST API inside BuddyPress core. Previously, the V1 was developed as a plugin in a separate repo (​https://github.com/buddypress/BP-REST).

• One of the main differences between the V1 and V2 is how objects are returned. Single items are no longer returned as an array;
• We have a new BP_Test_REST_Controller_Testcase for testing the new API endpoints;
• We changed the names of our controllers to follow our autoloader rules;
• Removed the BP-REST plugin from wp-env and from our release script;
• And we added notices for the deprecated V1 API (endpoints and files).

Props imath & I. ;)

Fixes #8200
See #9145
Closes ​https://github.com/buddypress/buddypress/pull/337

File:

• trunk/src/bp-messages/classes/class-bp-messages-component.php (modified) (8 diffs)

trunk/src/bp-messages/classes/class-bp-messages-component.php

@@ -42 +42 @@
                         array(
                                 'adminbar_myaccount_order' => 50,
-                                'features'                 => array( 'star' )
+                                'features'                 => array( 'star' ),
                         )
                 );
@@ -178 +178 @@
                 // All globals for messaging component.
                 // Note that global_tables is included in this array.
-                parent::setup_globals( array(
-                        'slug'                  => $default_slug,
-                        'has_directory'         => false,
-                        'notification_callback' => 'messages_format_notifications',
-                        'search_string'         => __( 'Search Messages...', 'buddypress' ),
-                        'global_tables'         => $global_tables,
-                        'meta_tables'           => $meta_tables
-                ) );
+                parent::setup_globals(
+                        array(
+                                'slug'                  => $default_slug,
+                                'has_directory'         => false,
+                                'notification_callback' => 'messages_format_notifications',
+                                'search_string'         => __( 'Search Messages...', 'buddypress' ),
+                                'global_tables'         => $global_tables,
+                                'meta_tables'           => $meta_tables,
+                        )
+                );
         }
 
@@ -224 +226 @@
                 if ( bp_is_active( $this->id, 'star' ) ) {
                         $sub_nav[] = array(
-                                'name'                      => __( 'Starred', 'buddypress' ),
+                                'name'                     => __( 'Starred', 'buddypress' ),
                                 'slug'                     => bp_get_messages_starred_slug(),
                                 'parent_slug'              => $slug,
@@ -325 +327 @@
                         // Unread message count.
                         $count = messages_get_unread_count( bp_loggedin_user_id() );
-                        if ( !empty( $count ) ) {
+                        if ( ! empty( $count ) ) {
                                 $title = sprintf(
                                         /* translators: %s: Unread message count for the current user */
@@ -338 +340 @@
                         } else {
                                 $title = __( 'Messages', 'buddypress' );
-                                $inbox = __( 'Inbox',    'buddypress' );
+                                $inbox = __( 'Inbox', 'buddypress' );
                         }
 
@@ -413 +415 @@
                                 $bp->bp_options_title = __( 'My Messages', 'buddypress' );
                         } else {
-                                $bp->bp_options_avatar = bp_core_fetch_avatar( array(
-                                        'item_id' => bp_displayed_user_id(),
-                                        'type'    => 'thumb',
-                                        /* translators: %s: member name */
-                                        'alt'     => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_displayed_user_fullname() )
-                                ) );
-                                $bp->bp_options_title = bp_get_displayed_user_fullname();
+                                $bp->bp_options_avatar = bp_core_fetch_avatar(
+                                        array(
+                                                'item_id' => bp_displayed_user_id(),
+                                                'type'    => 'thumb',
+                                                /* translators: %s: member name */
+                                                'alt'     => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_displayed_user_fullname() ),
+                                        )
+                                );
+                                $bp->bp_options_title  = bp_get_displayed_user_fullname();
                         }
                 }
@@ -434 +438 @@
 
                 // Global groups.
-                wp_cache_add_global_groups( array(
-                        'bp_messages',
-                        'bp_messages_threads',
-                        'bp_messages_unread_count',
-                        'message_meta'
-                ) );
+                wp_cache_add_global_groups(
+                        array(
+                                'bp_messages',
+                                'bp_messages_threads',
+                                'bp_messages_unread_count',
+                                'message_meta',
+                        )
+                );
 
                 parent::setup_cache_groups();
@@ -455 +461 @@
                 parent::rest_api_init(
                         array(
-                                'BP_REST_Messages_Endpoint',
-                                'BP_REST_Sitewide_Notices_Endpoint',
+                                'BP_Messages_REST_Controller',
+                                'BP_Messages_Sitewide_Notices_REST_Controller',
                         )
                 );