Changeset 3440

Timestamp:

06/18/2016 03:25:56 PM (8 years ago)

Author:

obenland

Message:

Plugin Directory: Revert [3439] until we have caching figured out.

See #1719, #1579.

Location:

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

Files:

• api/routes/class-plugin.php (modified) (3 diffs)
• class-tools.php (modified) (1 diff)
• shortcodes/class-reviews.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php

@@ -3 +3 @@
 use WordPressdotorg\Plugin_Directory\Plugin_Directory;
 use WordPressdotorg\Plugin_Directory\Template;
-use WordPressdotorg\Plugin_Directory\Tools;
 use WordPressdotorg\Plugin_Directory\API\Base;
 use WP_REST_Server;
@@ -188 +187 @@
     protected function get_plugin_reviews_markup( $plugin_slug ) {
         $output = '';
-        foreach ( Tools::get_plugin_reviews( $plugin_slug ) as $review ) {
+        foreach ( $this->get_plugin_reviews_data( $plugin_slug ) as $review ) {
             $output .= $this->get_plugin_reviews_markup_singular( $review );
         }
@@ -246 +245 @@
 
     }
+
+    /**
+     * Fetch the latest 10 reviews for a given plugin from the database.
+     *
+     * This uses raw SQL to query the bbPress tables to fetch reviews.
+     *
+     * @param string $plugin_slug The slug of the plugin.
+     * @return array An array of review details.
+     */
+    protected function get_plugin_reviews_data( $plugin_slug ) {
+        global $wpdb;
+        if ( ! defined( 'WPORGPATH' ) || ! defined( 'CUSTOM_USER_TABLE' ) ) {
+            // Reviews are stored in the main supoport forum, which isn't open source yet.
+            return array();
+        }
+
+        if ( $reviews = wp_cache_get( $plugin_slug, 'reviews' ) ) {
+            return $reviews;
+        }
+
+        // The forums are the source for users, and also where reviews live.
+        $table_prefix = str_replace( 'users', '', CUSTOM_USER_TABLE );
+        $forum_id = 18; // The Review Forums ID
+
+        $reviews = $wpdb->get_results( $wpdb->prepare( "
+            SELECT
+                t.topic_id, t.topic_title, t.topic_poster, t.topic_start_time,
+                p.post_text,
+                tm_wp.meta_value as wp_version
+            FROM {$table_prefix}topics AS t
+            JOIN {$table_prefix}meta AS tm ON ( tm.object_type = 'bb_topic' AND t.topic_id = tm.object_id AND tm.meta_key = 'is_plugin' )
+            JOIN {$table_prefix}posts as p ON ( t.topic_id = p.topic_id AND post_status = 0 AND post_position = 1 )
+            LEFT JOIN {$table_prefix}meta AS tm_wp ON ( tm_wp.object_type = 'bb_topic' AND t.topic_id = tm_wp.object_id AND tm_wp.meta_key = 'wp_version' )
+            WHERE t.forum_id = %d AND t.topic_status = 0 AND t.topic_sticky = 0 AND tm.meta_value = %s
+            ORDER BY t.topic_start_time DESC
+            LIMIT 10",
+            $forum_id,
+            $plugin_slug
+        ) );
+
+        wp_cache_set( $plugin_slug, $reviews, 'reviews' );
+        return $reviews;
+    }
 }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-tools.php

@@ -30 +30 @@
 
     /**
-     * Fetch the latest 10 reviews for a given plugin from the database.
-     *
-     * This uses raw SQL to query the bbPress tables to fetch reviews.
+     * Returns the two latest reviews of a specific plugin.
      *
      * @global \wpdb $wpdb WordPress database abstraction object.
      *
-     * @param string $plugin_slug The slug of the plugin.
-     * @param array  $args        {
-     *     Optional. Query arguments.
+     * @todo Populate with review title/content.
      *
-     *     @type int $number The amount of reviews to return. Default: 10.
-     * }
-     * @return array An array of reviews.
+     * @param string $plugin_slug The plugin slug.
+     * @return array|false
      */
