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

Add cache configuration key documentation #265

Open
boesing opened this issue Jul 18, 2023 · 3 comments
Open

Add cache configuration key documentation #265

boesing opened this issue Jul 18, 2023 · 3 comments

Comments

@boesing
Copy link
Member

boesing commented Jul 18, 2023

Feature Request

Q A
Documentation yes

Summary

There is a cache config key which can be used for some kind of "default" cache adapter for a project.

$cacheConfig = $config['cache'] ?? [];

Not yet a clue where this is used for, might be an MVC specific thing 🤷🏼‍♂️

@froschdesign
Copy link
Member

@boesing
I find it difficult to explain to the user that they can use caches and also cache. Because already the first one leads to the desired result and also only works with a single cache. So the question is: Is this factory really needed?

@boesing
Copy link
Member Author

boesing commented Jan 19, 2024

So the question is: Is this factory really needed?

@froschdesign I am not sure, could be dropped in v4 and we can see how this adds up.
I'd also recommend to use some kind of StorageInterface::class => 'entry-in-caches-array', aliases entry instead if someone needs a default cache adapter as that would avoid having a redundant cache configuration laying around somewhere.

@froschdesign
Copy link
Member

Any simplification or removal of confusion is welcome and absolutely necessary. I also think that the improvements implemented here must then be transferred to other components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants