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

remove APC support #304

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

remove APC support #304

wants to merge 2 commits into from

Conversation

stklcode
Copy link
Contributor

@stklcode stklcode commented Aug 6, 2024

APC is only supported up to PHP 5.5. As Cachify now requires PHP 5.6 or later, there is no point in maintaining legacy APC support.

We also introduce an admin notice, if the configured backend is not available. This applies to all backends except DB.

cachify_notice_apc

@stklcode stklcode mentioned this pull request Aug 6, 2024
6 tasks
@stklcode stklcode force-pushed the remove/apc branch 4 times, most recently from 575dff6 to 3253b8f Compare August 6, 2024 16:36
APC is only supported up to PHP 5.5. As Cachify now requires PHP 5.6 or
later, there is no point in maintaining legacy APC support.

We already ignore unavailable backends silently with fallback to DB, so
just removing it does not make it worse here.
stklcode added a commit that referenced this pull request Aug 6, 2024
APC is only supported up to PHP 5.5. As Cachify now requires PHP 5.6 or
later, there is no point in maintaining legacy APC support.

We already ignore unavailable backends silently with fallback to DB, so
just removing it does not make it worse here.
We should not silently ignore misconfiguration and fall back to
DB caching. We now check for availability and disable caching
via a "no operation" backend. We can also add a notice for APC
now which is no longer supported.
@stklcode stklcode force-pushed the remove/apc branch 2 times, most recently from f45e3f9 to 70a629d Compare August 6, 2024 18:16
Copy link

sonarcloud bot commented Aug 6, 2024

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

Successfully merging this pull request may close these issues.

Show a notice, if the configured caching method is no longer available
1 participant