-    public static function get_plugin_reviews( $plugin_slug, $args = array() ) {
-
-        // Reviews are stored in the main support forum, which isn't open sourced yet.
-        if ( ! defined( 'WPORGPATH' ) || ! defined( 'CUSTOM_USER_TABLE' ) ) {
-            return array();
-        }
-
-        if ( false === ( $reviews = wp_cache_get( $plugin_slug, 'reviews' ) ) ) {
+    public static function get_plugin_reviews( $plugin_slug ) {
+        if ( false === ( $reviews = wp_cache_get( "{$plugin_slug}_reviews", 'wporg-plugins' ) ) ) {
             global $wpdb;
 
-            $args = wp_parse_args( $args, array(
-                'number' => 10,
-            ) );
-
-            // The forums are the source for users, and also where reviews live.
-            $table_prefix = str_replace( 'users', '', CUSTOM_USER_TABLE );
-            $forum_id     = 18; // The Review Forums ID.
-
             $reviews = $wpdb->get_results( $wpdb->prepare( "
-                SELECT
-                    t.topic_id, t.topic_title, t.topic_poster, t.topic_start_time,
-                    p.post_text,
-                    ratings.rating,
-                    tm_wp.meta_value as wp_version
-                FROM {$table_prefix}topics AS t
-                JOIN {$table_prefix}meta AS tm ON ( tm.object_type = 'bb_topic' AND t.topic_id = tm.object_id AND tm.meta_key = 'is_plugin' )
-                JOIN {$table_prefix}posts as p ON ( t.topic_id = p.topic_id AND post_status = 0 AND post_position = 1 )
-                JOIN ratings ON (t.topic_id = ratings.review_id )
-                LEFT JOIN {$table_prefix}meta AS tm_wp ON ( tm_wp.object_type = 'bb_topic' AND t.topic_id = tm_wp.object_id AND tm_wp.meta_key = 'wp_version' )
-                WHERE t.forum_id = %d AND t.topic_status = 0 AND t.topic_sticky = 0 AND tm.meta_value = %s
-                ORDER BY t.topic_start_time DESC
-                LIMIT %d",
-                $forum_id,
-                $plugin_slug,
-                absint( $args['number'] )
-            ) );
-
-            wp_cache_set( $plugin_slug, $reviews, 'reviews' );
+            SELECT posts.post_text AS post_content, topics.topic_title AS post_title, ratings.rating AS post_rating, ratings.user_id AS post_author
+            FROM ratings
+                INNER JOIN minibb_topics AS topics ON ( ratings.review_id = topics.topic_id )
+                INNER JOIN minibb_posts AS posts ON ( ratings.review_id = posts.topic_id )
+            WHERE
+                ratings.object_type = 'plugin' AND
+                ratings.object_slug = %s AND
+                posts.post_position = 1
+            ORDER BY ratings.review_id DESC LIMIT 2", $plugin_slug ) );
+            wp_cache_set( "{$plugin_slug}_reviews", $reviews, 'wporg-plugins', HOUR_IN_SECONDS );
         }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-reviews.php

@@ -15 +15 @@
      */
     static function display() {
-        $reviews = Tools::get_plugin_reviews( get_post()->post_name, array(
-            'number' => 2,
-        ) );
+        $reviews = Tools::get_plugin_reviews( get_post()->post_name );
 
         if ( empty( $reviews ) ) {
@@ -29 +27 @@
             <?php
             foreach ( $reviews as $review ) :
-                $reviewer = get_user_by( 'id', $review->topic_poster );
+                $reviewer = get_user_by( 'id', $review->post_author );
                 if ( ! $reviewer ) :
                     continue;
@@ -39 +37 @@
                     </div><div class="review">
                         <header>
-                            <h3 class="review-title"><?php echo $review->topic_title; ?></h3>
-                            <?php echo Template::dashicons_stars( $review->rating ); ?>
-                            <span class="review-author author vcard"><a class="url fn n" href="<?php esc_url( 'https://profile.wordpress.org/' . $reviewer->user_nicename . '/' ); ?>"><?php echo Template::encode( $reviewer->display_name ); ?></a></span>
+                            <h3 class="review-title"><?php echo $review->post_title; ?></h3>
+                            <?php echo Template::dashicons_stars( $review->post_rating ); ?>
+                            <span class="review-author author vcard"><a class="url fn n" href="<?php esc_url( get_author_posts_url( $reviewer->ID ) ); ?>"><?php echo Template::encode( $reviewer->display_name ); ?></a></span>
                         </header>
-                        <p class="review-content"><?php echo $review->post_text; ?></p>
+                        <p class="review-content"><?php echo $review->post_content; ?></p>
                     </div>
                 </article>