Changeset 9257 for trunk/src/bp-activity/bp-activity-functions.php

Timestamp:

12/22/2014 09:46:50 PM (12 years ago)

Author:

r-a-y

Message:

Activity: Support multiple scopes in bp_has_activities().

This commit:

• Allows the 'scope' parameter to support multiple scopes. For example, scope=friends,mentions. This can be a comma-delimited string or an array.
• Makes the 'scope' parameter available across the bp_has_activities() stack. Previously, 'scope' was only available for use in the bp_has_activities() loop.
• Parses scopes using the BP_Activity_Query class to handle complex conditions.
• Allows components to declare their own custom activity scopes. Components can also override existing activity arguments by setting the 'override' key in their scope callback. For an example, see how the friends component declares the 'friends' scope in bp_friends_filter_activity_scope(). Previously, it was not possible for components or third-party plugins to declare their own scopes without doing a fair bit of workarounds.

See #4988.

File:

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

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

@@ -1424 +1424 @@
                 'update_meta_cache' => true,
                 'count_total'       => false,
+                'scope'             => false,
 
                 /**
@@ -1439 +1440 @@
 
         // Attempt to return a cached copy of the first page of sitewide activity.
-        if ( ( 1 === (int) $r['page'] ) && empty( $r['max'] ) && empty( $r['search_terms'] ) && empty( $r['meta_query'] ) && empty( $r['date_query'] ) && empty( $r['filter_query'] ) && empty( $r['filter'] ) && empty( $r['exclude'] ) && empty( $r['in'] ) && ( 'DESC' === $r['sort'] ) && empty( $r['exclude'] ) && ( 'ham_only' === $r['spam'] ) ) {
+        if ( ( 1 === (int) $r['page'] ) && empty( $r['max'] ) && empty( $r['search_terms'] ) && empty( $r['meta_query'] ) && empty( $r['date_query'] ) && empty( $r['filter_query'] ) && empty( $r['filter'] ) && empty( $r['scope'] )&& empty( $r['exclude'] ) && empty( $r['in'] ) && ( 'DESC' === $r['sort'] ) && empty( $r['exclude'] ) && ( 'ham_only' === $r['spam'] ) ) {
 
                 $activity = wp_cache_get( 'bp_activity_sitewide_front', 'bp' );
@@ -1454 +1455 @@
                                 'filter_query'      => $r['filter_query'],
                                 'filter'            => $r['filter'],
+                                'scope'             => $r['scope'],
                                 'display_comments'  => $r['display_comments'],
                                 'show_hidden'       => $r['show_hidden'],
@@ -1475 +1477 @@
                         'filter_query'     => $r['filter_query'],
                         'filter'           => $r['filter'],
+                        'scope'            => $r['scope'],
                         'display_comments' => $r['display_comments'],
                         'show_hidden'      => $r['show_hidden'],