Problem/Motivation

It seems questionable to replace the original file extension with webp instead of amending it like the ImageAPI Optimize WebP. It's not trivial to find the original source of the WebP image derivative. This makes it complicated to debug and hard to properly integrate with modules that need access to the original image, e.g. the Stage File Proxy module (see #3326961: After update to version >1.2 no image mirroring).

So this issue is mainly to review the reasoning behind the current approach and to see if a change could and should be done.

CommentFileSizeAuthor
#5 3328040-webp-filename-logic-example-5.patch12.18 KBberliner
#2 3328040-webp-filename-logic-example-2.patch12.6 KBberliner

Comments

berliner created an issue. See original summary.

berliner’s picture

berliner commented
StatusFileSize
new 3328040-webp-filename-logic-example-2.patch12.6 KB

The following patch might serve as a basis for discussion.

berliner’s picture

berliner commented
Related issues: +#3281606: Rendering duplicate images when the original images have the same name but different extension

Also related to or possible duplicate of #3281606: Rendering duplicate images when the original images have the same name but different extension.

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented
Status: Active » Closed (duplicate)

I cleaned up your code from out-of-scope things then made it a MR on the other issue that's more likely to be tested.
Thank you for leading the way!

To be continued here: #3281606: Rendering duplicate images when the original images have the same name but different extension

berliner’s picture

berliner commented
StatusFileSize
new 3328040-webp-filename-logic-example-5.patch12.18 KB

Re-rolled for current dev.