Changeset 51437 for branches/5.8

Timestamp:

07/15/2021 01:00:46 PM (3 years ago)

Author:

desrosj

Message:

Media: When resizing WebP images set the compression to "lossy" by default. Fixes a bug where the compression was set to "lossless" when the uploaded WebP images have extended file format (VP8X).

Props adamsilverstein, mikeschroder, mmxxi, linux4me2, azaozz.
Merges [51435] to the 5.8 branch.
Fixes #53653.

Location:

branches/5.8

Files:

• . (modified) (1 prop)
• src/wp-includes/class-wp-image-editor-imagick.php (modified) (1 diff)

branches/5.8/src/wp-includes/class-wp-image-editor-imagick.php

@@ -206 +206 @@
                     $webp_info = wp_get_webp_info( $this->file );
 
-                    if ( 'lossy' === $webp_info['type'] ) {
-                        $this->image->setImageCompressionQuality( $quality );
-                    } else {
+                    if ( 'lossless' === $webp_info['type'] ) {
                         // Use WebP lossless settings.
                         $this->image->setImageCompressionQuality( 100 );
                         $this->image->setOption( 'webp:lossless', 'true' );
+                    } else {
+                        $this->image->setImageCompressionQuality( $quality );
                     }
                     break;