Original file name changes on images upload and extra unregistered image sizes generated

[Iulia Cazan]

Hello,

I installed and tested the functionality provided with version 5.3-beta2 and there is an issue with images uploads. Basically, when uploading an image, the filename that is recorded in the attachment metadata for the original file is using a suffix that indicates the original image width, and the file is created as well.

For example, I uploaded the image called image.jpeg (with 5472×3648px) and the result looks like this:
I end up with the following images in the folder:

image-1024x683.jpeg - matches the large size - OK
image-1200x800.jpeg - matches the post-thumbnail size - OK
image-150x150.jpeg - matches the thumbnail size - OK
image-1536x1024.jpeg - 1536x1536 - as far as I know, this should not be generated, there is no image size registered programmatically for this
image-1980x1320.jpeg - matches the twentytwenty-fullscreen - OK
image-2048x1365.jpeg - 2048x2048 - as far as I know, this should not be generated, there is no image size registered programmatically for this
image-2560.jpeg - this is mapped as the original file, but this should be image.jpeg
image-300x200.jpeg - matches the medium size - OK
image.jpeg - this is the original file but is mapped separately

And in the database the metadata contains these:

a:6:{s:5:"width";i:2560;s:6:"height";i:1707;s:4:"file";s:23:"2019/10/image-2560.jpeg";s:5:"sizes";a:8:{s:6:"medium";a:4:{s:4:"file";s:18:"image-300x200.jpeg";s:5:"width";i:300;s:6:"height";i:200;s:9:"mime-type";s:10:"image/jpeg";}s:5:"large";a:4:{s:4:"file";s:19:"image-1024x683.jpeg";s:5:"width";i:1024;s:6:"height";i:683;s:9:"mime-type";s:10:"image/jpeg";}s:9:"thumbnail";a:4:{s:4:"file";s:18:"image-150x150.jpeg";s:5:"width";i:150;s:6:"height";i:150;s:9:"mime-type";s:10:"image/jpeg";}s:12:"medium_large";a:4:{s:4:"file";s:18:"image-768x512.jpeg";s:5:"width";i:768;s:6:"height";i:512;s:9:"mime-type";s:10:"image/jpeg";}s:9:"1536x1536";a:4:{s:4:"file";s:20:"image-1536x1024.jpeg";s:5:"width";i:1536;s:6:"height";i:1024;s:9:"mime-type";s:10:"image/jpeg";}s:9:"2048x2048";a:4:{s:4:"file";s:20:"image-2048x1365.jpeg";s:5:"width";i:2048;s:6:"height";i:1365;s:9:"mime-type";s:10:"image/jpeg";}s:14:"post-thumbnail";a:4:{s:4:"file";s:19:"image-1200x800.jpeg";s:5:"width";i:1200;s:6:"height";i:800;s:9:"mime-type";s:10:"image/jpeg";}s:23:"twentytwenty-fullscreen";a:4:{s:4:"file";s:20:"image-1980x1320.jpeg";s:5:"width";i:1980;s:6:"height";i:1320;s:9:"mime-type";s:10:"image/jpeg";}}s:10:"image_meta";a:12:{s:8:"aperture";s:1:"0";s:6:"credit";s:0:"";s:6:"camera";s:0:"";s:7:"caption";s:0:"";s:17:"created_timestamp";s:1:"0";s:9:"copyright";s:0:"";s:12:"focal_length";s:1:"0";s:3:"iso";s:1:"0";s:13:"shutter_speed";s:1:"0";s:5:"title";s:0:"";s:11:"orientation";s:1:"0";s:8:"keywords";a:0:{}}s:14:"original_image";s:10:"image.jpeg";}

My original test image.jpeg has 5472×3648 and the image that is now linked as the "original" image-2560.jpeg has been scaled and is not the original image. Basically, there is no real information about the actual file that was uploaded (the width and height are not mapped in the database at all). I confirm that my tests are done on a clean WordPress instance, with no custom settings and with the Twenty Twenty theme activated.

If this is not a bug, but the intended new functionality, then this is quite a change and needs some documentation.

As far as I understand, we will end up with a lot of hidden images (that will only pile up without ever being used), since most of the functionalities related to media are handling media attributes bases on the image sizes registered programmatically and readable/known image sizes names. Also, it's going to decrease the performance on upload, since there are extra image sizes that will be generated/handled per each file.

Regards,
Iulia

[sotiris_k]

I have reproduced this bug successfully. It does tend to happen with high resolution images.

[desrosj]

Related #40439.

Thanks, @iulia-cazan! Welcome to Trac!

This is related to some changes coming to the image upload process in 5.3. I’m moving to the milestone for visibility, but this may be the new intended behavior.

The meta value you attached looks like the one that keeps track of all subsizes generated for an image. Can you see if there is an additional meta field that contains the original image file name?

[Iulia Cazan]

I understand.

However, @desrosj, do you have an explanation for the image sizes that are generated outside of the registered image sizes? That is a red flag for me, regardless of the intended new functionality.

[desrosj]

The 2048x2048 and 1536x1536 are new sizes introduced in [46066] in an effort to enhance the way WordPress displays images on the front-end on larger, high-density devices. They make it possible to generate more suitable srcset and sizes attributes, and not use the original, often non-optimized image.

These new sizes will be generated for all newly uploaded images going forward.

I am seeing the original image in the meta data that you supplied in the original_image index, so it seems to be working as intended.

• The original image is larger than the new big image threshold used to cap images at an upper size limit.
• The original is scaled down to be more useful using the two new image sizes.
• The original is preserved in the attachment's meta data.

If you wish to disable this big image threshold (or tweak it to your liking), you can use the big_image_size_threshold filter to return a new size. Returning false will disable this.

I'm goign to close this out because it appears to be working as intended. If I am misinterpreting the questions, or you have more, feel free to reopen to discuss more.

[Iulia Cazan]

OK. Thank you!

[desrosj]

Also, I forgot to add! @iulia-cazan keep an eye out on ​https://make.wordpress.org/core for a developer note detailing all of the changes that were made here! A few small things were being tested and maybe tweaked, so that's why it's a bit delayed being published. Please do follow up there if you have any further questions!

[bbtodd]

Replying to desrosj:

If you wish to disable this big image threshold (or tweak it to your liking), you can use the big_image_size_threshold filter to return a new size. Returning false will disable this.

Did I misunderstand? You say returning false will disable this. I tried using this code and it has no effect.

add_filter( 'big_image_size_threshold', '__return_false' );

Serious issue for me, I created a new ticket for this here:
https://core.trac.wordpress.org/ticket/49161#ticket

[Iulia Cazan]

Replying to desrosj:

Also, I forgot to add! @iulia-cazan keep an eye out on ​https://make.wordpress.org/core for a developer note detailing all of the changes that were made here! A few small things were being tested and maybe tweaked, so that's why it's a bit delayed being published. Please do follow up there if you have any further questions!

Thanks, @desrosj! I have been keeping an eye on this, and I understood that the changes are permanent and how the upload and image processing is intended to work going further, and I already made the necessary changes in my plugin so that this change would not affect the users of my plugin.

[peterwilsoncc]

@bbtodd

Thank you for opening #49161 to discuss the large image size generation even with the filter disabled. I've reversed your status change on this ticket so the related issue can be discussed on the new ticket.