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

Timestamp:

09/27/2024 09:11:27 PM (15 months 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-notifications/classes/class-bp-notifications-component.php (modified) (5 diffs)

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

@@ -146 +146 @@
      */
     public function register_nav( $main_nav = array(), $sub_nav = array() ) {
-        $slug   = bp_get_notifications_slug();
+        $slug = bp_get_notifications_slug();
 
         // Add 'Notifications' to the main navigation.
@@ -248 +248 @@
                 );
             } else {
-                $title  = _x( 'Notifications', 'My Account Notification',         'buddypress' );
-                $unread = _x( 'Unread',        'My Account Notification sub nav', 'buddypress' );
+                $title  = _x( 'Notifications', 'My Account Notification', 'buddypress' );
+                $unread = _x( 'Unread', 'My Account Notification sub nav', 'buddypress' );
             }
 
@@ -297 +297 @@
             } else {
                 $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_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() ),
+                    )
+                );
             }
         }
@@ -317 +319 @@
 
         // Global groups.
-        wp_cache_add_global_groups( array(
-            'bp_notifications',
-            'notification_meta',
-            'bp_notifications_unread_count',
-            'bp_notifications_grouped_notifications',
-        ) );
+        wp_cache_add_global_groups(
+            array(
+                'bp_notifications',
+                'notification_meta',
+                'bp_notifications_unread_count',
+                'bp_notifications_grouped_notifications',
+            )
+        );
 
         parent::setup_cache_groups();
@@ -336 +340 @@
      */
     public function rest_api_init( $controllers = array() ) {
-        parent::rest_api_init( array( 'BP_REST_Notifications_Endpoint' ) );
+        parent::rest_api_init( array( 'BP_Notifications_REST_Controller' ) );
     }
 }