Changeset 1286

Timestamp:

02/20/2015 02:29:04 AM (10 years ago)

Author:

obenland

Message:

WP.org Themes: Finish browse setups.

• To achieve parity between server-side and API results of featured themes, the

API will continue to be responsible for determining which themes are featured.
It will store the featured theme's post IDs in cache, for the directory to use.

• We'll use the same query adjustment as in the API to get the same ouput of

popular themes.

• New themes will just be the latest themes, much like blog posts.

See #745.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/theme-directory.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/theme-directory.php

@@ -104 +104 @@
                 $wp_query->set( 'paged', 1 );
                 $wp_query->set( 'posts_per_page', 15 );
-                $wp_query->set( 'date_query', array(
-                    array(
-                        'column' => 'post_modified_gmt',
-                        'after'  => '-1 year',
-                    ),
-                ) );
-                $wp_query->set( 'orderby', 'rand' );
+                $wp_query->set( 'post__in', (array) wp_cache_get( 'browse-popular', 'theme-info' ) );
                 break;
 
             case 'popular':
-                //TODO: Sort by popularity.
+                add_filter( 'posts_clauses',  'wporg_themes_popular_posts_clauses' );
                 break;
 
             case 'new':
+                // Nothing to do here.
                 break;
         }
@@ -141 +136 @@
 }
 add_filter( 'found_posts', 'wporg_themes_found_posts', 10, 2 );
+
+/**
+ * Filters SQL clauses, to set up a query for the most popular themes based on downloads.
+ *
+ * @param array $clauses
+ * @return array
+ */
+function wporg_themes_popular_posts_clauses( $clauses ) {
+    global $wpdb;
+
+    $week = gmdate( 'Y-m-d', strtotime( 'last week' ) );
+    $clauses['where']  .= " AND s.stamp >= '{$week}'";
+    $clauses['groupby'] = "{$wpdb->posts}.ID";
+    $clauses['join']    = "JOIN bb_themes_stats AS s ON ( {$wpdb->posts}.post_name = s.slug )";
+    $clauses['orderby'] = 'week_downloads DESC';
+    $clauses['fields'] .= ', SUM(s.downloads) AS week_downloads';
+
+    return $clauses;
+}
 
 /**