media-command icon indicating copy to clipboard operation
media-command copied to clipboard

Regenerating BIG Images Edited by User

Open ddur opened this issue 4 years ago • 1 comments

Bug Report

Test case 1, Step 1

As first step, just upload JPEG image that is larger than your current setting for 'big_image_size_threshold' (default is 2560 pixels). That would trigger automatic SCALING of image, on upload (see function wp_create_image_subsizes() in image.php). Make sure that your 'big_image_size_threshold' filter is not disabled or has value higher than your image pixel sizes.

Test case 2, Step 1

Another test would be to upload JPEG image with exif metadata [orientation] set to 2, 3 or 4. That would trigger automatic ROTATING of image on upload (see function wp_create_image_subsizes() in image.php), regardless of 'big_image_size_threshold'.

Further Test Steps for Test cases 1 & 2

Step 2: Edit the image (resize, rotate or flip etc) to get new metadata [sizes] set with edited images (where names are changed by new dimensions (WxH) and suffixing base-name with '-e' + 13 numeric characters).

Step 3: Regenerate the thumbnails.

Result: As far as I can read the code, edited image won't be attached anymore and in attachment metadata, edited thumbnails will be replaced with original thumbnails as created at upload.

Preconditions to be met:

This case is related only to NOT PNG ( == JPEG and maybe more?) type of images.

Original issue: https://github.com/Automattic/regenerate-thumbnails/issues/102

Attn: @azaozz

ddur avatar Aug 16 '20 12:08 ddur

Steps, in other words, from https://github.com/Automattic/regenerate-thumbnails/issues/102:

  • Upload a large JPG image. One dimension must be higher than 2560. It must be a JPG, not a PNG (it's possible that other formats have the same problem).
  • Edit the uploaded image (rotate or flip will make it most obvious).
  • Regenerate thumbnails for the image using this plugin.
  • After regenerating, the image in the Media Library is reverted to the pre-edited version.

swissspidy avatar Nov 10 '23 15:11 swissspidy