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

Timestamp:

11/13/2021 03:16:36 PM (3 years ago)

Author:

imath

Message:

Add a new param to bp_activity_get() to only get the number of items

Using the count_total_only parameter with bp_activity_get() will only run the query to count the number of activity items. Using this new parameter into the Activity WP Admin Screen improves the performance of the query to get the total number of activities.

Props oztaser

Fixes #8591

File:

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

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

@@ -1784 +1784 @@
             'sort'              => 'DESC',       // sort ASC or DESC.
             'display_comments'  => false,        // False for no comments. 'stream' for within stream display, 'threaded' for below each activity item.
-
             'search_terms'      => false,        // Pass search terms as a string.
             'meta_query'        => false,        // Filter by activity meta. See WP_Meta_Query for format.
@@ -1791 +1790 @@
             'show_hidden'       => false,        // Show activity items that are hidden site-wide?
             'exclude'           => false,        // Comma-separated list of activity IDs to exclude.
-            'in'                => false,        // Comma-separated list or array of activity IDs to which you.
-                                                // want to limit the query.
+            'in'                => false,        // Comma-separated list or array of activity IDs to which you want to limit the query.
             'spam'              => 'ham_only',   // 'ham_only' (default), 'spam_only' or 'all'.
             'update_meta_cache' => true,
             'count_total'       => false,
+            'count_total_only'  => false,
             'scope'             => false,
 
@@ -1813 +1812 @@
     );
 
-    $activity = BP_Activity_Activity::get( array(
-        'page'              => $r['page'],
-        'per_page'          => $r['per_page'],
-        'max'               => $r['max'],
-        'sort'              => $r['sort'],
-        'search_terms'      => $r['search_terms'],
-        'meta_query'        => $r['meta_query'],
-        'date_query'        => $r['date_query'],
-        'filter_query'      => $r['filter_query'],
-        'filter'            => $r['filter'],
-        'scope'             => $r['scope'],
-        'display_comments'  => $r['display_comments'],
-        'show_hidden'       => $r['show_hidden'],
-        'exclude'           => $r['exclude'],
-        'in'                => $r['in'],
-        'spam'              => $r['spam'],
-        'update_meta_cache' => $r['update_meta_cache'],
-        'count_total'       => $r['count_total'],
-        'fields'            => $r['fields'],
-    ) );
+    $activity = BP_Activity_Activity::get(
+        array(
+            'page'              => $r['page'],
+            'per_page'          => $r['per_page'],
+            'max'               => $r['max'],
+            'sort'              => $r['sort'],
+            'search_terms'      => $r['search_terms'],
+            'meta_query'        => $r['meta_query'],
+            'date_query'        => $r['date_query'],
+            'filter_query'      => $r['filter_query'],
+            'filter'            => $r['filter'],
+            'scope'             => $r['scope'],
+            'display_comments'  => $r['display_comments'],
+            'show_hidden'       => $r['show_hidden'],
+            'exclude'           => $r['exclude'],
+            'in'                => $r['in'],
+            'spam'              => $r['spam'],
+            'update_meta_cache' => $r['update_meta_cache'],
+            'count_total'       => $r['count_total'],
+            'count_total_only'  => $r['count_total_only'],
+            'fields'            => $r['fields'],
+        )
+    );
 
     /**