Recommendation for php.ini mistake in nc 12 beta 2

[jospoortvliet]

The admin screen warns about unoptimized php settings, but the first value mentioned seems broken:
php_value opcache.enable=on
That should, I believe, be:
php_value opcache.enable=1

This is Nextcloud 12 Beta 2.

See:

[ghost]

yes: nextcloud/documentation#450
Do i get a t-shirt please?!?
i would really really appreciate: need XL-shirt

[MorrisJobke]

php_value opcache.enable=on

I did this by intention. I set it to 1 on PHP 5.6 and 7.0. In one of them the opcache was not activated. With on it worked fine. Even if the PHP documentation says something different I would stick to that, because I ran into issues with 1 which was fixed with the on.

[MorrisJobke]

That should, I believe, be:

Never believe and always test ;)

[ghost]

1

is working properly for php 7.1.
i maintain a lot of server running with that value as i wrote here and on my blog.
additionally on shared server hosted by all-inkl.com and 1&1.

...:::carsten

[MorrisJobke]

i maintain a lot of server running with that value as i wrote here and on my blog.
additionally on shared server hosted by all-inkl.com and 1&1.

Is this always PHP 7.1 and PHP 7.0 or also older PHP?

[ghost]

PHP 7 and 7.1
I do not have an environment < PHP7 anymore. So to be honest - i cannot state.

[MorrisJobke]

I do not have an environment < PHP7 anymore. So to be honest - i cannot state.

Okay ... let me try to find one and test this again.

[JSoko]

the (wrong) hint with opcache.enable=on persists in NC12RC4...

[MorrisJobke]

Fix is in #5014 and #5015