Changeset 59379 for trunk/src/wp-admin/includes/image.php

Timestamp:

11/10/2024 09:40:00 PM (5 months ago)

Author:

peterwilsoncc

Message:

Media: Remove dimension suffix from full size converted HEIC images.

Removes the dimension suffix, eg -1000x1000 from the file name of full size images automatically converted from HEIC to JPEGs by WordPress. Introduces unit tests for the default conversion of images and customized conversion settings via the image_editor_output_format filter.

Follow up to [58849], [58942], [59317], [59346], [59366]

Props mukesh27, peterwilsoncc, azaozz, apermo, flixos90, ironprogrammer.
Fixes #62359.
See #53645, #62305.

File:

• trunk/src/wp-admin/includes/image.php (modified) (1 diff)

trunk/src/wp-admin/includes/image.php

@@ -342 +342 @@
                 if ( $scale_down ) {
                     $saved = $editor->save( $editor->generate_filename( 'scaled' ) );
+                } elseif ( $convert ) {
+                    /*
+                     * Generate a new file name for the converted image.
+                     *
+                     * As the image file name will be unique due to the changed file extension,
+                     * it does not need a suffix to be unique. However, the generate_filename method
+                     * does not allow for an empty suffix, so the "-converted" suffix is required to
+                     * be added and subsequently removed.
+                     */
+                    $converted_file_name = $editor->generate_filename( 'converted' );
+                    $converted_file_name = preg_replace( '/(-converted\.)([a-z0-9]+)$/i', '.$2', $converted_file_name );
+                    $saved               = $editor->save( $converted_file_name );
                 } else {
                     $saved = $editor->save();