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

Timestamp:

06/05/2023 04:40:30 AM (2 years ago)

Author:

dd32

Message:

Plugin Directory: Upload Shortcode: Remove a level of if() nesting for slightly more readable code.

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

@@ -61 +61 @@
      */
     public function process_upload() {
+        if ( UPLOAD_ERR_OK !== $_FILES['zip_file']['error'] ) {
+            return new \WP_Error( 'error_upload', __( 'Error in file upload.', 'wporg-plugins' ) );
+        }
+
+        $zip_file         = $_FILES['zip_file']['tmp_name'];
         $has_upload_token = $this->has_valid_upload_token();
-        $zip_file         = $_FILES['zip_file']['tmp_name'];
         $this->plugin_dir = Filesystem::unzip( $zip_file );