bug #52994 [MonologBridge] Fix context data and display extra data (louismariegaborit)

nicolas-grekas · nicolas-grekas · commit 2fae7d575453 · 2024-01-29T15:47:00.000+01:00
This PR was merged into the 6.3 branch. Discussion ---------- [MonologBridge] Fix context data and display extra data | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | -> | License | MIT Fix the compatibility with Monolog 3. In the PR #52222 we had the compatibility with Monolog 3 but context parameter is incorrect and extra parameter is missing. No test can be added because testing this command is not possible at this time. Another PR (#53518) was opened to add test on this command. Commits ------- 8687ae0 Fix context data and display extra data
diff --git a/src/Symfony/Bridge/Monolog/Command/ServerLogCommand.php b/src/Symfony/Bridge/Monolog/Command/ServerLogCommand.php
@@ -164,7 +164,8 @@ private function displayLog(OutputInterface $output, int $clientId, array $recor
                 $record['channel'],
                 Level::fromValue($record['level']),
                 $record['message'],
-                $record['context']->getContext(),
+                $record['context']->getValue(true),
+                $record['extra']->getValue(true),
             );
         }
 

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,8 @@ private function displayLog(OutputInterface $output, int $clientId, array $recor`
`164`	`164`	`$record['channel'],`
`165`	`165`	`Level::fromValue($record['level']),`
`166`	`166`	`$record['message'],`
`167`		`- $record['context']->getContext(),`
	`167`	`+ $record['context']->getValue(true),`
	`168`	`+ $record['extra']->getValue(true),`
`168`	`169`	`);`
`169`	`170`	`}`
`170`	`171`