minor #51457 [HttpKernel] Fix checking for the runtime mode in DebugLoggerConfigurator (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit 84628741c5dd · 2023-08-23T09:04:03.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [HttpKernel] Fix checking for the runtime mode in DebugLoggerConfigurator | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Fixing #51284, my bad. Eating your own dog food works :) Commits ------- 1836a1e [HttpKernel] Fix checking for the runtime mode in DebugLoggerConfigurator
diff --git a/src/Symfony/Component/HttpKernel/Log/DebugLoggerConfigurator.php b/src/Symfony/Component/HttpKernel/Log/DebugLoggerConfigurator.php
@@ -22,7 +22,7 @@ class DebugLoggerConfigurator
 
     public function __construct(DebugLoggerInterface $processor, bool $enable = null)
     {
-        if ($enable ?? \in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {
+        if ($enable ?? !\in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {
             $this->processor = $processor;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ class DebugLoggerConfigurator`
`22`	`22`
`23`	`23`	`public function __construct(DebugLoggerInterface $processor, bool $enable = null)`
`24`	`24`	`{`
`25`		`- if ($enable ?? \in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {`
	`25`	`+ if ($enable ?? !\in_array(\PHP_SAPI, ['cli', 'phpdbg'], true)) {`
`26`	`26`	`$this->processor = $processor;`
`27`	`27`	`}`
`28`	`28`	`}`