Allow setting job error_reporting level via config

otsch · Aug 3, 2024 · 3ce4ca0 · 3ce4ca0
1 parent 823e2b6
commit 3ce4ca0
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 4 deletions.
diff --git a/src/Kernel.php b/src/Kernel.php
@@ -31,13 +31,18 @@ public function __construct(
         $this->logger = new EchoLogger();
     }
 
-    public static function ppqCommand(string $command, ?string $logPath = null): SymfonyProcess
-    {
+    public static function ppqCommand(
+        string $command,
+        ?string $logPath = null,
+        ?string $iniConfigOption = null,
+    ): SymfonyProcess {
         if ($logPath) {
             touch($logPath);
         }
 
-        $command = 'php ' . self::ppqPath() . ' ' . $command . ' --config=' . Config::getPath();
+        $iniConfigOption = $iniConfigOption ? '-d ' . $iniConfigOption . ' ' : '';
+
+        $command = 'php ' . $iniConfigOption . self::ppqPath() . ' ' . $command . ' --config=' . Config::getPath();
 
         if ($logPath) {
             $command .= ' >> ' . $logPath . ' 2>&1';

diff --git a/src/Queue.php b/src/Queue.php
@@ -50,7 +50,11 @@ public function hasAvailableSlot(): bool
      */
     public function startWaitingJob(QueueRecord $waitingJob): void
     {
-        $process = Kernel::ppqCommand('run ' . $waitingJob->id, Logs::queueJobLogPath($waitingJob));
+        $process = Kernel::ppqCommand(
+            'run ' . $waitingJob->id,
+            Logs::queueJobLogPath($waitingJob),
+            Config::get('error_reporting') ? 'error_reporting=' . Config::get('error_reporting') : '',
+        );
 
         $process->start();
 

diff --git a/tests/_testdata/config/error-handler-ignore-warnings.php b/tests/_testdata/config/error-handler-ignore-warnings.php
@@ -11,6 +11,8 @@
         ],
     ],
 
+    'error_reporting' => E_ALL,
+
     'error_handler' => [
         'class' => ErrorHandlerIgnoreWarnings::class,
         'active' => true,

diff --git a/tests/_testdata/config/error-handlers.php b/tests/_testdata/config/error-handlers.php
@@ -11,6 +11,8 @@
         ],
     ],
 
+    'error_reporting' => 'E_ALL',
+
     'error_handler' => [
         'class' => ErrorHandler::class,
         'active' => true,