-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
Use Apcu_fetch instead of apc_fetch #16830
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
Comments
The only non-deprecated class using APC is Then it's only a matter of also adding a |
Good idea to deprecate this and use Doctrine Cache instead. There already is a PR on this for Doctrine Cache. |
I don't see any mention of the classloader in #16822, and for me at least, it's important to keep the classloader and actual cache systems separate - the classloader is required long before the container is available. |
Fixed by #17358 |
Since minimum supported PHP for Symfony 3 is 5.5.9, maybe time to use apcu_fetch instead of apc_fetch. This way no need to enable APC backward compatibility mode with Apcu as APC does not exist for PHP 5.5 anyway. Especially with PHP 7, APC backward compatibility in Apcu is not enabled by default.
The text was updated successfully, but these errors were encountered: