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

Timestamp:

11/20/2018 07:03:45 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Prevent a user removing themselves as the committer, if they're the only committer.

Fixes #3923.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin-committers.php (modified) (1 diff)

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

@@ -140 +140 @@
         $plugin_slug = $request['plugin_slug'];
 
+        // Prevent a committer removing themselves, if they're the only committer.
+        if ( $user->user_login == wp_get_current_user()->user_login ) {
+            $committers = Tools::get_plugin_committers( $plugin_slug );
+            if ( count( $committers ) == 1 && in_array( $user->user_login, $committers ) ) {
+                return new WP_Error( 'failed', __( 'Sorry, you must have at least one committer.', 'wporg-plugins' ) );
+            }
+        }
+
         $result = Tools::revoke_plugin_committer( $plugin_slug, $user );
         if ( ! $result ) {