Changeset 12851

Timestamp:

02/21/2021 08:10:55 PM (5 years ago)

Author:

imath

Message:

The new_avatar activity belongs to the Members component

The activity generated when members add/update their profile photo wasn't migrated to the Members component during the 6.0.0 milestone. This commit's goal is to repare this oversight.

See #8156
See #8407
Fixes #8408

Location:

trunk

Files:

• src/bp-core/bp-core-update.php (modified) (2 diffs)
• src/bp-core/deprecated/8.0.php (added)
• src/bp-members/bp-members-activity.php (modified) (3 diffs)
• src/bp-xprofile/bp-xprofile-activity.php (modified) (3 diffs)
• src/class-buddypress.php (modified) (1 diff)
• tests/phpunit/testcases/members/activity.php (modified) (1 diff)
• tests/phpunit/testcases/xprofile/activity.php (modified) (1 diff)

trunk/src/bp-core/bp-core-update.php

@@ -274 +274 @@
                         bp_update_to_5_0();
                 }
+
+                // Version 8.0.0.
+                if ( $raw_db_version < 12850 ) {
+                        bp_update_to_8_0();
+                }
         }
 
@@ -591 +596 @@
 
 /**
+ * 8.0.0 update routine.
+ *
+ * - Edit the `new_avatar` activity type's component to `members`.
+ *
+ * @since 8.0.0
+ */
+function bp_update_to_8_0() {
+        global $wpdb;
+        $bp_prefix = bp_core_get_table_prefix();
+
+        // Update the `new_avatar` activity type's component to `members`.
+        $wpdb->update(
+                $bp_prefix . 'bp_activity',
+                array(
+                        'component' => 'members',
+                ),
+                array(
+                        'type' => 'new_avatar',
+                ),
+                array(
+                        '%s',
+                ),
+                array(
+                        '%s',
+                )
+        );
+}
+
+/**
  * Updates the component field for new_members type.
  *

trunk/src/bp-members/bp-members-activity.php

@@ -26 +26 @@
                 __( 'New Members', 'buddypress' ),
                 array( 'activity' )
+        );
+
+        // Register the activity stream actions for this component.
+        bp_activity_set_action(
+                // Older avatar activity items use 'profile' for component. See r4273.
+                buddypress()->members->id,
+                'new_avatar',
+                __( 'Member changed profile picture', 'buddypress' ),
+                'bp_members_format_activity_action_new_avatar',
+                __( 'Updated Profile Photos', 'buddypress' )
         );
 
@@ -69 +79 @@
 
 /**
+ * Format 'new_avatar' activity actions.
+ *
+ * @since 8.0.0
+ *
+ * @param string $action   Static activity action.
+ * @param object $activity Activity object.
+ * @return string
+ */
+function bp_members_format_activity_action_new_avatar( $action, $activity ) {
+        $userlink = bp_core_get_userlink( $activity->user_id );
+
+        /* translators: %s: user link */
+        $action = sprintf( esc_html__( '%s changed their profile picture', 'buddypress' ), $userlink );
+
+        // Legacy filter - pass $user_id instead of $activity.
+        if ( has_filter( 'bp_xprofile_new_avatar_action' ) ) {
+                $action = apply_filters( 'bp_xprofile_new_avatar_action', $action, $activity->user_id );
+        }
+
+        /** This filter is documented in wp-includes/deprecated.php */
+        $action = apply_filters_deprecated( 'bp_xprofile_format_activity_action_new_avatar', array( $action, $activity ), '8.0.0', 'bp_members_format_activity_action_new_avatar' );
+
+        /**
+         * Filters the formatted 'new_avatar' activity stream action.
+         *
+         * @since 8.0.0
+         *
+         * @param string $action   Formatted action for activity stream.
+         * @param object $activity Activity object.
+         */
+        return apply_filters( 'bp_members_format_activity_action_new_avatar', $action, $activity );
+}
+
+/**
  * Create a "became a registered user" activity item when a user activates his account.
  *
@@ -98 +142 @@
 }
 add_action( 'bp_core_activated_user', 'bp_core_new_user_activity' );
+
+/**
+ * Adds an activity stream item when a user has uploaded a new avatar.
+ *
+ * @since 8.0.0
+ *
+ * @param int $user_id The user id the avatar was set for.
+ */
+function bp_members_new_avatar_activity( $user_id = 0 ) {
+
+        // Bail if activity component is not active.
+        if ( ! bp_is_active( 'activity' ) ) {
+                return false;
+        }
+
+        if ( empty( $user_id ) ) {
+                $user_id = bp_displayed_user_id();
+        }
+
+        /** This filter is documented in wp-includes/deprecated.php */
+        $user_id = apply_filters_deprecated( 'bp_xprofile_new_avatar_user_id', array( $user_id ), '8.0.0', 'bp_members_new_avatar_user_id' );
+
+        /**
+         * Filters the user ID when a user has uploaded a new avatar.
+         *
+         * @since 8.0.0
+         *
+         * @param int $user_id ID of the user the avatar was set for.
+         */
+        $user_id = apply_filters( 'bp_members_new_avatar_user_id', $user_id );
+
+        // Add the activity.
+        bp_activity_add( array(
+                'user_id'   => $user_id,
+                'component' => buddypress()->members->id,
+                'type'      => 'new_avatar'
+        ) );
+}
+add_action( 'bp_members_avatar_uploaded', 'bp_members_new_avatar_activity' );

