minor symfony#52426 [HttpKernel] the debug log processor must be a callable (xabbuh)

nicolas-grekas · nicolas-grekas · commit fbc44f27a3ad · 2023-11-07T13:58:44.000+01:00
This PR was merged into the 6.4 branch. Discussion ---------- [HttpKernel] the debug log processor must be a callable | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | | License | MIT A Monolog processor must be a callable, but a `DebugLoggerInterface` implementation is not necessarily a callable. Commits ------- df37834 the debug log processor must be a callable
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 ?DebugLoggerInterface $processor = null;
+    private ?\Closure $processor = null;
 
-    public function __construct(DebugLoggerInterface $processor, bool $enable = null)
+    public function __construct(callable $processor, bool $enable = null)
     {
         if ($enable ?? !\in_array(\PHP_SAPI, ['cli', 'phpdbg', 'embed'], true)) {
-            $this->processor = $processor;
+            $this->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 ?DebugLoggerInterface $processor = null;`
	`21`	`+ private ?\Closure $processor = null;`
`22`	`22`
`23`		`- public function __construct(DebugLoggerInterface $processor, bool $enable = null)`
	`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 = $processor(...);`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`