Changeset 13177 for trunk/src/bp-members/screens/change-avatar.php

Timestamp:

12/11/2021 01:40:24 PM (3 years ago)

Author:

imath

Message:

Make sure the new_avatar activity can use the avatar crop results

Since [13175] the BP_Attachment_Avatar::crop() method include the timestamp the avatar was generated on into its returned array. This array is now transported into the action the function creating a new_avatar activity hooks to. This function (bp_members_new_avatar_activity()) now accepts 3 more parameters included the transported crop results. This makes it possible to use the timestamp when the avatar was generated on as the new_avatar activity recorded time. Doing so, we don't need to add an extra activity meta to store the name of the avatar file to be sure to display the avatar the user had at the time the activity was created.

Props vapvarun, oztaser

See #8581

File:

• trunk/src/bp-members/screens/change-avatar.php (modified) (2 diffs)

trunk/src/bp-members/screens/change-avatar.php

@@ -62 +62 @@
         );
 
-        if ( ! bp_core_avatar_handle_crop( $args ) ) {
+        // Handle crop.
+        $cropped_avatar = bp_core_avatar_handle_crop( $r, 'array' );
+
+        if ( ! $cropped_avatar ) {
             bp_core_add_message( __( 'There was a problem cropping your profile photo.', 'buddypress' ), 'error' );
         } else {
@@ -73 +76 @@
              *
              * @since 6.0.0
+             * @since 10.0.0 Adds a new param: an array containing the full, thumb avatar and the timestamp.
              *
-             * @param string $item_id Inform about the user id the avatar was set for.
-             * @param string $value   Inform about the way the avatar was set ('crop').
+             * @param string $item_id        Inform about the user id the avatar was set for.
+             * @param string $type           Inform about the way the avatar was set ('camera').
+             * @param array  $args           Array of parameters passed to the crop handler.
+             * @param array  $cropped_avatar Array containing the full, thumb avatar and the timestamp.
              */
-            do_action( 'bp_members_avatar_uploaded', (int) $args['item_id'], 'crop' );
+            do_action( 'bp_members_avatar_uploaded', (int) $args['item_id'], 'crop', $args, $cropped_avatar );
 
             bp_core_add_message( __( 'Your new profile photo was uploaded successfully.', 'buddypress' ) );