Changeset 45538 for trunk/src/wp-includes/class-wp-image-editor-gd.php

Timestamp:

06/15/2019 01:01:48 AM (7 years ago)

Author:

azaozz

Message:

Save progress of intermediate image creation after upload. First run.

• Introduces wp_get_missing_image_subsizes() and wp_update_image_subsizes() to generate image sub-sizes that are missing or were not created after the upload.
• Adds a way to display errors that happened while creating sub-sizes.
• Introduces wp_create_image_subsizes() intended for use after an image was uploaded. It saves/updates the image metadata immediately after each sub-size is created. This fixes the (long standing) problem when some of the sub-size image files were created but there was a timeout or an error and the metadata was not saved. Until now such uploads were considered "failed" which usually resulted in the user trying to upload the same image again, creating even more "orphan" image files.

Note that the patch also includes some unrelated WPCS fixes.

Props mikeschroder, azaozz.
See #40439.

File:

• trunk/src/wp-includes/class-wp-image-editor-gd.php (modified) (5 diffs)

trunk/src/wp-includes/class-wp-image-editor-gd.php

@@ -44 +44 @@
                 // On some setups GD library does not provide imagerotate() - Ticket #11536
                 if ( isset( $args['methods'] ) &&
-                        in_array( 'rotate', $args['methods'] ) &&
+                        in_array( 'rotate', $args['methods'], true ) &&
                         ! function_exists( 'imagerotate' ) ) {
 
@@ -179 +179 @@
         protected function _resize( $max_w, $max_h, $crop = false ) {
                 $dims = image_resize_dimensions( $this->size['width'], $this->size['height'], $max_w, $max_h, $crop );
+
                 if ( ! $dims ) {
                         return new WP_Error( 'error_getting_dimensions', __( 'Could not calculate resized image dimensions' ), $this->file );
                 }
+
                 list( $dst_x, $dst_y, $src_x, $src_y, $dst_w, $dst_h, $src_w, $src_h ) = $dims;
 
@@ -218 +220 @@
          */
         public function multi_resize( $sizes ) {
-                $metadata  = array();
+                $metadata = array();
+
+                foreach ( $sizes as $size => $size_data ) {
+                        $meta = $this->make_subsize( $size_data );
+
+                        if ( ! is_wp_error( $meta ) ) {
+                                $metadata[ $size ] = $meta;
+                        }
+                }
+
+                return $metadata;
+        }
+
+        /**
+         * Create an image sub-size and return the image meta data value for it.
+         *
+         * @since 5.3.0
+         *
+         * @param array $size_data Array of width, height, and whether to crop.
+         * @return WP_Error|array WP_Error on error, or the image data array for inclusion in the `sizes` array in the image meta.
+         */
+        public function make_subsize( $size_data ) {
+                if ( ! isset( $size_data['width'] ) && ! isset( $size_data['height'] ) ) {
+                        return new WP_Error( 'image_subsize_create_error', __( 'Cannot resize the image. Both width and height are not set.' ) );
+                }
+
                 $orig_size = $this->size;
 
-                foreach ( $sizes as $size => $size_data ) {
-                        if ( ! isset( $size_data['width'] ) && ! isset( $size_data['height'] ) ) {
-                                continue;
-                        }
-
-                        if ( ! isset( $size_data['width'] ) ) {
-                                $size_data['width'] = null;
-                        }
-                        if ( ! isset( $size_data['height'] ) ) {
-                                $size_data['height'] = null;
-                        }
-
-                        if ( ! isset( $size_data['crop'] ) ) {
-                                $size_data['crop'] = false;
-                        }
-
-                        $image     = $this->_resize( $size_data['width'], $size_data['height'], $size_data['crop'] );
-                        $duplicate = ( ( $orig_size['width'] == $size_data['width'] ) && ( $orig_size['height'] == $size_data['height'] ) );
-
-                        if ( ! is_wp_error( $image ) && ! $duplicate ) {
-                                $resized = $this->_save( $image );
-
-                                imagedestroy( $image );
-
-                                if ( ! is_wp_error( $resized ) && $resized ) {
-                                        unset( $resized['path'] );
-                                        $metadata[ $size ] = $resized;
-                                }
-                        }
-
-                        $this->size = $orig_size;
-                }
-
-                return $metadata;
+                if ( ! isset( $size_data['width'] ) ) {
+                        $size_data['width'] = null;
+                }
+
+                if ( ! isset( $size_data['height'] ) ) {
+                        $size_data['height'] = null;
+                }
+
+                if ( ! isset( $size_data['crop'] ) ) {
+                        $size_data['crop'] = false;
+                }
+
+                $resized = $this->_resize( $size_data['width'], $size_data['height'], $size_data['crop'] );
+
+                if ( is_wp_error( $resized ) ) {
+                        $saved = $resized;
+                } else {
+                        $saved = $this->_save( $resized );
+                        imagedestroy( $resized );
+                }
+
+                $this->size = $orig_size;
+
+                if ( ! is_wp_error( $saved ) ) {
+                        unset( $saved['path'] );
+                }
+
+                return $saved;
         }
 
@@ -392 +413 @@
                 }
 
-                if ( 'image/gif' == $mime_type ) {
+                if ( 'image/gif' === $mime_type ) {
                         if ( ! $this->make_image( $filename, 'imagegif', array( $image, $filename ) ) ) {
                                 return new WP_Error( 'image_save_error', __( 'Image Editor Save Failed' ) );
                         }
-                } elseif ( 'image/png' == $mime_type ) {
+                } elseif ( 'image/png' === $mime_type ) {
                         // convert from full colors to index colors, like original PNG.
                         if ( function_exists( 'imageistruecolor' ) && ! imageistruecolor( $image ) ) {
@@ -405 +426 @@
                                 return new WP_Error( 'image_save_error', __( 'Image Editor Save Failed' ) );
                         }
-                } elseif ( 'image/jpeg' == $mime_type ) {
+                } elseif ( 'image/jpeg' === $mime_type ) {
                         if ( ! $this->make_image( $filename, 'imagejpeg', array( $image, $filename, $this->get_quality() ) ) ) {
                                 return new WP_Error( 'image_save_error', __( 'Image Editor Save Failed' ) );