Changeset 9094

Timestamp:

08/07/2019 10:45:11 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Plugin Directory: Track plugin ownership change in audit log.

Props Ipstenu, SergeyBiryukov.
Fixes #4663.

Location:

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

Files:

• admin/class-status-transitions.php (modified) (3 diffs)
• class-plugin-directory.php (modified) (1 diff)

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

@@ -27 +27 @@
     private function __construct() {
         add_action( 'transition_post_status', array( $this, 'transition_post_status' ), 11, 3 );
+        add_action( 'post_updated', array( $this, 'record_owner_change' ), 11, 3 );
     }
 
@@ -384 +385 @@
         update_post_meta( $post_id, 'plugin_closed_date', current_time( 'mysql' ) );
 
-        $this->audit_log( sprintf( 'Plugin closed for: %s', $close_reason ), $post_id );
+        $this->audit_log( sprintf( 'Plugin closed. Reason: %s', $close_reason ), $post_id );
+    }
+
+    /**
+     * Records a plugin owner change.
+     *
+     * @param integer $post_id     Post ID.
+     * @param WP_Post $post_after  Post object following the update.
+     * @param WP_Post $post_before Post object before the update.
+     */
+    public function record_owner_change( $post_id, $post_after, $post_before ) {
+        if ( 'plugin' !== $post_after->post_type ) {
+            return;
+        }
+
+        if ( $post_after->post_author === $post_before->post_author ) {
+            return;
+        }
+
+        $new_owner = get_userdata( $post_after->post_author );
+
+        $this->audit_log( sprintf(
+            'Ownership transferred to <a href="%s">%s</a>.',
+            esc_url( '//profiles.wordpress.org/' . $new_owner->user_nicename ),
+            $new_owner->user_login
+        ), $post_id );
     }
 
@@ -393 +419 @@
      * @param integer $post_id The Post ID.
      */
-    private function audit_log( $message, $post_id ) {
+    public function audit_log( $message, $post_id ) {
         $user = wp_get_current_user();
 

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

@@ -1519 +1519 @@
             wp_cache_set( $result, $slug, 'plugin-slugs' );
             $result = get_post( $result );
+
+            $owner = get_userdata( $result->post_author );
+
+            Admin\Status_Transitions::instance()->audit_log( sprintf(
+                'Submitted by <a href="%s">%s</a>.',
+                esc_url( '//profiles.wordpress.org/' . $owner->user_nicename ),
+                $owner->user_login
+            ), $result->ID );
         }