From c356a12e47a4c38f894d36e0a5cc34927760b175 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gr=C3=A9goire=20Paris?= <postmaster@greg0ire.fr>
Date: Thu, 14 Mar 2024 16:20:20 +0100
Subject: [PATCH] Add support for --complete

When using orm:schema-tool:update together with --complete, all assets
not described by the current metadata is dropped.

This is an issue when using doctrine/migrations, because not using that
option is deprecated, and because when it is used, the table that holds
the migrations is dropped as well since it is not described by ORM
metadata.

A solution to that is configuring an asset filter that filters out the
metadata table except when running commands inside the migrations
namespace.
---
 EventListener/SchemaFilterListener.php        | 51 +++++++++++++++++++
 Resources/config/services.xml                 |  5 ++
 .../SchemaFilterListenerTest.php              | 48 +++++++++++++++++
 phpstan.neon.dist                             |  1 +
 psalm.xml                                     |  1 +
 5 files changed, 106 insertions(+)
 create mode 100644 EventListener/SchemaFilterListener.php
 create mode 100644 Tests/Collector/EventListener/SchemaFilterListenerTest.php

diff --git a/EventListener/SchemaFilterListener.php b/EventListener/SchemaFilterListener.php
new file mode 100644
index 0000000..b88fc2b
--- /dev/null
+++ b/EventListener/SchemaFilterListener.php
@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\Bundle\MigrationsBundle\EventListener;
+
+use Doctrine\DBAL\Schema\AbstractAsset;
+use Doctrine\Migrations\Metadata\Storage\TableMetadataStorageConfiguration;
+use Doctrine\Migrations\Tools\Console\Command\DoctrineCommand;
+use Symfony\Component\Console\Event\ConsoleCommandEvent;
+
+/**
+ * Acts as a schema filter able that hides the migration metadata table except
+ * when the execution context is that of command inside the migrations
+ * namespace.
+ */
+final class SchemaFilterListener
+{
+    /** @var bool */
+    private $enabled = true;
+
+    /** @param AbstractAsset|string $asset */
+    public function __invoke($asset): bool
+    {
+        if (! $this->enabled) {
+            return true;
+        }
+
+        if ($asset instanceof AbstractAsset) {
+            $asset = $asset->getName();
+        }
+
+        return $asset !== (new TableMetadataStorageConfiguration())->getTableName();
+    }
+
+    public function disable(): void
+    {
+        $this->enabled = false;
+    }
+
+    public function onConsoleCommand(ConsoleCommandEvent $event): void
+    {
+        $command = $event->getCommand();
+
+        if (! $command instanceof DoctrineCommand) {
+            return;
+        }
+
+        $this->disable();
+    }
+}
diff --git a/Resources/config/services.xml b/Resources/config/services.xml
index cb97871..17ff210 100644
--- a/Resources/config/services.xml
+++ b/Resources/config/services.xml
@@ -146,6 +146,11 @@
             <tag name="console.command" command="doctrine:migrations:version" />
         </service>
 
+        <service id="doctrine_migrations.schema_filter_listener" class="Doctrine\Migrations\EventListener\SchemaFilterListener">
+            <tag name="kernel.event_listener" event="console.command" method="onConsoleCommand" />
+            <tag name="doctrine.dbal.schema_filter" />
+        </service>
+
     </services>
 
 </container>
diff --git a/Tests/Collector/EventListener/SchemaFilterListenerTest.php b/Tests/Collector/EventListener/SchemaFilterListenerTest.php
new file mode 100644
index 0000000..5a6b7f3
--- /dev/null
+++ b/Tests/Collector/EventListener/SchemaFilterListenerTest.php
@@ -0,0 +1,48 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\Bundle\MigrationsBundle\Tests\Collector\EventListener;
+
+use Doctrine\Bundle\MigrationsBundle\EventListener\SchemaFilterListener;
+use Doctrine\DBAL\Schema\Table;
+use Doctrine\Migrations\Tools\Console\Command\DoctrineCommand;
+use PHPUnit\Framework\TestCase;
+use Symfony\Component\Console\Event\ConsoleCommandEvent;
+use Symfony\Component\Console\Input\ArrayInput;
+use Symfony\Component\Console\Output\NullOutput;
+
+class SchemaFilterListenerTest extends TestCase
+{
+    public function testItFiltersOutMigrationMetadataTableByDefault(): void
+    {
+        $listener = new SchemaFilterListener();
+
+        self::assertFalse($listener(new Table('doctrine_migration_versions')));
+        self::assertTrue($listener(new Table('some_other_table')));
+    }
+
+    public function testItFiltersNothingWhenDisabled(): void
+    {
+        $listener = new SchemaFilterListener();
+        $listener->disable();
+
+        self::assertTrue($listener(new Table('doctrine_migration_versions')));
+        self::assertTrue($listener(new Table('some_other_table')));
+    }
+
+    public function testItDisablesItselfWhenTheCurrentCommandIsAMigrationsCommand(): void
+    {
+        $listener          = new SchemaFilterListener();
+        $migrationsCommand = new class extends DoctrineCommand {
+        };
+
+        $listener->onConsoleCommand(new ConsoleCommandEvent(
+            $migrationsCommand,
+            new ArrayInput([]),
+            new NullOutput()
+        ));
+
+        self::assertTrue($listener(new Table('doctrine_migration_versions')));
+    }
+}
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
index 30c6a8a..471c221 100644
--- a/phpstan.neon.dist
+++ b/phpstan.neon.dist
@@ -3,6 +3,7 @@ parameters:
     phpVersion: 80200
     paths:
         - DependencyInjection
+        - EventListener
         - Tests
 
     excludePaths:
diff --git a/psalm.xml b/psalm.xml
index 578e54f..32efe93 100644
--- a/psalm.xml
+++ b/psalm.xml
@@ -11,6 +11,7 @@
 >
     <projectFiles>
         <directory name="DependencyInjection" />
+        <directory name="EventListener" />
         <directory name="Tests" />
         <file name="DoctrineMigrationsBundle.php" />
         <ignoreFiles>