Changeset 9094

Timestamp:

08/07/2019 10:45:11 PM (7 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 );
                 }