Merge pull request #3584 from ostrolucky/rem-addlogger

Remove LoggerChain::addLogger

Author: morozov

UPGRADE.md

@@ -53,6 +53,7 @@ The `Doctrine\DBAL\Driver::getName()` has been removed.
  * Removed `Table::addUnnamedForeignKeyConstraint()` and `Table::addNamedForeignKeyConstraint()`.
  * Removed `Table::renameColumn()`.
  * Removed `SQLParserUtils::getPlaceholderPositions()`.
+ * Removed `LoggerChain::addLogger`.
  * Removed `AbstractSchemaManager::getFilterSchemaAssetsExpression()`, `Configuration::getFilterSchemaAssetsExpression()`
    and `Configuration::getFilterSchemaAssetsExpression()`.
  * `SQLParserUtils::*_TOKEN` constants made private.

lib/Doctrine/DBAL/Logging/LoggerChain.php

@@ -9,15 +9,15 @@
  */
 final class LoggerChain implements SQLLogger
 {
-    /** @var SQLLogger[] */
+    /** @var iterable<SQLLogger> */
     private $loggers = [];
 
     /**
-     * Adds a logger in the chain.
+     * @param iterable<SQLLogger> $loggers
      */
-    public function addLogger(SQLLogger $logger) : void
+    public function __construct(iterable $loggers = [])
     {
-        $this->loggers[] = $logger;
+        $this->loggers = $loggers;
     }
 
     /**