Command: Rewrite make:migration, fix #1

Author: jgauthi

README.md

@@ -56,4 +56,6 @@ Your commit is checked by several dev tools (like phpstan, php cs fixer...). The
 
 
 ## Development notes
-For make new migration, don't use the command `php bin/console make:migration` (no compatible with DoctrineMigrationsMultipleDatabaseBundle), instead use: `php bin/console doctrine:migrations:diff`.
+For make new migrations in dev environnement, use the command `php bin/console make:migration` (this poc update the command with adding multiple databases support).
+
+When you use the command `make:entity`, you should add the database namespace before the entity name. Example: _Second\Article_.

config/services.yaml

@@ -31,3 +31,6 @@ services:
     tags:
       - { name: doctrine.event_subscriber, event: prePersist }
       - { name: doctrine.event_subscriber, event: preUpdate }
+
+  maker.maker.make_migration:
+    class: App\Command\MakeMigrationCommand

src/Command/FixtureCommand.php

@@ -15,7 +15,7 @@
 class FixtureCommand extends Command
 {
     // Entity Manager name (groups should be used the same value)
-    private const LIST_DATABASE = ['main', 'second'];
+    public const LIST_DATABASE = ['main', 'second'];
 
     /** @var bool[] */
     private array $requirement;

src/Command/MakeMigrationCommand.php

@@ -0,0 +1,52 @@
+<?php
+namespace App\Command;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\{ArrayInput, InputInterface};
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Style\SymfonyStyle;
+use Symfony\Component\HttpKernel\KernelInterface;
+
+#[AsCommand(
+    name: 'make:migration',
+    description: 'Enhance the command for support multiple databases.',
+)]
+class MakeMigrationCommand extends Command
+{
+    /** @var bool[] */
+    private array $requirement;
+
+    public function __construct(KernelInterface $kernel, ?string $name = null)
+    {
+        parent::__construct($name);
+
+        $this->requirement = [
+            'doctrine_migration' => array_key_exists('DoctrineMigrationsBundle', $kernel->getBundles()),
+        ];
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        if (!$this->getApplication()) {
+            return Command::INVALID;
+        } elseif (!$this->requirement['doctrine_migration']) {
+            $io->error('The bundle DoctrineMigrationsBundle is inactive or the APP_ENV value is not dev.');
+
+            return Command::INVALID;
+        }
+
+        foreach (FixtureCommand::LIST_DATABASE as $database) {
+            try {
+                $io->title('Creation migration for database: '.$database);
+                $symfonyCommand = $this->getApplication()->find('doctrine:migrations:diff');
+                $symfonyCommand->run(new ArrayInput(['--em' => $database]), $output);
+            } catch (\Throwable $exception) {
+                $io->writeln($exception->getMessage());
+            }
+        }
+
+        return Command::INVALID;
+    }
+}