Changeset 8125 for trunk/bp-friends/bp-friends-activity.php

Timestamp:

03/13/2014 06:34:06 PM (11 years ago)

Author:

boonebgorges

Message:

Generate activity actions dynamically, instead of using static values stored in the database

Activity actions - like 'Joe became a registered member' - have always been
stored as static strings in the activity table. This has been a perennial thorn
in the side of BP site administrators. For one thing, static strings cannot be
translated for multilingual audiences (without unreasonable workarounds).
For another, information in these static strings becomes out of date as users
change their display names, groups change their names, and so on.

This changeset makes it possible for activity types to be registered with a
new $format_callback function, passed to bp_activity_set_action(), which BP
will use to generate the action dynamically when building the activity loop.
This makes the activity stream fully localizable, continually up-to-date, and
generally the bomb dot com.

Existing plugins that do not register format_callback functions will continue
to have their actions pulled directly from the database. Likewise, since a copy
of the static string is saved in the activity table, static strings can be
served if the given component is disabled at any point (and thus the callback
is no longer available).

The original argument for caching the action strings was that generating them
inline was too resource intensive; it often requires pulling up display names,
user permalinks, group links, blog post names, and so forth. To address this
problem, each component can now prefetch required data at the beginning of an
activity loop by hooking to bp_activity_prefetch_object_data, and loading any
relevant data into the cache. The Activity, Friends, Groups, and Blogs
component now do this natively. It's likely that this prefetching will have
other performance benefits, as plugin authors will now be able to access user
data etc inline without performance penalties.

The case of the Blogs component is a special one; it's not practical to
prefetch data from multiple blogs at the beginning of a loop, due to the
resources required by switch_to_blog(). For this reason, the format_callback
functions in the blog component cache some relevant data (like the post name)
in blogmeta, where it's readily accessible within the loop.

Fixes #3856

File:

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

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

@@ -93 +93 @@
 
     // These two added in BP 1.6
-    bp_activity_set_action( $bp->friends->id, 'friendship_accepted', __( 'Friendships accepted', 'buddypress' ) );
-    bp_activity_set_action( $bp->friends->id, 'friendship_created',  __( 'New friendships',      'buddypress' ) );
+    bp_activity_set_action(
+        $bp->friends->id,
+        'friendship_accepted',
+        __( 'Friendships accepted', 'buddypress' ),
+        'bp_friends_format_activity_action_friendship_accepted'
+    );
+
+    bp_activity_set_action(
+        $bp->friends->id,
+        'friendship_created',
+        __( 'New friendships', 'buddypress' ),
+        'bp_friends_format_activity_action_friendship_created'
+    );
 
     // < BP 1.6 backpat
@@ -102 +113 @@
 }
 add_action( 'bp_register_activity_actions', 'friends_register_activity_actions' );
+
+/**
+ * Format 'friendship_accepted' activity actions.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param object $activity Activity data.
+ * @return string $action Formatted activity action.
+ */
+function bp_friends_format_activity_action_friendship_accepted( $activity ) {
+    $initiator_link = bp_core_get_userlink( $activity->user_id );
+    $friend_link    = bp_core_get_userlink( $activity->secondary_item_id );
+
+    $action = sprintf( __( '%1$s and %2$s are now friends', 'buddypress' ), $initiator_link, $friend_link );
+
+    // Backward compatibility for legacy filter
+    // The old filter has the $friendship object passed to it. We want to
+    // avoid having to build this object if it's not necessary
+    if ( has_filter( 'friends_activity_friendship_accepted_action' ) ) {
+        $friendship = new BP_Friends_Friendship( $activity->item_id );
+        $action     = apply_filters( 'friends_activity_friendsip_accepted_action', $action, $friendship );
+    }
+
+    return apply_filters( 'bp_friends_format_activity_action_friendship_accepted', $action, $activity );
+}
+
+/**
+ * Format 'friendship_created' activity actions.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param object $activity Activity data.
+ * @return string $action Formatted activity action.
+ */
+function bp_friends_format_activity_action_friendship_created( $activity ) {
+    $initiator_link = bp_core_get_userlink( $activity->user_id );
+    $friend_link    = bp_core_get_userlink( $activity->secondary_item_id );
+
+    $action = sprintf( __( '%1$s and %2$s are now friends', 'buddypress' ), $initiator_link, $friend_link );
+
+    // Backward compatibility for legacy filter
+    // The old filter has the $friendship object passed to it. We want to
+    // avoid having to build this object if it's not necessary
+    if ( has_filter( 'friends_activity_friendship_accepted_action' ) ) {
+        $friendship = new BP_Friends_Friendship( $activity->item_id );
+        $action     = apply_filters( 'friends_activity_friendsip_accepted_action', $action, $friendship );
+    }
+
+    return apply_filters( 'bp_friends_format_activity_action_friendship_created', $action, $activity );
+}
+
+/**
+ * Fetch data related to friended users at the beginning of an activity loop.
+ *
+ * This reduces database overhead during the activity loop.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param array $activities Array of activity items.
+ * @return array
+ */
+function bp_friends_prefetch_activity_object_data( $activities ) {
+    if ( empty( $activities ) ) {
+        return $activities;
+    }
+
+    $friend_ids = array();
+
+    foreach ( $activities as $activity ) {
+        if ( buddypress()->friends->id !== $activity->component ) {
+            continue;
+        }
+
+        $friend_ids[] = $activity->secondary_item_id;
+    }
+
+    if ( ! empty( $friend_ids ) ) {
+        // Fire a user query to prime user caches
+        new BP_User_Query( array(
+            'user_ids'          => $friend_ids,
+            'populate_extras'   => false,
+            'update_meta_cache' => false,
+        ) );
+    }
+}
+add_filter( 'bp_activity_prefetch_object_data', 'bp_friends_prefetch_activity_object_data' );
 
 /**
@@ -129 +226 @@
         'user_id'           => $initiator_user_id,
         'type'              => 'friendship_created',
-        'action'            => apply_filters( 'friends_activity_friendship_accepted_action', sprintf( __( '%1$s and %2$s are now friends', 'buddypress' ), $initiator_link, $friend_link ), $friendship ),
         'item_id'           => $friendship_id,
         'secondary_item_id' => $friend_user_id
@@ -138 +234 @@
         'user_id'           => $friend_user_id,
         'type'              => 'friendship_created',
-        'action'            => apply_filters( 'friends_activity_friendship_accepted_action', sprintf( __( '%1$s and %2$s are now friends', 'buddypress' ), $friend_link, $initiator_link ), $friendship ),
         'item_id'           => $friendship_id,
         'secondary_item_id' => $initiator_user_id,