Changeset 13794 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-upload-handler.php

Timestamp:

06/10/2024 07:45:28 AM (6 months ago)

Author:

dd32

Message:

Plugin Directory: Upload: Specify that the post_author should remain the same when uploading additional ZIPs.

When a plugin is in certain states, the act of a plugin mod uploading an additional ZIP via wp-admin could cause the plugins ownership to change to the plugin mod instead of remaining the same. This prevents that.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-upload-handler.php (modified) (2 diffs)

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

@@ -79 +79 @@
 
         $zip_file         = $_FILES['zip_file']['tmp_name'];
-        $upload_comment   = trim( wp_unslash( $_POST['comment'] ) );
+        $upload_comment   = trim( wp_unslash( $_POST['comment'] ?? '' ) );
         $has_upload_token = $this->has_valid_upload_token();
         $this->plugin_dir = Filesystem::unzip( $zip_file );
@@ -423 +423 @@
         $post_args = array(
             'ID'            => $plugin_post->ID ?? 0,
+            'post_author'   => $plugin_post->post_author ?? get_current_user_id(),
             'post_title'    => $this->plugin['Name'],
             'post_name'     => $this->plugin_slug,