Ticket #2860: 2860.3.diff

wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php

@@ -28 +28 @@
 
                 add_action( 'approved_plugin', array( $this, 'approved' ), 10, 2 );
                 add_action( 'rejected_plugin', array( $this, 'rejected' ), 10, 2 );
+                add_action( 'closed_plugin', array( $this, 'closed' ), 10, 2 );
+                add_action( 'disabled_plugin', array( $this, 'closed' ), 10, 2 );
         }
 
         /**
@@ -246 +248 @@
                 wp_mail( $email, $subject, $content, 'From: plugins@wordpress.org' );
         }
 
+        /**
+         * Fires when a post is transitioned to 'closed'.
+         *
+         * @param int      $post_id Post ID.
+         * @param \WP_Post $post    Post object.
+         */
+        public function closed( $post_id, $post ) {
+
+                if ( ! current_user_can( 'plugin_approve', $post ) ) {
+                        return;
+                }
+
+                if ( ! isset( $_POST['close_reason'] ) ) {
+                        return;
+                }
+
+                update_post_meta( $post->ID, '_close_reason', sanitize_key( $_POST['close_reason'] ) );
+        }
+
         /**
          * Returns the path to a plugins root directory.
          *

wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-controls.php

@@ -64 +64 @@
                 return $label;
         }
 
+        /**
+         * Get labels for reasons for closing a plugin.
+         *
+         * @return array Close reason labels.
+         */
+        public static function get_close_reasons() {
+                return array(
+                        'security'                      => __( 'Security Issue', 'wporg-plugins' ),
+                        'author-request'                => __( 'Author Request', 'wporg-plugins' ),
+                        'guideline-violation'           => __( 'Guideline Violation', 'wporg-plugins' ),
+                        'licensing-trademark-violation' => __( 'Licensing/Trademark Violation', 'wporg-plugins' ),
+                        'merged-into-core'              => __( 'Merged into Core', 'wporg-plugins' ),
+                );
+        }
+
         /**
          * Displays the Plugin Status control in the Publish metabox.
          */
@@ -80 +95 @@
                 if ( current_user_can( 'plugin_approve', $post ) ) {
                         $statuses = Status_Transitions::get_allowed_transitions( $post->post_status );
                 }
+
+                $close_reasons = self::get_close_reasons();
                 ?>
                 <div class="misc-pub-section misc-pub-plugin-status">
                         <label for="post_status"><?php _e( 'Status:', 'wporg-plugins' ); ?></label>
                         <strong id="plugin-status-display"><?php echo esc_html( get_post_status_object( $post->post_status )->label ); ?></strong>
 
+                        <?php if ( 'closed' === $post->post_status || 'disabled' === $post->post_status ) : ?>
+                                <?php
+                                $close_reason = (string) get_post_meta( $post->ID, '_close_reason', true );
+
+                                if ( isset( $close_reasons[ $close_reason ] ) ) {
+                                        $close_reason = $close_reasons[ $close_reason ];
+                                } elseif ( empty( $close_reason ) ) {
+                                        $close_reason = _x( 'Unknown', 'unknown close reason', 'wporg-plugins' );
+                                }
+
+                                if ( 'closed' === $post->post_status ) {
+                                        $label = __( 'Close Reason:', 'wporg-plugins' );
+                                } else {
+                                        $label = __( 'Disable Reason:', 'wporg-plugins' );
+                                }
+                                ?>
+                                <p><?php echo esc_html( $label ) . ' ' . esc_html( $close_reason ); ?></p>
+                        <?php elseif ( in_array( 'closed', $statuses, true ) || in_array( 'disabled', $statuses, true ) ) : ?>
+                                <p>
+                                        <label for="close_reason"><?php esc_html_e( 'Close/Disable Reason:', 'wporg-plugins' ); ?></label>
+                                        <select name="close_reason" id="close_reason">
+                                                <?php foreach ( $close_reasons as $reason => $label ) : ?>
+                                                        <option value="<?php echo esc_attr( $reason ); ?>"><?php echo esc_html( $label ); ?></option>
+                                                <?php endforeach; ?>
+                                        </select>
+                                </p>
+                        <?php endif; ?>
+
                         <?php foreach ( $statuses as $status ) : ?>
                                 <p><button type="submit" name="post_status" value="<?php echo esc_attr( $status ); ?>" class="button set-plugin-status">
                                         <?php echo self::get_status_button_label( $status ); ?>