Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Prevent warnings when invalidating the cache #8335

Closed
wants to merge 1 commit into from
Closed

Prevent warnings when invalidating the cache #8335

wants to merge 1 commit into from

Conversation

rolandschuetz
Copy link
Contributor

Easyname uses OPcache, but restricts the opcache_invalidate() usage.

Therefore Easyname Hosting the following warning in thrown:

Warning: Zend OPcache API is restricted by "restrict_api" configuration directive in system/modules/rocksolid-custom-elements/src/MadeYourDay/Contao/CustomElements.php on line 1262

The customer support write:

Das Problem ist das die Funktionen ja existieren (opcache_invalidate), aber es kann leider kein Zugriff auf diese Funktionen stattfinden da der opcache von allen Kunden am Webserver geteilt wird.

@ausi
Copy link
Member

ausi commented May 11, 2016

I commented in madeyourday/contao-rocksolid-custom-elements#66 regarding the same issue:

I think this is a problem of Easyname. They should either remove the method completely so that function_exists('opcache_invalidate') returns false or add the option to disable the warning messages.

@leofeyer
Copy link
Member

I second @ausi's opinion.

@Toflar
Copy link
Member

Toflar commented May 11, 2016

I second @ausi's opinion too.

@rolandschuetz
Copy link
Contributor Author

Yeah, I think too it sucks. Just wasn't sure how else to handle it.
So guess I need to adopt the local Contao core for this hoster :(

@ausi
Copy link
Member

ausi commented May 11, 2016

You could point your hoster to this discussion and ask them if they could implement one of the two proposed solutions.

@leofeyer leofeyer closed this May 11, 2016
@rolandschuetz
Copy link
Contributor Author

rolandschuetz commented May 11, 2016

Yeah, I don't think they will change that soon.
Thanks for the replys!

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

Successfully merging this pull request may close these issues.

None yet

4 participants