Changeset 7800

Timestamp:

10/28/2018 10:10:23 AM (6 years ago)

Author:

SergeyBiryukov

Message:

Plugins: On upload, if a previously rejected plugin with the same slug was found, free up the slug and proceed with the upload.

This brings parity between plugins rejected before and after [5460].

See #3864. See #2737.

File:

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

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

@@ -115 +115 @@
 
         $plugin_post = Plugin_Directory::get_plugin_post( $this->plugin_slug );
+
+        // Is there a previously rejected plugin with the same slug?
+        if ( $plugin_post && 'rejected' === $plugin_post->post_status ) {
+            // Change slug to 'rejected-plugin-name-rejected' to free up 'plugin-name'.
+            wp_update_post( array(
+                'ID'        => $plugin_post->ID,
+                'post_name' => sprintf( 'rejected-%s-rejected', $plugin_post->post_name ),
+            ) );
+
+            $plugin_post = null;
+        }
 
         // Is there already a plugin with the same slug by a different author?