You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Upload two identical images with a width of less than 480 (the default width to trigger a resize) in to a course. If you look at the image sources, imageopt won't have processed them.
Purging caches causes everything to start working, so it should be a problem with the image not being added to the public files cache when it's first uploaded, then when the second one is uploaded, the image is still considered not public because it's not in the cache.
We need to figure out a way to invalidate the cache when the image is uploaded the second time.
The text was updated successfully, but these errors were encountered:
I'm not super fussed by this bug. In practice we will want to set the min fairly high, say 20 or even higher. And this will be very slow moving data so it's not the end of the world if it doesn't get a proper hit for a week and then we get it for free after a security patch or some unrelated cache purge.
One fix would be to invalidate the cache every time an image file was uploaded by using a file hook.
There could be some edge cases where this might affect performance though - we'd need to think about it carefully.
Set up the plugin with the min duplicates = 2
Upload two identical images with a width of less than 480 (the default width to trigger a resize) in to a course. If you look at the image sources, imageopt won't have processed them.
Purging caches causes everything to start working, so it should be a problem with the image not being added to the public files cache when it's first uploaded, then when the second one is uploaded, the image is still considered not public because it's not in the cache.
We need to figure out a way to invalidate the cache when the image is uploaded the second time.
The text was updated successfully, but these errors were encountered: