diff --git src/wp-includes/class-wp-image-editor-imagick.php src/wp-includes/class-wp-image-editor-imagick.php index a14fa40..561e26f 100644 --- src/wp-includes/class-wp-image-editor-imagick.php +++ src/wp-includes/class-wp-image-editor-imagick.php @@ -249,18 +249,91 @@ class WP_Image_Editor_Imagick extends WP_Image_Editor { return $this->crop( $src_x, $src_y, $src_w, $src_h, $dst_w, $dst_h ); } + // Execute the resize + $this->_resize( $dst_w, $dst_h ); + + return $this->update_size( $dst_w, $dst_h ); + } + + /** + * Private function to efficiently resize an image. + * + * @since 4.5.0 + * + * @param int $dst_w The destination width. + * @param int $dst_h The destination height. + * @param string $filter Optional. The Imagick filter to use when resizing. Default 'triangle'. + * @param bool $strip_meta Optional. Strip all profiles, exluding color profiles, from the image. Default true. + * @return bool|WP_Error + */ + protected function _resize( $dst_w, $dst_h, $filter = 'triangle', $strip_meta = true ) { + $allowed_filters = array( + 'point' => Imagick::FILTER_POINT, + 'box' => Imagick::FILTER_BOX, + 'triangle' => Imagick::FILTER_TRIANGLE, + 'hermite' => Imagick::FILTER_HERMITE, + 'hanning' => Imagick::FILTER_HANNING, + 'hamming' => Imagick::FILTER_HAMMING, + 'blackman' => Imagick::FILTER_BLACKMAN, + 'gaussian' => Imagick::FILTER_GAUSSIAN, + 'quadratic' => Imagick::FILTER_QUADRATIC, + 'cubic' => Imagick::FILTER_CUBIC, + 'catrom' => Imagick::FILTER_CATROM, + 'mitchell' => Imagick::FILTER_MITCHELL, + 'lanczos' => Imagick::FILTER_LANCZOS, + 'bessel' => Imagick::FILTER_BESSEL, + 'sinc' => Imagick::FILTER_SINC + ); + + // Set a default filter if an acceptable one wasn’t passed. + $filter = isset( $filter, $allowed_filters ) ? $allowed_filters[$filter] : Imagick::FILTER_TRIANGLE; + try { - /** - * @TODO: Thumbnail is more efficient, given a newer version of Imagemagick. - * $this->image->thumbnailImage( $dst_w, $dst_h ); + // Strip image meta. + if ( $strip_meta ) { + $this->strip_meta(); + } + + /* + * To be more efficient, resample large images to 5x the destination size before resizing whenever the output size + * is less that 1/3 of the original image size (1/3^2 ~= .111), unless we would be resampling to a scale smaller than 128x128. */ - $this->image->scaleImage( $dst_w, $dst_h ); + $resize_ratio = ( $dst_w / $this->size['width'] ) * ( $dst_h / $this->size['height'] ); + $sample_factor = 5; + + if ( $resize_ratio < .111 && ( $dst_w * $sample_factor > 128 && $dst_h * $sample_factor > 128 ) ) { + $this->image->sampleImage( $dst_w * $sample_factor, $dst_h * $sample_factor ); + } + + // Resize to the final output size. + $this->image->setOption( 'filter:support', '2.0' ); + $this->image->resizeImage( $dst_w, $dst_h, $filter, 1 ); + + // Set appropriate quality settings after resizing. + if ( 'image/jpeg' == $this->mime_type ) { + $this->image->unsharpMaskImage( 0.25, 0.25, 8, 0.065 ); + $this->image->setOption( 'jpeg:fancy-upsampling', 'off' ); + } + + if ( 'image/png' === $this->mime_type ) { + $this->image->setOption( 'png:compression-filter', '5' ); + $this->image->setOption( 'png:compression-level', '9' ); + $this->image->setOption( 'png:compression-strategy', '1' ); + $this->image->setOption( 'png:exclude-chunk', 'all' ); + } + + // If the alpha channel is not defined, make it opaque. + if ( $this->image->getImageAlphaChannel() === Imagick::ALPHACHANNEL_UNDEFINED ) { + $this->image->setImageAlphaChannel( Imagick::ALPHACHANNEL_OPAQUE ); + } + + $this->image->setImageDepth( 8 ); + $this->image->setInterlaceScheme( Imagick::INTERLACE_NO ); + $this->image->setColorspace( Imagick::COLORSPACE_SRGB ); } catch ( Exception $e ) { return new WP_Error( 'image_resize_error', $e->getMessage() ); } - - return $this->update_size( $dst_w, $dst_h ); } /** @@ -367,7 +440,7 @@ class WP_Image_Editor_Imagick extends WP_Image_Editor { if ( ! $dst_h ) $dst_h = $src_h; - $this->image->scaleImage( $dst_w, $dst_h ); + $this->$this->_resize( $dst_w, $dst_h ); return $this->update_size(); } } @@ -530,4 +603,49 @@ class WP_Image_Editor_Imagick extends WP_Image_Editor { return true; } + + /** + * Strip all image meta except color profiles from an image. + * + * @access public + * @since 4.5.0 + */ + public function strip_meta() { + try { + // Strip profiles. + foreach ( $this->image->getImageProfiles( '*', true ) as $key => $value ) { + if ( $key != 'icc' && $key != 'icm' ) { + $this->image->removeImageProfile( $key ); + } + } + + // Strip image properties. + if ( method_exists( $this->image, 'deleteImageProperty' ) ) { + $this->image->deleteImageProperty( 'comment' ); + $this->image->deleteImageProperty( 'Thumb::URI' ); + $this->image->deleteImageProperty( 'Thumb::MTime' ); + $this->image->deleteImageProperty( 'Thumb::Size' ); + $this->image->deleteImageProperty( 'Thumb::Mimetype' ); + $this->image->deleteImageProperty( 'software' ); + $this->image->deleteImageProperty( 'Thumb::Image::Width' ); + $this->image->deleteImageProperty( 'Thumb::Image::Height' ); + $this->image->deleteImageProperty( 'Thumb::Document::Pages' ); + } else { + $this->image->setImageProperty( 'comment', '' ); + $this->image->setImageProperty( 'Thumb::URI', '' ); + $this->image->setImageProperty( 'Thumb::MTime', '' ); + $this->image->setImageProperty( 'Thumb::Size', '' ); + $this->image->setImageProperty( 'Thumb::Mimetype', '' ); + $this->image->setImageProperty( 'software', '' ); + $this->image->setImageProperty( 'Thumb::Image::Width', '' ); + $this->image->setImageProperty( 'Thumb::Image::Height', '' ); + $this->image->setImageProperty( 'Thumb::Document::Pages', '' ); + } + } catch ( Excpetion $e ) { + return new WP_Error( 'image_strip_meta_error', $e->getMessage() ); + } + + return true; + } + } diff --git src/wp-includes/class-wp-image-editor.php src/wp-includes/class-wp-image-editor.php index 5b6f4ac..bb29617 100644 --- src/wp-includes/class-wp-image-editor.php +++ src/wp-includes/class-wp-image-editor.php @@ -492,4 +492,3 @@ abstract class WP_Image_Editor { return $extensions[0]; } } -