trunk/src/bp-xprofile/bp-xprofile-activity.php

@@ -21 +21 @@
  */
 function xprofile_register_activity_actions() {
-
-        // Register the activity stream actions for this component.
-        bp_activity_set_action(
-                // Older avatar activity items use 'profile' for component. See r4273.
-                'profile',
-                'new_avatar',
-                __( 'Member changed profile picture', 'buddypress' ),
-                'bp_xprofile_format_activity_action_new_avatar',
-                __( 'Updated Profile Photos', 'buddypress' )
-        );
 
         bp_activity_set_action(
@@ -49 +39 @@
 }
 add_action( 'bp_register_activity_actions', 'xprofile_register_activity_actions' );
-
-/**
- * Format 'new_avatar' activity actions.
- *
- * @since 2.0.0
- *
- * @param string $action   Static activity action.
- * @param object $activity Activity object.
- * @return string
- */
-function bp_xprofile_format_activity_action_new_avatar( $action, $activity ) {
-        $userlink = bp_core_get_userlink( $activity->user_id );
-
-        /* translators: %s: user link */
-        $action = sprintf( esc_html__( '%s changed their profile picture', 'buddypress' ), $userlink );
-
-        // Legacy filter - pass $user_id instead of $activity.
-        if ( has_filter( 'bp_xprofile_new_avatar_action' ) ) {
-                $action = apply_filters( 'bp_xprofile_new_avatar_action', $action, $activity->user_id );
-        }
-
-        /**
-         * Filters the formatted 'new_avatar' activity stream action.
-         *
-         * @since 2.0.0
-         *
-         * @param string $action   Formatted action for activity stream.
-         * @param object $activity Activity object.
-         */
-        return apply_filters( 'bp_xprofile_format_activity_action_new_avatar', $action, $activity );
-}
 
 /**
@@ -201 +160 @@
 
 /**
- * Adds an activity stream item when a user has uploaded a new avatar.
- *
- * @since 1.0.0
- * @since 2.3.4 Add new parameter to get the user id the avatar was set for.
- *
- *                         specific activity
- *
- * @param int $user_id The user id the avatar was set for.
- * @return bool
- */
-function bp_xprofile_new_avatar_activity( $user_id = 0 ) {
-
-        // Bail if activity component is not active.
-        if ( ! bp_is_active( 'activity' ) ) {
-                return false;
-        }
-
-        if ( empty( $user_id ) ) {
-                $user_id = bp_displayed_user_id();
-        }
-
-        /**
-         * Filters the user ID when a user has uploaded a new avatar.
-         *
-         * @since 1.5.0
-         *
-         * @param int $user_id ID of the user the avatar was set for.
-         */
-        $user_id = apply_filters( 'bp_xprofile_new_avatar_user_id', $user_id );
-
-        // Add the activity.
-        bp_activity_add( array(
-                'user_id'   => $user_id,
-                'component' => 'profile',
-                'type'      => 'new_avatar'
-        ) );
-}
-add_action( 'bp_members_avatar_uploaded', 'bp_xprofile_new_avatar_activity' );
-
-/**
  * Add an activity item when a user has updated his profile.
  *

trunk/src/class-buddypress.php

@@ -305 +305 @@
 
                 $this->version    = '8.0.0-alpha';
-                $this->db_version = 12385;
+                $this->db_version = 12850;
 
                 /** Loading ***********************************************************/

trunk/tests/phpunit/testcases/members/activity.php

@@ -20 +20 @@
 
                 $expected = sprintf( __( '%s became a registered member', 'buddypress' ), bp_core_get_userlink( $u ) );
+
+                $a_obj = new BP_Activity_Activity( $a );
+
+                $this->assertSame( $expected, $a_obj->action );
+        }
+
+        /**
+         * @group activity_action
+         * @group bp_members_format_activity_action_new_avatar
+         */
+        public function test_bp_members_format_activity_action_new_avatar() {
+                $u = self::factory()->user->create();
+                $a = self::factory()->activity->create( array(
+                        'component' => 'members',
+                        'type' => 'new_avatar',
+                        'user_id' => $u,
+                ) );
+
+                $expected = sprintf( __( '%s changed their profile picture', 'buddypress' ), bp_core_get_userlink( $u ) );
 
                 $a_obj = new BP_Activity_Activity( $a );

trunk/tests/phpunit/testcases/xprofile/activity.php

@@ -297 +297 @@
         /**
          * @group activity_action
-         * @group bp_xprofile_format_activity_action_new_avatar
-         */
-        public function test_bp_xprofile_format_activity_action_new_avatar() {
-                $u = self::factory()->user->create();
-                $a = self::factory()->activity->create( array(
-                        'component' => 'profile',
-                        'type' => 'new_avatar',
-                        'user_id' => $u,
-                ) );
-
-                $expected = sprintf( __( '%s changed their profile picture', 'buddypress' ), bp_core_get_userlink( $u ) );
-
-                $a_obj = new BP_Activity_Activity( $a );
-
-                $this->assertSame( $expected, $a_obj->action );
-        }
-
-        /**
-         * @group activity_action
          * @group bp_xprofile_format_activity_action_updated_profile
          */