Changeset 59589 for trunk/tests/phpunit/tests/image/editorImagick.php

Timestamp:

01/07/2025 10:33:37 PM (17 months ago)

Author:

adamsilverstein

Message:

Media: improve Imagick handling of colors and alpha channel for PNG image uploads.

Fix an issue where index color (8 bit) PNG uploads were output as true color (24 bit) PNGs, significantly increasing their size. When using Imagick, PNG output images will now match the colors of the uploaded image.

Also, correct handling of PNG alpha channel information so it is preserved in output images.

Props adamsilverstein, pbearne, nosilver4u, peterdavehello, joemcgill, azaozz, codex-m, kirasong, justlevine, jokanane, sallyruchman, wpfed, tgsrvrs, antpb, tb1909.
Fixes #36477.

File:

• trunk/tests/phpunit/tests/image/editorImagick.php (modified) (1 diff)

trunk/tests/phpunit/tests/image/editorImagick.php

@@ -759 +759 @@
         return 8;
     }
+
+    /**
+     * Test that resizes are smaller for 16 bit PNG images.
+     *
+     * @ticket 36477
+     *
+     * @dataProvider data_resizes_are_small_for_16bit_images
+     */
+    public function test_resizes_are_small_for_16bit_images( $file ) {
+
+        $temp_file = DIR_TESTDATA . '/images/test-temp.png';
+
+        $imagick_image_editor = new WP_Image_Editor_Imagick( $file );
+        $imagick_image_editor->load();
+        $size = $imagick_image_editor->get_size();
+
+        $org_filesize = filesize( $file );
+
+        $imagick_image_editor->resize( $size['width'] * .5, $size['height'] * .5 );
+
+        $saved = $imagick_image_editor->save( $temp_file );
+
+        $new_filesize = filesize( $temp_file );
+
+        unlink( $temp_file );
+
+        $this->assertLessThan( $org_filesize, $new_filesize, 'The resized image file size is not smaller than the original file size.' );
+    }
+
+    /**
+     * data_test_resizes_are_small_for_16bit
+     *
+     * @return array[]
+     */
+    public static function data_resizes_are_small_for_16bit_images() {
+        return array(
+            'cloudflare-status'       => array(
+                DIR_TESTDATA . '/images/png-tests/cloudflare-status.png',
+            ),
+            'deskcat8'                => array(
+                DIR_TESTDATA . '/images/png-tests/deskcat8.png',
+            ),
+            '17-c3-duplicate-entries' => array(
+                DIR_TESTDATA . '/images/png-tests/Palette_icon-or8.png',
+            ),
+            'rabbit-time-paletted'    => array(
+                DIR_TESTDATA . '/images/png-tests/rabbit-time-paletted-or8.png',
+            ),
+            'test8'                   => array(
+                DIR_TESTDATA . '/images/png-tests/test8.png',
+            ),
+        );
+    }
 }