Concrete5: 8.3.RC1 - Thumbnails created by galleries still get deleted when cache is cleared

[africanforest]

When the cache is cleared with unchecked "Clear cached thumbnails", all thumbnails created by galleries are also gone. This is because thumbnails created by galleries still are stored in

application/files/cache

instead of

application/files/cache/thumbnails

As a consequence, it can take ages for a gallery page to load for the first time after the cache has been cleared, because all thumbnails have to be rebuild.

Should be fixed by PR #5900, but as it seems, these fixes won't find the way into 8.3.?

[africanforest]

It works with thumbnails created with asynchronious thumbnail creation mode enabled, but thumbnails processing took 10 to 15 seconds per thumbnail (tested on Fluid Gallery).
Interestingly, it seems that the longer the page is loading, while thumbnails are processed, the faster it gets.

I did other tests with the latest version of Touch Gallery: While i'm seeing thumbnail creation working as expected for a fileset containing 9 images in a reasonable time, i have no luck at all with a fileset of 180 images: thumbnails just aren't created, even if it looks like they would be created in the background in edit mode, but nothing happens, and even after 20 minutes no thumbnail is created in application/files/cache/thumbnails, while in the meantime the gallery page has become inaccessible on the frontend side (loads forever until timeout).

[aembler]

That particular pull request is held up because it's not backward compatible – it'll result in a large amount of broken thumbnails on any site that upgrades. We'd accept a pull request that just fixed the application/files/cache/thumbnails issue.

[mlocati]

PS: next week I shod have time to work on #5900