Changeset 3440

Timestamp:

06/18/2016 03:25:56 PM (10 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>