Changeset 8125 for trunk/bp-xprofile/bp-xprofile-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-xprofile/bp-xprofile-activity.php (modified) (3 diffs)

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

@@ -18 +18 @@
 
     // Register the activity stream actions for this component
-    bp_activity_set_action( $bp->profile->id, 'new_avatar',      __( 'Member changed profile picture', 'buddypress' ) );
-    bp_activity_set_action( $bp->profile->id, 'new_member',      __( 'New member registered', 'buddypress' ) );
-    bp_activity_set_action( $bp->profile->id, 'updated_profile', __( 'Updated Profile', 'buddypress' ) );
+    bp_activity_set_action(
+        $bp->profile->id,
+        'new_avatar',
+        __( 'Member changed profile picture', 'buddypress' ),
+        'bp_xprofile_format_activity_action_new_avatar'
+    );
+
+    bp_activity_set_action(
+        $bp->profile->id,
+        'new_member',
+        __( 'New member registered', 'buddypress' ),
+        'bp_xprofile_format_activity_action_new_member'
+    );
+
+    bp_activity_set_action(
+        $bp->profile->id,
+        'updated_profile',
+        __( 'Updated Profile', 'buddypress' ),
+        'bp_xprofile_format_activity_action_updated_profile'
+    );
 
     do_action( 'xprofile_register_activity_actions' );
 }
 add_action( 'bp_register_activity_actions', 'xprofile_register_activity_actions' );
+
+/**
+ * Format 'new_avatar' activity actions.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param object $activity Activity object.
+ * @return string
+ */
+function bp_xprofile_format_activity_action_new_avatar( $activity ) {
+    $userlink = bp_core_get_userlink( $activity->user_id );
+    $action   = sprintf( __( '%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 );
+    }
+
+    return apply_filters( 'bp_xprofile_format_activity_action_new_avatar', $action, $activity );
+}
+
+/**
+ * Format 'new_member' activity actions.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param object $activity Activity object.
+ * @return string
+ */
+function bp_xprofile_format_activity_action_new_member( $activity ) {
+    $userlink = bp_core_get_userlink( $activity->user_id );
+    $action   = sprintf( __( '%s became a registered member', 'buddypress' ), $userlink );
+
+    // Legacy filter - pass $user_id instead of $activity
+    if ( has_filter( 'bp_core_activity_registered_member_action' ) ) {
+        $action = apply_filters( 'bp_core_activity_registered_member_action', $action, $activity->user_id );
+    }
+
+    return apply_filters( 'bp_xprofile_format_activity_action_new_member', $action, $activity );
+}
+
+/**
+ * Format 'updated_profile' activity actions.
+ *
+ * @since BuddyPress (2.0.0)
+ *
+ * @param object $activity Activity object.
+ * @return string
+ */
+function bp_xprofile_format_activity_action_updated_profile( $activity ) {
+    // Note for translators: The natural phrasing in English, "Joe updated
+    // his profile", requires that we know Joe's gender, which we don't. If
+    // your language doesn't have this restriction, feel free to use a more
+    // natural translation.
+    $profile_link = trailingslashit( bp_core_get_user_domain( $activity->user_id ) . buddypress()->profile->slug );
+    $action       = sprintf( __( '%1$s&#8217;s profile was updated', 'buddypress' ), '<a href="' . $profile_link . '">' . bp_core_get_user_displayname( $activity->user_id ) . '</a>' );
+
+    return apply_filters( 'bp_xprofile_format_activity_action_updated_profile', $action, $activity );
+}
 
 /**
@@ -127 +203 @@
     bp_activity_add( array(
         'user_id' => $user_id,
-        'action' => apply_filters( 'bp_xprofile_new_avatar_action', sprintf( __( '%s changed their profile picture', 'buddypress' ), $userlink ), $user_id ),
         'component' => 'profile',
         'type' => 'new_avatar'
@@ -208 +283 @@
 
     // If we've reached this point, assemble and post the activity item
-
-    // Note for translators: The natural phrasing in English, "Joe updated
-    // his profile", requires that we know Joe's gender, which we don't. If
-    // your language doesn't have this restriction, feel free to use a more
-    // natural translation.
     $profile_link = trailingslashit( bp_core_get_user_domain( $user_id ) . buddypress()->profile->slug );
-    $action = sprintf( __( '%1$s&#8217;s profile was updated', 'buddypress' ), '<a href="' . $profile_link . '">' . bp_core_get_user_displayname( $user_id ) . '</a>' );
 
     $retval = xprofile_record_activity( array(
         'user_id'      => $user_id,
-        'action'       => $action,
         'primary_link' => $profile_link,
         'component'    => buddypress()->profile->id,