Changeset 9326 for trunk/src/bp-friends/bp-friends-activity.php

Timestamp:

01/09/2015 08:56:45 AM (12 years ago)

Author:

imath

Message:

Only create a single activity when a friendship is created and make sure to display it on each involved user personal activities.

When a friendship is created, a single public activity will be created. Thanks to the new BP_Activity_Query class introduced in version 2.2, we are able to fetch this activity on each involved user personal activity stream.
Once BuddyPress will be updated to 2.2, all hide_sitewide activities related to a created friendship will be removed as they are not necessary anymore.

Fixes #6040

props boonebgorges, r-a-y

File:

• trunk/src/bp-friends/bp-friends-activity.php (modified) (2 diffs)

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

@@ -272 +272 @@
 
 /**
+ * Set up activity arguments for use with the 'just-me' scope.
+ *
+ * For details on the syntax, see {@link BP_Activity_Query}.
+ *
+ * @since BuddyPress (2.2.0)
+ *
+ * @param array $retval Empty array by default
+ * @param array $filter Current activity arguments
+ * @return array
+ */
+function bp_friends_filter_activity_just_me_scope( $retval, $filter ) {
+        // Get the requested action
+        $action = $filter['filter']['action'];
+
+        // Make sure actions are listed in an array
+        if ( ! is_array( $action ) ) {
+                $action = explode( ',', $filter['filter']['action'] );
+        }
+
+        $action = array_flip( array_filter( $action ) );
+
+        /**
+         * If filtering activities for something other than the friendship_created action
+         * return without changing anything
+         */
+        if ( ! empty( $action ) && ! isset( $action['friendship_created'] ) ) {
+                return $retval;
+        }
+
+        /**
+         * Else make sure to get the friendship_created action, the user is involved in
+         * - user initiated the friendship
+         * - user has been requested a friendship
+         */
+        return array(
+                'relation' => 'OR',
+                array(
+                        'column' => 'user_id',
+                        'value'  => $filter['user_id']
+                ),
+                array(
+                        'relation' => 'AND',
+                        array(
+                                'column' => 'component',
+                                'value'  => 'friends',
+                        ),
+                        array(
+                                'column' => 'secondary_item_id',
+                                'value'  => $filter['user_id'],
+                        ),
+                ),
+                'override' => array(
+                        'display_comments' => 'stream',
+                        'filter'           => array( 'user_id' => 0 ),
+                ),
+        );
+}
+add_filter( 'bp_activity_set_just-me_scope_args', 'bp_friends_filter_activity_just_me_scope', 20, 2 );
+
+/**
  * Add activity stream items when one members accepts another members request
  * for virtual friendship.
@@ -294 +354 @@
                 'secondary_item_id' => $friend_user_id
         ) );
-
-        // Record in activity streams for the friend
-        friends_record_activity( array(
-                'user_id'           => $friend_user_id,
-                'type'              => 'friendship_created',
-                'item_id'           => $friendship_id,
-                'secondary_item_id' => $initiator_user_id,
-                'hide_sitewide'     => true // We've already got the first entry site wide
-        ) );
 }
 add_action( 'friends_friendship_accepted', 'bp_friends_friendship_accepted_activity', 10, 4 );