Changeset 11351 for trunk/src/bp-activity/classes/class-bp-activity-list-table.php

Timestamp:

12/29/2016 02:08:09 AM (8 years ago)

Author:

boonebgorges

Message:

bp_get_activity_content_body filter should receive same params in all contexts.

This changeset makes all instances of bp_get_activity_content_body
consistent by ensuring that they all receive the activity object
as a second parameter. This requires switching to apply_filters_ref_array(),
so that the second param can always be passed by reference.

Props sanket.parmar.
Fixes #6971.

File:

• trunk/src/bp-activity/classes/class-bp-activity-list-table.php (modified) (1 diff)

trunk/src/bp-activity/classes/class-bp-activity-list-table.php

@@ -642 +642 @@
         // Get activity content - if not set, use the action.
         if ( ! empty( $item['content'] ) ) {
-
-            /**
-             * Filters current activity item content.
-             *
-             * @since 1.2.0
-             *
-             * @param array $item Array index holding current activity item content.
-             */
-            $content = apply_filters_ref_array( 'bp_get_activity_content_body', array( $item['content'] ) );
+            $activity = new BP_Activity_Activity( $item['id'] );
+
+            /** This filter is documented in bp-activity/bp-activity-template.php */
+            $content = apply_filters_ref_array( 'bp_get_activity_content_body', array( $item['content'], &$activity ) );
         } else {
             /**