Changeset 7841

Timestamp:

11/13/2018 03:18:20 PM (6 years ago)

Author:

vedjain

Message:

WordCamp: Use `_status_change_log_%' to filter WordCamp Applications.

On WordCamp Application Status report page, we were fetching 300 WordCamp posts, iterating over them one by one and figuring out if we should show in the report.
Changed this to directly filter in database for the posts that we want to show. This allows us to limit the 300 limit as we are already querying in the Database.

Fixes #3911

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-event/tracker.php (modified) (4 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wcpt/wcpt-event/tracker.php

@@ -24 +24 @@
  */
 function get_active_wordcamps() {
+    global $wpdb;
     $wordcamps          = array();
     $statuses           = WordCamp_Loader::get_post_statuses();
@@ -32 +33 @@
     unset( $shown_statuses[ WCPT_FINAL_STATUS ] );
     $shown_statuses = array_keys( $shown_statuses );
+    $wordcamp_post_type = WCPT_POST_TYPE_ID;
+
+    $wordcamp_post_objs = $wpdb->get_results(
+        $wpdb->prepare(
+            "
+            SELECT DISTINCT post_id
+            FROM {$wpdb->prefix}postmeta
+            WHERE
+                meta_key like '_status_change_log_$wordcamp_post_type%'
+            AND
+                meta_value >= %d
+            ",
+            $inactive_timestamp
+        )
+    );
+    $wordcamp_post_ids = wp_list_pluck( $wordcamp_post_objs, 'post_id' );
 
     $raw_posts = get_posts(
@@ -37 +54 @@
             'post_type'      => WCPT_POST_TYPE_ID,
             'post_status'    => $shown_statuses,
-            'posts_per_page' => 300,
+            'posts_per_page' => -1,
             'order'          => 'ASC',
             'orderby'        => 'post_title',
+            'post__in'       => $wordcamp_post_ids,
         )
     );
@@ -45 +63 @@
     foreach ( $raw_posts as $key => $post ) {
         $last_update_timestamp = get_last_update_timestamp( $post->ID );
-
-        if ( $last_update_timestamp <= $inactive_timestamp ) {
-            continue;
-        }
 
         $wordcamps[] = array(