Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken Image Links in Media Library After Editing Offloaded Images #778

Open
kushh23 opened this issue Feb 20, 2024 · 3 comments
Open

Broken Image Links in Media Library After Editing Offloaded Images #778

kushh23 opened this issue Feb 20, 2024 · 3 comments
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer.
Milestone

Comments

@kushh23
Copy link

kushh23 commented Feb 20, 2024

Description

We have encountered an issue where editing offloaded images directly from the WordPress media library (e.g., rotating or cropping) causes the images to appear broken in the media library view. Additionally, these edited images do not get offloaded again as expected. If images are edited before being offloaded, everything functions correctly. However, editing images that have already been offloaded results in broken image links and strange URL patterns.

For example - https://succinctuse.s4-tastewp.com/wp-content/uploads/id:27906dd4c67ff11d6241c6c467ffbd58/https://succinctuse.s4-tastewp.com/IMG_7112-e1708458595987.jpg

  • This issue does not occur if the image is edited before being offloaded; it only happens when attempting to edit an image that has already been offloaded.
  • The appending of a string to the file name during the edit process might be interfering with the offloading process or the way the media library references the offloaded images.

This bug affects especially those users who need to make post-offload edits to images.

The user who reported this provided some insights and a solution but when I tried it didn't worked for me. I was facing the same issue with the filter they provided. Any insights or fixes for this issue would be greatly appreciated.

Step-by-step reproduction instructions

  1. Offload an image from the WordPress media library using our offloading functionality.
  2. Once the image is offloaded, attempt to edit the image (e.g., rotate or crop) using WordPress media edit options.
  3. Save the changes and return to the media library view.

Screenshots, screen recording, code snippet or Help Scout ticket

https://vertis.d.pr/v/CzCYYf
https://secure.helpscout.net/conversation/2514329513/405110/

Environment info

No response

Is the issue you are reporting a regression

No

@kushh23 kushh23 added the bug This label could be used to identify issues that are caused by a defect in the product. label Feb 20, 2024
@pirate-bot pirate-bot added the customer report Indicates the request came from a customer. label Feb 20, 2024
@girishpanchal30
Copy link
Contributor

Hey @kushh23,

I reviewed this issue, and I found that the WP default image editor does not work if we select the Optimole Cloud only setting.
Ref: https://tinyurl.com/yuh3wkla

Once we select this option, images will be saved on the Optimole Cloud server only, and the WP default image editor needs a local image path to edit an image.

Solution

We can redirect to the Optimole Libary page when use click on the edit link if the Optimole Cloud only setting is enabled.

cc @selul

Thanks

@selul
Copy link
Contributor

selul commented Aug 27, 2024

@girishpanchal30 can't we do something to use the Optimole image on edit and the processing option? Maybe we custom editor class? would be a complex thing to do?

@girishpanchal30
Copy link
Contributor

@selul I noticed that Optimole provides an image editor within an iframe that loads all images. Is there any way to support loading a single image in the iframe using a specific URL?

If it is possible we could explore options to replace the WordPress image editor with the Optimole image editor.

Thanks

@selul selul added this to the 4.0 milestone Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer.
Projects
None yet
Development

No branches or pull requests

4 participants