Changeset 13146 for trunk/tests/phpunit/testcases/activity/functions.php

Timestamp:

11/13/2021 03:16:36 PM (5 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/tests/phpunit/testcases/activity/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/activity/functions.php

@@ -1975 +1975 @@
                 $this->assertSame( $latest_update, bp_get_user_meta( $u1, 'bp_latest_update', true ) );
         }
+
+        /**
+         * @ticket BP8591
+         */
+        public function test_activity_admin_screen_count_activities() {
+                $u1 = self::factory()->user->create();
+                $a1 = self::factory()->activity->create_many(
+                        5,
+                        array(
+                                'user_id'   => $u1,
+                                'component' => 'activity',
+                                'type'      => 'activity_update',
+                        )
+                );
+                bp_update_user_last_activity( $u1, date( 'Y-m-d H:i:s', bp_core_current_time( true, 'timestamp' ) ) );
+
+                $count_activities = bp_activity_get(
+                        array(
+                                'show_hidden'      => true,
+                                'count_total_only' => true,
+                        )
+                );
+
+                $this->assertTrue( 5 === (int) $count_activities['total'] );
+        }
 }