Changeset 59247 for trunk/src/wp-includes/class-wp-image-editor-imagick.php

Timestamp:

10/17/2024 05:03:23 PM (14 months ago)

Author:

adamsilverstein

Message:

Media: Ensure Imagick quality is set correctly.

Fix an issue where Imagick did not properly set image quality for AVIF images. Adds a call to setCompressionQuality where we currently call setImageCompressionQuality

Props: jamesosborne, adamsilverstein, mukeshpanchal27.

Fixes: #61614.

File:

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

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

@@ -207 +207 @@
                 case 'image/jpeg':
                     $this->image->setImageCompressionQuality( $quality );
+                    $this->image->setCompressionQuality( $quality );
                     $this->image->setImageCompression( imagick::COMPRESSION_JPEG );
                     break;
@@ -215 +216 @@
                         // Use WebP lossless settings.
                         $this->image->setImageCompressionQuality( 100 );
+                        $this->image->setCompressionQuality( 100 );
                         $this->image->setOption( 'webp:lossless', 'true' );
                         parent::set_quality( 100 );
                     } else {
                         $this->image->setImageCompressionQuality( $quality );
+                        $this->image->setCompressionQuality( $quality );
                     }
                     break;
@@ -225 +228 @@
                     $this->image->setOption( 'heic:speed', 7 );
                     $this->image->setImageCompressionQuality( $quality );
+                    $this->image->setCompressionQuality( $quality );
                     break;
                 default:
                     $this->image->setImageCompressionQuality( $quality );
+                    $this->image->setCompressionQuality( $quality );
             }
         } catch ( Exception $e ) {