Changeset 13033 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php

Timestamp:

12/08/2023 03:47:14 AM (10 months ago)

Author:

dd32

Message:

Plugin Directory: Record the reason a plugin is rejected.

See #5653.
Closes ​https://github.com/WordPress/wordpress.org/pull/107.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php (modified) (4 diffs)

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

@@ -142 +142 @@
 
             case 'rejected':
+                $this->save_rejected_reason( $post->ID );
                 $this->rejected( $post->ID, $post );
                 $this->clear_reviewer( $post );
@@ -271 +272 @@
                 'slug'            => $original_permalink,
                 'submission_date' => $submission_date,
+                'reason'          => sanitize_key( $_POST['rejection_reason'] ?? '' )
             ]
         );
-        // ..and log rejection.
-        if ( $email->send() ) {
-            Tools::audit_log( 'Plugin rejected.', $post_id ); 
-        } else {
-            Tools::audit_log( 'Plugin rejected. Email not sent.', $post_id );
-        }
+
+        $email->send();
     }
 
@@ -331 +329 @@
         }
 
-        if ( ! current_user_can( 'plugin_approve', $post_id ) ) {
+        if ( ! current_user_can( 'plugin_close', $post_id ) ) {
             return;
         }
@@ -341 +339 @@
 
         Tools::audit_log( sprintf( 'Plugin closed. Reason: %s', $close_reason ), $post_id );
+    }
+
+    /**
+     * Save the reason for rejecting a plugin.
+     *
+     * @param int $post_id Post ID.
+     */
+    public function save_rejected_reason( $post_id ) {
+        if ( ! isset( $_REQUEST['rejection_reason'] ) ) {
+            return;
+        }
+
+        if ( ! current_user_can( 'plugin_reject', $post_id ) ) {
+            return;
+        }
+
+        $rejection_reason = sanitize_key( $_REQUEST['rejection_reason'] );
+
+        update_post_meta( $post_id, '_rejection_reason', $rejection_reason );
+        update_post_meta( $post_id, 'plugin_rejected_date', current_time( 'mysql' ) );
+
+        Tools::audit_log( sprintf( 'Plugin rejected. Reason: %s', $rejection_reason ), $post_id );
     }