Media Gallery - Clicking "Restore Original Image" in "Scale Image" pane loses 'Thumbnail Settings' pane.

[gr33nman]

Reproduce the problem thusly:

Click "Edit image" in the "Edit Media" interface for any image.

/wp-admin/post.php?post=1119&action=edit

Scale the image a couple times in the 'Scale Image' pane. Update.

Click "Restore Original Image" in the 'Scale Image' pane.

Try to crop just the thumbnail.

The 'Thumbnail Settings' Pane is gone.

The image has to be deleted and re-uploaded to gain thumbnail control once again.

[markoheijnen]

Only tested in 3.4 and can reproduce it there as well.

[mboynes]

Patch for restore image bug #23398

[mboynes]

I added a patch that resolves this, however I'm uncertain of any potential repercussions.

The issue here is that wp_restore_image is only setting meta for image sizes that have been edited (and thus have an entry in the _wp_attachment_backup_sizes postmeta for the image). For instance, if you crop the image and apply to all image sizes, then go through the steps to reproduce this error, you won't replicate it. If you only scale the image, the only image size you edit is the original, therefore the only backup meta data is for the original and wp_restore_image unsets the other sizes in _wp_attachment_metadata.

As far as I can tell, unsetting the non-edited sizes is unnecessary. I cannot think of a use-case scenario where this would need to be done.

This patch will only fix this issue for images that are scaled after applying the patch. Images scaled prior to applying the patch will need to have their other image sizes regenerated via a plugin like ​Regenerate Thumbnails.

[markjaquith]

Would like a better explanation of what is going on to feel confident about the way to fix it.

[lkraav]

Yep, same on 4.4.2 and this definitely made for a wacky user experience. No way to understand why suddenly can't get back to the starting point, until you notice the uuid added to the original image's name, meaning it's not the original any longer.

"Restore original image" therefore lies.

Is there someone currently hacking the media library and is in enough of a flow to say something meaningful about the next step options here?