From 67042bff3199540bbef8b65f22f2abaedca152ff Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 3 Feb 2025 01:52:26 +0000 Subject: [PATCH 1/8] Bump doctrine/.github from 7.1.0 to 7.2.0 Bumps [doctrine/.github](https://github.com/doctrine/.github) from 7.1.0 to 7.2.0. - [Release notes](https://github.com/doctrine/.github/releases) - [Commits](https://github.com/doctrine/.github/compare/7.1.0...7.2.0) --- updated-dependencies: - dependency-name: doctrine/.github dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- .github/workflows/coding-standards.yml | 2 +- .github/workflows/composer-lint.yml | 2 +- .github/workflows/continuous-integration.yml | 4 ++-- .github/workflows/release-on-milestone-closed.yml | 2 +- .github/workflows/static-analysis.yml | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/coding-standards.yml b/.github/workflows/coding-standards.yml index 9f949bde..e9aee0ef 100644 --- a/.github/workflows/coding-standards.yml +++ b/.github/workflows/coding-standards.yml @@ -12,4 +12,4 @@ on: jobs: coding-standards: name: "Coding Standards" - uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.2.0" diff --git a/.github/workflows/composer-lint.yml b/.github/workflows/composer-lint.yml index 638a1418..45fd43e8 100644 --- a/.github/workflows/composer-lint.yml +++ b/.github/workflows/composer-lint.yml @@ -17,4 +17,4 @@ on: jobs: composer-lint: name: "Composer Lint" - uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.2.0" diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml index ee8c38b5..00849985 100644 --- a/.github/workflows/continuous-integration.yml +++ b/.github/workflows/continuous-integration.yml @@ -12,7 +12,7 @@ on: jobs: phpunit: name: "PHPUnit" - uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.2.0" with: php-versions: '["7.2", "7.3", "7.4", "8.0", "8.1", "8.2", "8.3", "8.4"]' secrets: @@ -20,7 +20,7 @@ jobs: phpunit-with-dev-deps: name: "PHPUnit with dev dependencies" - uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.2.0" with: symfony-version-constraint: "^7" secrets: diff --git a/.github/workflows/release-on-milestone-closed.yml b/.github/workflows/release-on-milestone-closed.yml index 0c4ee4a5..6af3b106 100644 --- a/.github/workflows/release-on-milestone-closed.yml +++ b/.github/workflows/release-on-milestone-closed.yml @@ -8,7 +8,7 @@ on: jobs: release: name: "Git tag, release & create merge-up PR" - uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.2.0" secrets: GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }} GIT_AUTHOR_NAME: ${{ secrets.GIT_AUTHOR_NAME }} diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml index d1e53833..5c963c45 100644 --- a/.github/workflows/static-analysis.yml +++ b/.github/workflows/static-analysis.yml @@ -12,4 +12,4 @@ on: jobs: static-analysis: name: "Static Analysis" - uses: "doctrine/.github/.github/workflows/phpstan.yml@7.1.0" + uses: "doctrine/.github/.github/workflows/phpstan.yml@7.2.0" From ca926d7ff0eafbebb51d4a802da204c7ebeab6d9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Paris?= Date: Fri, 7 Feb 2025 08:56:14 +0100 Subject: [PATCH 2/8] Remove constraint on doctrine/persistence The last reference to that library was removed in 120cca6b7c8abada0772706e121520f50db46bfa --- composer.json | 1 - 1 file changed, 1 deletion(-) diff --git a/composer.json b/composer.json index ee7c51fd..11786bdc 100644 --- a/composer.json +++ b/composer.json @@ -34,7 +34,6 @@ "composer/semver": "^3.0", "doctrine/coding-standard": "^12", "doctrine/orm": "^2.6 || ^3", - "doctrine/persistence": "^2.0 || ^3", "phpstan/phpstan": "^1.4 || ^2", "phpstan/phpstan-deprecation-rules": "^1 || ^2", "phpstan/phpstan-phpunit": "^1 || ^2", From 9f27613322dd9905f9c0b4ae711f45ecd6e282a1 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 10 Feb 2025 07:56:18 +0100 Subject: [PATCH 3/8] Bump doctrine/.github from 7.2.0 to 7.2.1 (#598) Bumps [doctrine/.github](https://github.com/doctrine/.github) from 7.2.0 to 7.2.1. - [Release notes](https://github.com/doctrine/.github/releases) - [Commits](https://github.com/doctrine/.github/compare/7.2.0...7.2.1) --- updated-dependencies: - dependency-name: doctrine/.github dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- .github/workflows/coding-standards.yml | 2 +- .github/workflows/composer-lint.yml | 2 +- .github/workflows/continuous-integration.yml | 4 ++-- .github/workflows/release-on-milestone-closed.yml | 2 +- .github/workflows/static-analysis.yml | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/coding-standards.yml b/.github/workflows/coding-standards.yml index e9aee0ef..1614fbf8 100644 --- a/.github/workflows/coding-standards.yml +++ b/.github/workflows/coding-standards.yml @@ -12,4 +12,4 @@ on: jobs: coding-standards: name: "Coding Standards" - uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.2.1" diff --git a/.github/workflows/composer-lint.yml b/.github/workflows/composer-lint.yml index 45fd43e8..c121b51b 100644 --- a/.github/workflows/composer-lint.yml +++ b/.github/workflows/composer-lint.yml @@ -17,4 +17,4 @@ on: jobs: composer-lint: name: "Composer Lint" - uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.2.1" diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml index 00849985..0e4db781 100644 --- a/.github/workflows/continuous-integration.yml +++ b/.github/workflows/continuous-integration.yml @@ -12,7 +12,7 @@ on: jobs: phpunit: name: "PHPUnit" - uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.2.1" with: php-versions: '["7.2", "7.3", "7.4", "8.0", "8.1", "8.2", "8.3", "8.4"]' secrets: @@ -20,7 +20,7 @@ jobs: phpunit-with-dev-deps: name: "PHPUnit with dev dependencies" - uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.2.1" with: symfony-version-constraint: "^7" secrets: diff --git a/.github/workflows/release-on-milestone-closed.yml b/.github/workflows/release-on-milestone-closed.yml index 6af3b106..45ed91f1 100644 --- a/.github/workflows/release-on-milestone-closed.yml +++ b/.github/workflows/release-on-milestone-closed.yml @@ -8,7 +8,7 @@ on: jobs: release: name: "Git tag, release & create merge-up PR" - uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.2.1" secrets: GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }} GIT_AUTHOR_NAME: ${{ secrets.GIT_AUTHOR_NAME }} diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml index 5c963c45..1037232b 100644 --- a/.github/workflows/static-analysis.yml +++ b/.github/workflows/static-analysis.yml @@ -12,4 +12,4 @@ on: jobs: static-analysis: name: "Static Analysis" - uses: "doctrine/.github/.github/workflows/phpstan.yml@7.2.0" + uses: "doctrine/.github/.github/workflows/phpstan.yml@7.2.1" From 7569f070ac41b5f900287a76cf36f96095c00de9 Mon Sep 17 00:00:00 2001 From: Tobias Feijten Date: Tue, 11 Feb 2025 19:59:26 +0100 Subject: [PATCH 4/8] Fix filter support for multiple connections --- src/DependencyInjection/DoctrineMigrationsExtension.php | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/DependencyInjection/DoctrineMigrationsExtension.php b/src/DependencyInjection/DoctrineMigrationsExtension.php index 5e6645f2..85a85a76 100644 --- a/src/DependencyInjection/DoctrineMigrationsExtension.php +++ b/src/DependencyInjection/DoctrineMigrationsExtension.php @@ -124,6 +124,15 @@ public function load(array $configs, ContainerBuilder $container): void } $configurationDefinition->addMethodCall('setMetadataStorageConfiguration', [new Reference('doctrine.migrations.storage.table_storage')]); + + // Add tag to the filter for each Doctrine connection, so the table is ignored for multiple connections + if ($container->hasParameter('doctrine.connections')) { + /** @var array $connections */ + $connections = $container->getParameter('doctrine.connections'); + foreach (array_keys($connections) as $connection) { + $filterDefinition->addTag('doctrine.dbal.schema_filter', ['connection' => $connection]); + } + } } if ($config['em'] !== null && $config['connection'] !== null) { From af735b55e3697a62a695ae8051a2fb1ee71cf41c Mon Sep 17 00:00:00 2001 From: Tobias Feijten Date: Tue, 18 Feb 2025 21:08:25 +0100 Subject: [PATCH 5/8] Remove default tag for schema filter, it is added for each connection dynamically --- config/services.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/services.xml b/config/services.xml index a96aa098..2f9f079d 100644 --- a/config/services.xml +++ b/config/services.xml @@ -148,7 +148,7 @@ - + From 0d9e53d96ccdb3daa574b4d2c72d52f7be7e895c Mon Sep 17 00:00:00 2001 From: Tobias Feijten Date: Tue, 18 Feb 2025 21:08:51 +0100 Subject: [PATCH 6/8] Add test for filtering multiple connections --- .../DoctrineMigrationsExtensionTest.php | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/tests/DependencyInjection/DoctrineMigrationsExtensionTest.php b/tests/DependencyInjection/DoctrineMigrationsExtensionTest.php index 7fba5b1f..b690516e 100644 --- a/tests/DependencyInjection/DoctrineMigrationsExtensionTest.php +++ b/tests/DependencyInjection/DoctrineMigrationsExtensionTest.php @@ -279,6 +279,10 @@ public function testCustomConnection(): void $di = $container->get('doctrine.migrations.dependency_factory'); self::assertInstanceOf(DependencyFactory::class, $di); self::assertSame($doctrine->getConnection('custom'), $di->getConnection()); + // Check if the multiple connections also added the required tags to the filter + $filterDefinition = $container->findDefinition('doctrine_migrations.schema_filter_listener'); + $tags = $filterDefinition->getTag('doctrine.dbal.schema_filter'); + self::assertCount(2, $tags); } public function testPrefersEntityManagerOverConnection(): void @@ -466,9 +470,6 @@ private function getContainer(array $config, ?array $dbalConfig = null, ?array $ $bundle = new DoctrineMigrationsBundle(); $bundle->build($container); - $extension = new DoctrineMigrationsExtension(); - $extension->load(['doctrine_migrations' => $config], $container); - $extension = new DoctrineExtension(); $doctrineBundleConfigs = $dbalConfig === null ? ['dbal' => ['url' => 'sqlite:///:memory:']] : ['dbal' => $dbalConfig]; @@ -478,8 +479,15 @@ private function getContainer(array $config, ?array $dbalConfig = null, ?array $ $extension->load(['doctrine' => $doctrineBundleConfigs], $container); + $extension = new DoctrineMigrationsExtension(); + $extension->load(['doctrine_migrations' => $config], $container); + $container->getDefinition('doctrine.migrations.dependency_factory')->setPublic(true); $container->getDefinition('doctrine.migrations.configuration')->setPublic(true); + if ($container->hasDefinition('doctrine_migrations.schema_filter_listener')) { + $container->getDefinition('doctrine_migrations.schema_filter_listener')->setPublic(true); + } + $container->addCompilerPass(new CacheCompatibilityPass()); return $container; From dc4f5b51964a2239d90922817ffe7b2b0dedec02 Mon Sep 17 00:00:00 2001 From: Tobias Feijten Date: Tue, 25 Feb 2025 22:43:36 +0100 Subject: [PATCH 7/8] Update whitespace in comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: Grégoire Paris --- config/services.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/services.xml b/config/services.xml index 2f9f079d..53ded478 100644 --- a/config/services.xml +++ b/config/services.xml @@ -148,7 +148,7 @@ - + From be3ba36f92bde36a739f6bb3d3eec5a1e08b9ad0 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 10 Mar 2025 01:08:51 +0000 Subject: [PATCH 8/8] Bump doctrine/.github from 7.2.1 to 7.2.2 Bumps [doctrine/.github](https://github.com/doctrine/.github) from 7.2.1 to 7.2.2. - [Release notes](https://github.com/doctrine/.github/releases) - [Commits](https://github.com/doctrine/.github/compare/7.2.1...7.2.2) --- updated-dependencies: - dependency-name: doctrine/.github dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- .github/workflows/coding-standards.yml | 2 +- .github/workflows/composer-lint.yml | 2 +- .github/workflows/continuous-integration.yml | 4 ++-- .github/workflows/release-on-milestone-closed.yml | 2 +- .github/workflows/static-analysis.yml | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/coding-standards.yml b/.github/workflows/coding-standards.yml index 1614fbf8..1bff5c3d 100644 --- a/.github/workflows/coding-standards.yml +++ b/.github/workflows/coding-standards.yml @@ -12,4 +12,4 @@ on: jobs: coding-standards: name: "Coding Standards" - uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/coding-standards.yml@7.2.2" diff --git a/.github/workflows/composer-lint.yml b/.github/workflows/composer-lint.yml index c121b51b..a7ad8848 100644 --- a/.github/workflows/composer-lint.yml +++ b/.github/workflows/composer-lint.yml @@ -17,4 +17,4 @@ on: jobs: composer-lint: name: "Composer Lint" - uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/composer-lint.yml@7.2.2" diff --git a/.github/workflows/continuous-integration.yml b/.github/workflows/continuous-integration.yml index 0e4db781..f75bab45 100644 --- a/.github/workflows/continuous-integration.yml +++ b/.github/workflows/continuous-integration.yml @@ -12,7 +12,7 @@ on: jobs: phpunit: name: "PHPUnit" - uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/continuous-integration.yml@7.2.2" with: php-versions: '["7.2", "7.3", "7.4", "8.0", "8.1", "8.2", "8.3", "8.4"]' secrets: @@ -20,7 +20,7 @@ jobs: phpunit-with-dev-deps: name: "PHPUnit with dev dependencies" - uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/continuous-integration-symfony-unstable.yml@7.2.2" with: symfony-version-constraint: "^7" secrets: diff --git a/.github/workflows/release-on-milestone-closed.yml b/.github/workflows/release-on-milestone-closed.yml index 45ed91f1..c0bfaa86 100644 --- a/.github/workflows/release-on-milestone-closed.yml +++ b/.github/workflows/release-on-milestone-closed.yml @@ -8,7 +8,7 @@ on: jobs: release: name: "Git tag, release & create merge-up PR" - uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@7.2.2" secrets: GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }} GIT_AUTHOR_NAME: ${{ secrets.GIT_AUTHOR_NAME }} diff --git a/.github/workflows/static-analysis.yml b/.github/workflows/static-analysis.yml index 1037232b..809fb334 100644 --- a/.github/workflows/static-analysis.yml +++ b/.github/workflows/static-analysis.yml @@ -12,4 +12,4 @@ on: jobs: static-analysis: name: "Static Analysis" - uses: "doctrine/.github/.github/workflows/phpstan.yml@7.2.1" + uses: "doctrine/.github/.github/workflows/phpstan.yml@7.2.2"