-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix #10427 - APC does not work for SuiteCRM 7.14, PHP8 #10428
base: hotfix
Are you sure you want to change the base?
Conversation
This pull request has been mentioned on SuiteCRM. There might be relevant details there: https://community.suitecrm.com/t/apc-cache-not-working/69261/2 |
Hi Gunnicom, Thanks for contributing to SuiteCRM. I have tested your changes and they work well. Having looked at the code, there are some related changes that may be worthwhile.
The check In addition it may be worth lowering the priority in |
@simonr44 At first I thought about the file include/SugarCache/SugarCache.php, too, but on second look I assumed, that that is only for OPcache part of APC, that vanished long ago. So I decided to not include that here. |
Agreed yes APCu no longer includes opcode caches, however clearing the APCu cache when this function is called may still have benefits. Perhaps the function name is no longer too relevant but it may still have value in one form or another. |
On further inspection
|
@simonr44 That could all be discussed in a different issue/PR. Keep this PR to fixing APCu functionality, and leave cleanup of old obsolete APC OPcache to a different PR. |
There is a good chance we would look to bring these changes in together and test together so we'll make a new PR to cover those. Thanks again for your contribution. |
See #10427