Switch php_admin_value to php_value to allow overwrites by ini_set() in fpm.confs

[marcorieser]

Started to use Herd yesterday and booted up a WordPress site. The debug.log was not generated at the usual path so I started to search the internet and found an issue (#134 (comment)) here which suggests to comment out php_admin_value[error_log] in the fpm.conf file.

Tinkered a bit and found out that replacing php_admin_value with php_value allows WordPress to overwrite the error log path via set_ini() while keeping the error_log Herd path set as default.

So I was wondering why php_admin_value was chosen over php_value at all?

I'm no server guy and maybe missing something but I'm suggesting to use php_value in the future.