wp_get_missing_image_subsizes can return incorrect subsizes for a rotated image

[ianmjones]

During the upload of a rotated image to the Media Library, the wp_get_missing_image_subsizes() function often returns missing subsizes for the non-rotated original image that will never be generated for the rotated image.

This means for example, if a plugin is filtering wp_update_attachment_metadata and waiting for all thumbnails to have been generated before performing some action, it may be told that there are missing thumbnails for a rotated image, that will not be generated. Therefore it will not be able to perform that action as not all the thumbnails are deemed to be available.

During the upload of a rotated image, the wp_get_missing_image_subsizes() function currently incorrectly uses the dimensions of the non-rotated original image to determine which subsizes should exist.

This changeset adds a condition that the original image should only be used for setting the image dimensions if the orientation of the Media Library item has not been changed.

[azaozz]

Moving this to 6.4 as it needs a patch.

[marybaum]

Asking @antpb if maybe the EXIF data import should not include the rotation data.
Then we can consider an acceptable patch.

[nicolefurlan]

@antpb just checking in on this ticket. Could you review #comment:10?

[antpb]

with this needing a rework on the solution to account for the suggestion @ianmjones made we should move to 6.5

From @ianmjones : "To properly determine whether the original image can be used to generate missing thumbnail sizes, we probably need to grab its width vs. height ratio and compare with the full size image's ration. If they're the same, then yay, we can use the original and maybe pick up a couple extra sizes, but otherwise, stick with the full size image."

[audrasjb]

As per today's bug scrub, we decided to move this ticket to Future Release to give it more time to get a patch.