Changeset 13180 for trunk/src/bp-members/bp-members-activity.php

Timestamp:

12/11/2021 07:40:10 PM (3 years ago)

Author:

imath

Message:

Remove a new_avatar activity when an avatar was deleted from history

Once the user removed an item from their avatars story, the corresponding new_avatar also needs to be deleted.

Props vapvarun, oztaser

See #8581

File:

• trunk/src/bp-members/bp-members-activity.php (modified) (1 diff)

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

@@ -251 +251 @@
 }
 add_action( 'bp_members_avatar_uploaded', 'bp_members_new_avatar_activity', 10, 4 );
+
+/**
+ * Remove the `new_avatar` activity corresponding to the deleted previous avatar.
+ *
+ * @since 10.0.0
+ *
+ * @param int $user_id   The user ID.
+ * @param int $timestamp The timestamp when the activity was created.
+ * @return bool True on success. False otherwise.
+ */
+function bp_members_remove_previous_avatar_activity( $user_id = 0, $timestamp = 0 ) {
+    if ( ! $user_id || ! $timestamp || ! bp_is_active( 'activity' ) ) {
+        return false;
+    }
+
+    // Look for a `new_avatar` activity corresponding to the date and user.
+    $activity_id = BP_Activity_Activity::get_id(
+        array(
+            'user_id'       => $user_id,
+            'component'     => buddypress()->members->id,
+            'type'          => 'new_avatar',
+            'date_recorded' => date( 'Y-m-d H:i:s', $timestamp ),
+        )
+    );
+
+    if ( $activity_id ) {
+        return bp_activity_delete(
+            array(
+                'id' => $activity_id,
+            )
+        );
+    }
+
+    return false;
+}
+add_action( 'bp_previous_user_avatar_deleted', 'bp_members_remove_previous_avatar_activity', 10, 2 );