1up-lab / OneupFlysystemBundle

A Flysystem integration for your Symfony projects.
MIT License
630 stars 119 forks source link

Remove cache property from documentation #276

Closed simoheinonen closed 1 year ago

bytehead commented 1 year ago

Why?

simoheinonen commented 1 year ago

Why what?

bytehead commented 1 year ago

Why is there no cache anymore?

simoheinonen commented 1 year ago

As far as I understand it was removed in 4.0 but this documentation wasn't changed

bytehead commented 1 year ago

Ah, didn't get the point :) I'll have a look, thanks!

simoheinonen commented 1 year ago

Wish there was cache feature again though :(

bytehead commented 1 year ago

The cache feature was actually removed in the underlaying flysystem library: https://flysystem.thephpleague.com/docs/upgrade-from-1.x/#miscellaneous-changes The cached adapter was not ported to V2.

bytehead commented 1 year ago

Did you https://github.com/Lustmored/flysystem-v2-simple-cache-adapter give a try?

simoheinonen commented 1 year ago

Yeah, there's https://github.com/Lustmored/flysystem-v2-simple-cache-adapter but not sure if that one would be mature enough to add to this bundle?