minor #52500 [HttpKernel] Fix DebugLoggerConfigurator (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit 132d8e4b9eb2 · 2023-11-09T09:16:58.000+01:00
This PR was merged into the 6.4 branch. Discussion ---------- [HttpKernel] Fix DebugLoggerConfigurator | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | - | License | MIT Issue introduced in #52426: by turning the callable into a closure, we broke the `instanceof` check later in the class. Commits ------- 33721f5 [HttpKernel] Fix DebugLoggerConfigurator
diff --git a/src/Symfony/Component/HttpKernel/Log/DebugLoggerConfigurator.php b/src/Symfony/Component/HttpKernel/Log/DebugLoggerConfigurator.php
@@ -18,12 +18,12 @@
  */
 class DebugLoggerConfigurator
 {
-    private ?\Closure $processor = null;
+    private ?object $processor = null;
 
     public function __construct(callable $processor, bool $enable = null)
     {
         if ($enable ?? !\in_array(\PHP_SAPI, ['cli', 'phpdbg', 'embed'], true)) {
-            $this->processor = $processor(...);
+            $this->processor = \is_object($processor) ? $processor : $processor(...);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -18,12 +18,12 @@`
`18`	`18`	`*/`
`19`	`19`	`class DebugLoggerConfigurator`
`20`	`20`	`{`
`21`		`- private ?\Closure $processor = null;`
	`21`	`+ private ?object $processor = null;`
`22`	`22`
`23`	`23`	`public function __construct(callable $processor, bool $enable = null)`
`24`	`24`	`{`
`25`	`25`	`if ($enable ?? !\in_array(\PHP_SAPI, ['cli', 'phpdbg', 'embed'], true)) {`
`26`		`- $this->processor = $processor(...);`
	`26`	`+ $this->processor = \is_object($processor) ? $processor : $processor(...);`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`