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

Timestamp:

05/18/2023 03:26:43 AM (17 months ago)

Author:

dd32

Message:

Plugin Directory: Clear the assigned reviewer on the appropriate status transitions (->approved and ->publish).

See #6993.

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

@@ -8 +8 @@
 use WordPressdotorg\Plugin_Directory\Email\Plugin_Approved as Plugin_Approved_Email;
 use WordPressdotorg\Plugin_Directory\Email\Plugin_Rejected as Plugin_Rejected_Email;
+use WordPressdotorg\Plugin_Directory\Admin\Metabox\Reviewer as Reviewer_Metabox;
 
 /**
@@ -135 +136 @@
             case 'approved':
                 $this->approved( $post->ID, $post );
+                $this->clear_reviewer( $post );
                 break;
 
             case 'rejected':
                 $this->rejected( $post->ID, $post );
+                $this->clear_reviewer( $post );
                 break;
 
@@ -144 +147 @@
                 $this->clean_closed_date( $post->ID );
                 $this->set_translation_status( $post, 'active' );
+                $this->clear_reviewer( $post );
                 break;
 
@@ -364 +368 @@
     }
 
+    /**
+     * Clear the assigned reviewer.
+     *
+     * @param \WP_Post $post Post object.
+     */
+    public function clear_reviewer( $post ) {
+        // Unset the reviewer, but don't log it, as the triggering status changes should've been logged in some form.
+        Reviewer_Metabox::set_reviewer( $post, false, false );
+    }
 }