From c71c62636f2cef9c469d9510a4833103cdeea432 Mon Sep 17 00:00:00 2001 From: "ityaozm@gmail.com" Date: Sun, 13 Oct 2024 22:18:32 +0800 Subject: [PATCH] refactor(logging): replace NullLogger with ConsoleLogger for verbose output This commit enhances the logging mechanism by introducing ConsoleLogger for verbose output in the AppServiceProvider. The change allows for more detailed logging in the console while maintaining the option to use a NullLogger for non-verbose environments. The implementation ensures compatibility with existing configurations by only using ConsoleLogger when the current logger is not a NullLogger. - Added ConsoleLogger import to the AppServiceProvider - Modified the logger extension to conditionally return a ConsoleLogger - Preserves existing behavior for non-console or non-verbose contexts --- app/Providers/AppServiceProvider.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/app/Providers/AppServiceProvider.php b/app/Providers/AppServiceProvider.php index 41604ba..01a7647 100644 --- a/app/Providers/AppServiceProvider.php +++ b/app/Providers/AppServiceProvider.php @@ -24,6 +24,7 @@ use Psr\Log\LoggerInterface; use Psr\Log\NullLogger; use Symfony\Component\Console\Input\ArgvInput; +use Symfony\Component\Console\Logger\ConsoleLogger; use Symfony\Component\Console\Output\ConsoleOutput; final class AppServiceProvider extends ServiceProvider @@ -63,6 +64,7 @@ public function boot(): void $_SERVER['VAR_DUMPER_FORMAT'] = 'server'; $this->app->extend(LoggerInterface::class, static function (LoggerInterface $logger): NullLogger { + // return $logger instanceof ConsoleLogger ? $logger : new ConsoleLogger(app(OutputStyle::class)); return $logger instanceof NullLogger ? $logger : new NullLogger(); });