Changeset 13510

Timestamp:

04/12/2024 05:22:33 AM (22 months ago)

Author:

dd32

Message:

Plugin Directory, Support Forums: Automatically subscribe plugin committers and support reps to a plugins forum activity.

Users can opt-out through the unsubscribe link included with all support forum emails.

This is only applied for future committer/support rep changes.

Fixes #7566.

Location:

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

Files:

• plugin-directory/class-tools.php (modified) (3 diffs)
• support-forums/inc/class-directory-compat.php (modified) (2 diffs)
• support-forums/inc/class-plugin-directory-compat.php (modified) (1 diff)
• support-forums/inc/class-plugin.php (modified) (2 diffs)
• support-forums/inc/class-rest-api.php (added)
• support-forums/support-forums.php (modified) (1 diff)

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

@@ -239 +239 @@
             $email->send();
         }
+
+        Tools::subscribe_user_to_forum_threads( $user, $post );
 
         return $result;
@@ -407 +409 @@
         }
 
+        Tools::subscribe_user_to_forum_threads( $user, $post );
+
         return $result;
     }
@@ -639 +643 @@
         ] );
     }
+
+    /**
+     * Subscribe a user to the forum topics for a given plugin.
+     *
+     * @param WP_User|int $user_id The user to subscribe.
+     * @param WP_Post     $post    The plugin to subscribe to.
+     * @return bool
+     */
+    public static function subscribe_user_to_forum_threads( $user_id, $post = null ) {
+        $post = Plugin_Directory::get_plugin_post( $post );
+
+        if ( ! $user_id || ! $post || ! defined( 'PLUGIN_API_INTERNAL_BEARER_TOKEN' ) ) {
+            return false;
+        }
+
+        if ( $user_id instanceof \WP_User ) {
+            $user_id = $user_id->ID;
+        }
+
+        $request = wp_remote_post(
+            'https://wordpress.org/support/wp-json/wporg-support/v1/subscribe-user-to-term',
+            [
+                'body'    => [
+                    'type'    => 'plugin',
+                    'slug'    => $post->post_name,
+                    'user_id' => $user_id,
+                ],
+                'headers' => [
+                    'Authorization' => 'Bearer ' . PLUGIN_API_INTERNAL_BEARER_TOKEN,
+                ],
+            ]
+        );
+
+        return 200 === wp_remote_retrieve_response_code( $request );
+    }
 }

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

@@ -289 +289 @@
 
             // Set the term for this view so we can reuse it.
-            $this->term = get_term_by( 'slug', $this->slug(), $this->taxonomy() );
-
-            // New compats won't have any support topics or reviews, so will
-            // not yet exist as a compat term.
-            if ( ! $this->term && $this->get_object( $this->slug() ) ) {
-                $term_name = $this->slug();
-                if ( ! sanitize_title( $term_name ) ) {
-                    // This happens when the slug is all non-ascii such as %e5%8f%8b%e8%a8%80, which fails to insert.
-                    $term_name = urldecode( $term_name );
-                }
-                $term = wp_insert_term( $term_name, $this->taxonomy(), array( 'slug' => $this->slug() ) );
-
-                // Term exists already? Race-condition, or get_term_by() couldn't find $slug..
-                if ( is_wp_error( $term ) && $term->get_error_data( 'term_exists' ) ) {
-                    $this->term = get_term( $term->get_error_data( 'term_exists' ) );
-                } elseif ( ! is_wp_error( $term ) && isset( $term['term_id'] ) ) {
-                    $this->term = get_term( $term['term_id'] );
-                }
-            }
+            $this->initialize_term();
 
             // Add plugin- and theme-specific filters and actions.
@@ -345 +327 @@
 
             $this->loaded = true;
+        }
+    }
+
+    /**
+     * Initialises the WP_Term for the compat view.
+     *
+     * If the term does not exist, it will be created.
+     */
+    public function initialize_term() {
+        if ( ! $this->slug() ) {
+            return;
+        }
+
+        $this->term = get_term_by( 'slug', $this->slug(), $this->taxonomy() );
+
+        if ( $this->term || ! $this->get_object( $this->slug() ) ) {
+            return;
+        }
+
+        $term_name = $this->slug();
+        if ( ! sanitize_title( $term_name ) ) {
+            // This happens when the slug is all non-ascii such as %e5%8f%8b%e8%a8%80, which fails to insert.
+            $term_name = urldecode( $term_name );
+        }
+
+        $term = wp_insert_term( $term_name, $this->taxonomy(), array( 'slug' => $this->slug() ) );
+
+        // Term exists already? Race-condition, or get_term_by() couldn't find $slug..
+        if ( is_wp_error( $term ) && $term->get_error_data( 'term_exists' ) ) {
+            $this->term = get_term( $term->get_error_data( 'term_exists' ) );
+        } elseif ( ! is_wp_error( $term ) && isset( $term['term_id'] ) ) {
+            $this->term = get_term( $term['term_id'] );
         }
     }

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

@@ -113 +113 @@
         $this->support_reps = $this->get_support_reps( $slug );
 
+        $this->initialize_term();
+
         return true;
     }

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

@@ -40 +40 @@
     public $theme_subscriptions  = false; // Defined via Support_Compat
     public $blocks               = false;
+    public $rest_api             = false;
 
     /**
@@ -74 +75 @@
         $this->emails       = new Emails;
         $this->audit_log    = new Audit_Log;
+        $this->rest_api     = new REST_API;
 
         // Set a flag to indicate whether this is the global forums, or a locale forum.

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

@@ -31 +31 @@
 include( __DIR__ . '/inc/class-audit-log.php' );
 include( __DIR__ . '/inc/class-blocks.php' );
+include( __DIR__ . '/inc/class-rest-api.php' );
 
 // Compat-only includes.