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

Timestamp:

08/06/2020 07:44:59 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Add a notification email to committers when a new committer or support rep is added.

Fixes #5351.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-tools.php (modified) (3 diffs)

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

@@ -3 +3 @@
 
 use WP_User;
+use WordPressdotorg\Plugin_Directory\Email\Committer_Added_Notification;
+use WordPressdotorg\Plugin_Directory\Email\Support_Rep_Added_Notification;
 
 /**
@@ -205 +207 @@
                         $plugin_slug
                 );
+
+                // Only notify if the current process is interactive - a user is logged in.
+                $should_notify = (bool) get_current_user_id();
+                // Don't notify if a plugin admin is taking action on a plugin they're not (yet) a committer for.
+                if ( current_user_can( 'plugin_approve' ) && ! in_array( wp_get_current_user()->user_login, $existing_committers, true ) ) {
+                        $should_notify = false;
+                }
+
+                if ( $should_notify ) {
+                        $existing_committers[] = $user->user_login;
+
+                        $email = new Committer_Added_Notification(
+                                $plugin_slug,
+                                $existing_committers,
+                                [
+                                        'committer' => $user,
+                                ]
+                        );
+                        $email->send();
+                }
 
                 return $result;
@@ -353 +375 @@
                 );
 
+                $committers = self::get_plugin_committers( $plugin_slug );
+
+                // Only notify if the current process is interactive - a user is logged in.
+                $should_notify = (bool) get_current_user_id();
+                // Don't notify if a plugin admin is taking action on a plugin they're not a committer for.
+                if ( current_user_can( 'plugin_approve' ) && ! in_array( wp_get_current_user()->user_login, $committers, true ) ) {
+                        $should_notify = false;
+                }
+
+                if ( $should_notify ) {
+                        $email = new Support_Rep_Added_Notification(
+                                $plugin_slug,
+                                $committers,
+                                [
+                                        'rep' => $user,
+                                ]
+                        );
+                        $email->send();
+                }
+
                 return $result;
         }