Pass a PSR logger to LoadDataFixturesDoctrineCommand

Not doing so is deprecated.
See doctrine/data-fixtures#462

Author: greg0ire

composer.json

@@ -21,7 +21,7 @@
     ],
     "require": {
         "php": "^7.4 || ^8.0",
-        "doctrine/data-fixtures": "^1.3",
+        "doctrine/data-fixtures": "^1.8",
         "doctrine/doctrine-bundle": "^2.2",
         "doctrine/orm": "^2.14.0 || ^3.0",
         "doctrine/persistence": "^2.4|^3.0",

config/services.xml

@@ -8,7 +8,10 @@
         <service id="doctrine.fixtures_load_command" class="Doctrine\Bundle\FixturesBundle\Command\LoadDataFixturesDoctrineCommand">
             <argument type="service" id="doctrine.fixtures.loader" />
             <argument type="service" id="doctrine" />
+            <argument type="collection"></argument>
+            <argument type="service" id="logger" on-invalid="null"/>
             <tag name="console.command" command="doctrine:fixtures:load" />
+            <tag name="monolog.logger" channel="doctrine" />
         </service>
 
         <service id="doctrine.fixtures.loader" class="Doctrine\Bundle\FixturesBundle\Loader\SymfonyFixturesLoader" public="false">

src/Command/LoadDataFixturesDoctrineCommand.php

@@ -12,6 +12,7 @@
 use Doctrine\Common\DataFixtures\Executor\ORMExecutor;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\Persistence\ManagerRegistry;
+use Psr\Log\LoggerInterface;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
@@ -34,9 +35,15 @@ class LoadDataFixturesDoctrineCommand extends DoctrineCommand
     /** @var PurgerFactory[] */
     private array $purgerFactories;
 
+    private ?LoggerInterface $logger;
+
     /** @param PurgerFactory[] $purgerFactories */
-    public function __construct(SymfonyFixturesLoader $fixturesLoader, ?ManagerRegistry $doctrine = null, array $purgerFactories = [])
-    {
+    public function __construct(
+        SymfonyFixturesLoader $fixturesLoader,
+        ?ManagerRegistry $doctrine = null,
+        array $purgerFactories = [],
+        ?LoggerInterface $logger = null
+    ) {
         if ($doctrine === null) {
             trigger_deprecation(
                 'doctrine/fixtures-bundle',
@@ -51,6 +58,7 @@ public function __construct(SymfonyFixturesLoader $fixturesLoader, ?ManagerRegis
 
         $this->fixturesLoader  = $fixturesLoader;
         $this->purgerFactories = $purgerFactories;
+        $this->logger          = $logger;
     }
 
     /** @return void */
@@ -134,7 +142,7 @@ private function doExecute(InputInterface $input, OutputInterface $output): int
             $input->getOption('purge-with-truncate'),
         );
         $executor = new ORMExecutor($em, $purger);
-        $executor->setLogger(static function ($message) use ($ui): void {
+        $executor->setLogger($this->logger ?? static function ($message) use ($ui): void {
             $ui->text(sprintf('  <comment>></comment> <info>%s</info>', $message));
         });
         $executor->execute($fixtures, $input->getOption('append'));

tests/Command/LoadDataFixturesDoctrineCommandTest.php

@@ -9,6 +9,7 @@
 use Doctrine\Bundle\FixturesBundle\Loader\SymfonyFixturesLoader;
 use Doctrine\Persistence\ManagerRegistry;
 use PHPUnit\Framework\TestCase;
+use Psr\Log\NullLogger;
 use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
 use Symfony\Component\DependencyInjection\Container;
 use TypeError;
@@ -29,7 +30,7 @@ public function testInstantiatingWithoutManagerRegistry(): void
         $this->expectDeprecation('Since doctrine/fixtures-bundle 3.2: Argument 2 of Doctrine\Bundle\FixturesBundle\Command\LoadDataFixturesDoctrineCommand::__construct() expects an instance of Doctrine\Persistence\ManagerRegistry, not passing it will throw a \TypeError in DoctrineFixturesBundle 4.0.');
 
         try {
-            new LoadDataFixturesDoctrineCommand($loader);
+            new LoadDataFixturesDoctrineCommand($loader, null, [], new NullLogger());
         } catch (TypeError $e) {
             $this->expectExceptionMessage(sprintf(
                 PHP_VERSION_ID >= 80000 ?
@@ -49,6 +50,6 @@ public function testInstantiatingWithManagerRegistry(): void
         $registry = $this->createMock(ManagerRegistry::class);
         $loader   = new SymfonyFixturesLoader(new Container());
 
-        new LoadDataFixturesDoctrineCommand($loader, $registry);
+        new LoadDataFixturesDoctrineCommand($loader, $registry, [], new NullLogger());
     }
 }

tests/IntegrationTest.php

@@ -21,6 +21,7 @@
 use Doctrine\Persistence\ManagerRegistry;
 use LogicException;
 use PHPUnit\Framework\TestCase;
+use Psr\Log\NullLogger;
 use Symfony\Component\Console\Tester\CommandTester;
 use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -287,6 +288,8 @@ public function testRunCommandWithDefaultPurger(): void
                 ->setPublic(true)
                 ->setSynthetic(true);
 
+            $c->setDefinition('logger', new Definition(NullLogger::class));
+
             $c->setAlias('test.doctrine.fixtures.purger.orm_purger_factory', new Alias('doctrine.fixtures.purger.orm_purger_factory', true));
 
             $c->setAlias('test.doctrine.fixtures_load_command', new Alias('doctrine.fixtures_load_command', true));
@@ -334,6 +337,8 @@ public function testRunCommandWithPurgeExclusions(): void
                 ->setPublic(true)
                 ->setSynthetic(true);
 
+            $c->setDefinition('logger', new Definition(NullLogger::class));
+
             $c->setAlias('test.doctrine.fixtures.purger.orm_purger_factory', new Alias('doctrine.fixtures.purger.orm_purger_factory', true));
 
             $c->setAlias('test.doctrine.fixtures_load_command', new Alias('doctrine.fixtures_load_command', true));
@@ -386,6 +391,8 @@ public function testRunCommandWithCustomPurgerAndCustomEntityManager(): void
                 ->setSynthetic(true)
                 ->addTag(PurgerFactoryCompilerPass::PURGER_FACTORY_TAG, ['alias' => 'test']));
 
+            $c->setDefinition('logger', new Definition(NullLogger::class));
+
             $c->setAlias('test.doctrine.fixtures_load_command', new Alias('doctrine.fixtures_load_command', true));
         });
         $kernel->boot();
@@ -431,6 +438,8 @@ public function testRunCommandWithPurgeMode(): void
                 ->setPublic(true)
                 ->setSynthetic(true);
 
+            $c->setDefinition('logger', new Definition(NullLogger::class));
+
             $c->setAlias('test.doctrine.fixtures.purger.orm_purger_factory', new Alias('doctrine.fixtures.purger.orm_purger_factory', true));
 
             $c->setAlias('test.doctrine.fixtures_load_command', new Alias('doctrine.fixtures_load_command', true));