Deprecate using the old DBAL logger system

derrabus · derrabus · commit 48faa407f9c2 · 2023-06-06T18:28:03.000+02:00
diff --git a/UPGRADE-6.4.md b/UPGRADE-6.4.md
@@ -1,6 +1,13 @@
 UPGRADE FROM 6.3 to 6.4
 =======================
 
+DoctrineBridge
+--------------
+
+ * Deprecate `DbalLogger`, use a middleware instead
+ * Deprecate not constructing `DoctrineDataCollector` with an instance of `DebugDataHolder`
+ * Deprecate `DoctrineDataCollector::addLogger()`, use a `DebugDataHolder` instead
+
 HttpFoundation
 --------------
 
diff --git a/src/Symfony/Bridge/Doctrine/CHANGELOG.md b/src/Symfony/Bridge/Doctrine/CHANGELOG.md
@@ -1,6 +1,13 @@
 CHANGELOG
 =========
 
+6.4
+---
+
+ * Deprecate `DbalLogger`, use a middleware instead
+ * Deprecate not constructing `DoctrineDataCollector` with an instance of `DebugDataHolder`
+ * Deprecate `DoctrineDataCollector::addLogger()`, use a `DebugDataHolder` instead
+
 6.3
 ---
 
diff --git a/src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php b/src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php
@@ -43,15 +43,23 @@ public function __construct(
     ) {
         $this->connections = $registry->getConnectionNames();
         $this->managers = $registry->getManagerNames();
+
+        if (null === $debugDataHolder) {
+            trigger_deprecation('symfony/doctrine-bridge', '6.4', 'Not passing an instance of "%s" as "$debugDataHolder" to "%s()" is deprecated.', DebugDataHolder::class, __METHOD__);
+        }
     }
 
     /**
      * Adds the stack logger for a connection.
      *
      * @return void
+     *
+     * @deprecated since Symfony 6.4, use a DebugDataHolder instead.
      */
     public function addLogger(string $name, DebugStack $logger)
     {
+        trigger_deprecation('symfony/doctrine-bridge', '6.4', '"%s()" is deprecated. Pass an instance of "%s" to the constructor instead.', __METHOD__, DebugDataHolder::class);
+
         $this->loggers[$name] = $logger;
     }
 
diff --git a/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php b/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php
@@ -15,8 +15,12 @@
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Stopwatch\Stopwatch;
 
+trigger_deprecation('symfony/doctrine-bridge', '6.4', '"%s" is deprecated, use a middleware instead.', DbalLogger::class);
+
 /**
  * @author Fabien Potencier <fabien@symfony.com>
+ *
+ * @deprecated since Symfony 6.4, use a middleware instead.
  */
 class DbalLogger implements SQLLogger
 {

Original file line number	Diff line number	Diff line change
`@@ -43,15 +43,23 @@ public function __construct(`
`43`	`43`	`) {`
`44`	`44`	`$this->connections = $registry->getConnectionNames();`
`45`	`45`	`$this->managers = $registry->getManagerNames();`
	`46`	`+`
	`47`	`+ if (null === $debugDataHolder) {`
	`48`	`+ trigger_deprecation('symfony/doctrine-bridge', '6.4', 'Not passing an instance of "%s" as "$debugDataHolder" to "%s()" is deprecated.', DebugDataHolder::class, __METHOD__);`
	`49`	`+ }`
`46`	`50`	`}`
`47`	`51`
`48`	`52`	`/**`
`49`	`53`	`* Adds the stack logger for a connection.`
`50`	`54`	`*`
`51`	`55`	`* @return void`
	`56`	`+ *`
	`57`	`+ * @deprecated since Symfony 6.4, use a DebugDataHolder instead.`
`52`	`58`	`*/`
`53`	`59`	`public function addLogger(string $name, DebugStack $logger)`
`54`	`60`	`{`
	`61`	`+ trigger_deprecation('symfony/doctrine-bridge', '6.4', '"%s()" is deprecated. Pass an instance of "%s" to the constructor instead.', __METHOD__, DebugDataHolder::class);`
	`62`	`+`
`55`	`63`	`$this->loggers[$name] = $logger;`
`56`	`64`	`}`
`57`	`65`