Deprecate using the old DBAL logger system

Author: derrabus

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
 ---
 

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;
     }
 

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 <[email protected]>
+ *
+ * @deprecated since Symfony 6.4, use a middleware instead.
  */
 class DbalLogger implements SQLLogger
 {

Tests/DataCollector/DoctrineDataCollectorWithDebugStackTest.php

@@ -17,6 +17,7 @@
 use Doctrine\Persistence\ManagerRegistry;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\Doctrine\DataCollector\DoctrineDataCollector;
+use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\VarDumper\Cloner\Data;
@@ -31,6 +32,7 @@ class_exists(\Doctrine\DBAL\Platforms\MySqlPlatform::class);
 class DoctrineDataCollectorWithDebugStackTest extends TestCase
 {
     use DoctrineDataCollectorTestTrait;
+    use ExpectDeprecationTrait;
 
     public function testReset()
     {
@@ -178,9 +180,12 @@ private function createCollector(array $queries): DoctrineDataCollector
             ->method('getConnection')
             ->willReturn($connection);
 
+        $this->expectDeprecation('Since symfony/doctrine-bridge 6.4: Not passing an instance of "Symfony\Bridge\Doctrine\Middleware\Debug\DebugDataHolder" as "$debugDataHolder" to "Symfony\Bridge\Doctrine\DataCollector\DoctrineDataCollector::__construct()" is deprecated.');
         $collector = new DoctrineDataCollector($registry);
         $logger = $this->createMock(DebugStack::class);
         $logger->queries = $queries;
+
+        $this->expectDeprecation('Since symfony/doctrine-bridge 6.4: "Symfony\Bridge\Doctrine\DataCollector\DoctrineDataCollector::addLogger()" is deprecated. Pass an instance of "Symfony\Bridge\Doctrine\Middleware\Debug\DebugDataHolder" to the constructor instead.');
         $collector->addLogger('default', $logger);
 
         return $collector;