Changeset 5868

Timestamp:

09/03/2017 09:46:04 PM (7 years ago)

Author:

SergeyBiryukov

Message:

Support Forums: Introduce Directory_Compat::get_support_reps() method to retrieve users assigned as the plugin/theme support reps.

See #2699.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc

Files:

• class-directory-compat.php (modified) (6 diffs)
• class-plugin-directory-compat.php (modified) (1 diff)
• class-stickies-compat.php (modified) (3 diffs)
• class-theme-directory-compat.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-directory-compat.php

@@ -24 +24 @@
     var $authors      = null;
     var $contributors = null;
+    var $support_reps = null;
     var $query        = null;
     var $term         = null;
@@ -317 +318 @@
                 $this->authors           = $this->get_authors( $slug );
                 $this->contributors      = $this->get_contributors( $slug );
+                $this->support_reps      = $this->get_support_reps( $slug );
                 $this->term              = $terms[0];
 
@@ -332 +334 @@
                 // Instantiate WPORG_Stickies mode for topic view.
                 if ( class_exists( 'WordPressdotorg\Forums\Stickies_Compat' ) ) {
-                    $this->stickies = new Stickies_Compat( $this->compat(), $this->slug(), $this->taxonomy(), $this->get_object( $this->slug() ), $this->term, $this->authors, $this->contributors );
+                    $this->stickies = new Stickies_Compat( $this->compat(), $this->slug(), $this->taxonomy(), $this->get_object( $this->slug() ), $this->term, $this->authors, $this->contributors, $this->support_reps );
                 }
 
@@ -341 +343 @@
 
     /**
-     * Allow plugin/theme authors and contributors to resolve a topic on their support forum.
+     * Allow plugin/theme authors, contributors, and support reps to resolve a topic
+     * on their support forum.
      *
      * @param bool $retval If the user can set a topic resolution for the topic
@@ -358 +361 @@
             ( ! empty( $this->contributors ) && in_array( $user->user_nicename, $this->contributors ) )
         || 
+            ( ! empty( $this->support_reps ) && in_array( $user->user_nicename, $this->support_reps ) )
+        || 
+            // Back-compat for support reps added before https://meta.trac.wordpress.org/changeset/5867,
+            // can be removed once they are re-added via the Plugin Directory UI.
             ( is_a( $user, 'WP_User' ) && $user->supportrep == $this->slug() )
         ) {
@@ -902 +909 @@
         return $contributors;
     }
+
+    public function get_support_reps( $slug ) {
+        global $wpdb;
+
+        if ( null !== $this->support_reps ) {
+            return $this->support_reps;
+        }
+
+        // Themes do not have support reps right now.
+        if ( $this->compat() == 'theme' ) {
+            $support_reps = array();
+            return $support_reps;
+        }
+
+        // Check the cache.
+        $cache_key = $slug;
+        $cache_group = $this->compat() . '-support-reps-slugs';
+        $support_reps = wp_cache_get( $cache_key, $cache_group );
+        if ( ! $support_reps ) {
+            $plugin = $this->get_object( $slug );
+            $prefix = $wpdb->base_prefix . WPORG_PLUGIN_DIRECTORY_BLOGID . '_';
+            $support_reps = $wpdb->get_col( $wpdb->prepare(
+                "SELECT slug
+                 FROM {$prefix}terms AS t
+                 LEFT JOIN {$prefix}term_taxonomy AS tt ON tt.term_id = t.term_id
+                 LEFT JOIN {$prefix}term_relationships AS tr ON tr.term_taxonomy_id = tt.term_taxonomy_id 
+                 WHERE tt.taxonomy = 'plugin_support_reps' AND tr.object_id = %d",
+                 $plugin->ID
+            ) );
+
+            wp_cache_set( $cache_key, $support_reps, $cache_group, HOUR_IN_SECONDS );
+        }
+        return $support_reps;
+    }
 }

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-plugin-directory-compat.php

@@ -79 +79 @@
             $this->authors      = $this->get_authors( $slug );
             $this->contributors = $this->get_contributors( $slug );
+            $this->support_reps = $this->get_support_reps( $slug );
         }
     }

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-stickies-compat.php

@@ -13 +13 @@
     var $term     = null;
 
-    public function __construct( $compat, $slug, $taxonomy, $object, $term, $authors = array(), $contributors = array() ) {
+    public function __construct( $compat, $slug, $taxonomy, $object, $term, $authors = array(), $contributors = array(), $support_reps = array() ) {
         if ( empty( $compat ) || empty( $slug ) || empty( $taxonomy ) || empty( $object ) || empty( $term ) ) {
             return;
@@ -25 +25 @@
         $this->authors      = $authors;
         $this->contributors = $contributors;
+        $this->support_reps = $support_reps;
 
         // Remove global stickies from sticky array.
@@ -248 +249 @@
                 $retval = true;
             }
+
+            // Compat support reps.
+            if ( $this->support_reps && in_array( $user->user_nicename, $this->support_reps ) ) {
+                $retval = true;
+            }
         }
         return $retval;

sites/trunk/wordpress.org/public_html/wp-content/plugins/support-forums/inc/class-theme-directory-compat.php

@@ -79 +79 @@
             $this->authors      = $this->get_authors( $slug );
             $this->contributors = $this->get_contributors( $slug );
+            $this->support_reps = $this->get_support_reps( $slug );
         }
     }