feat: additional opcache setting in check PHP.ini

Author: ddevsr

system/Security/CheckPhpIni.php

@@ -118,26 +118,33 @@ private static function outputForWeb(array $output, array $thead, array $tbody):
     public static function checkIni(): array
     {
         $items = [
-            'error_reporting'         => ['recommended' => '5111'],
-            'display_errors'          => ['recommended' => '0'],
-            'display_startup_errors'  => ['recommended' => '0'],
-            'log_errors'              => [],
-            'error_log'               => [],
-            'default_charset'         => ['recommended' => 'UTF-8'],
+            'error_reporting'                 => ['recommended' => '5111'],
+            'display_errors'                  => ['recommended' => '0'],
+            'display_startup_errors'          => ['recommended' => '0'],
+            'log_errors'                      => [],
+            'error_log'                       => [],
+            'default_charset'                 => ['recommended' => 'UTF-8'],
             'max_execution_time'      => ['remark' => 'The default is 30.'],
-            'memory_limit'            => ['remark' => '> post_max_size'],
-            'post_max_size'           => ['remark' => '> upload_max_filesize'],
-            'upload_max_filesize'     => ['remark' => '< post_max_size'],
+            'memory_limit'                    => ['remark' => '> post_max_size'],
+            'post_max_size'                   => ['remark' => '> upload_max_filesize'],
+            'upload_max_filesize'             => ['remark' => '< post_max_size'],
             'max_input_vars'          => ['remark' => 'The default is 1000.'],
-            'request_order'           => ['recommended' => 'GP'],
-            'variables_order'         => ['recommended' => 'GPCS'],
-            'date.timezone'           => ['recommended' => 'UTC'],
-            'mbstring.language'       => ['recommended' => 'neutral'],
-            'opcache.enable'          => ['recommended' => '1'],
-            'opcache.enable_cli'      => [],
-            'opcache.jit'             => [],
-            'opcache.jit_buffer_size' => [],
+            'request_order'                   => ['recommended' => 'GP'],
+            'variables_order'                 => ['recommended' => 'GPCS'],
+            'date.timezone'                   => ['recommended' => 'UTC'],
+            'mbstring.language'               => ['recommended' => 'neutral'],
+            'opcache.enable'                  => ['recommended' => '1'],
+            'opcache.enable_cli'              => [],
+            'opcache.jit'                     => [],
+            'opcache.jit_buffer_size'         => [],
             'zend.assertions'         => ['recommended' => '-1'],
+            'opcache.memory_consumption'      => ['recommended' => '728', 'remark' => 'Increasing the configured memory size (MB) will improve performance by caching those files'],
+            'opcache.interned_strings_buffer' => ['recommended' => '64'],
+            'opcache.max_accelerated_files'   => ['recommended' => '56000', 'remark' => 'Increasing the configured accelerated will many cache files into OPCache hash table'],
+            'opcache.max_wasted_percentage'   => ['recommended' => '15'],
+            'opcache.validate_timestamps'     => ['recommended' => '0'],
+            'opcache.revalidate_freq'         => ['recommended' => '0'],
+            'opcache.save_comments'           => ['recommended' => '1'],
         ];
 
         $output = [];

tests/system/Security/CheckPhpIniTest.php

@@ -37,6 +37,19 @@ public function testCheckIni(): void
         $this->assertSame($expected, $output['display_errors']);
     }
 
+    public function testCheckIniOpcache(): void
+    {
+        $output = CheckPhpIni::checkIni();
+
+        $expected = [
+            'global'      => '',
+            'current'     => '1',
+            'recommended' => '1',
+            'remark'      => '',
+        ];
+        $this->assertSame($expected, $output['opcache.save_comments']);
+    }
+
     public function testRunCli(): void
     {
         // Set MockInputOutput to CLI.