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

Bug: Database cache not working #3597

Closed
MichalPB1 opened this issue Sep 3, 2020 · 3 comments · Fixed by #3608
Closed

Bug: Database cache not working #3597

MichalPB1 opened this issue Sep 3, 2020 · 3 comments · Fixed by #3608
Labels
bug Verified issues on the current code behavior or pull requests that will fix them

Comments

@MichalPB1
Copy link

I not sure if a database cache is implemented in CI 4. I have sets cache on true and cacheDir for an already exists directory but still not working.

I looked for using that configuration in the system and I found only a definition of these variables in system/Database/BaseConnection.php but nothing more.

Context

  • OS: Linux
  • Web server PHP local
  • PHP version 7.3
@MichalPB1 MichalPB1 added the bug Verified issues on the current code behavior or pull requests that will fix them label Sep 3, 2020
@michalsn
Copy link
Member

michalsn commented Sep 3, 2020

Sorry, but there is no cache built-in for the database layer and definitely there won't be any. Please use the Cache class instead.

But you have a fair point here - we probably should clean this up as these are leftovers from the CI3 rewrite. I think I will leave this open just so we don't forget about this.

@MichalPB1
Copy link
Author

Ok, so docs should be also fixed.

Can I found some documentation of using Cache for Model?

@michalsn
Copy link
Member

michalsn commented Sep 5, 2020

Basically there is no difference between using the Cache class in or outside the model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Verified issues on the current code behavior or pull requests that will fix them
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants