Revert "bug #50761 [DoctrineBridge] Ignore invalid stores in LockStoreSchemaListener raised by StoreFactory (alexandre-daubois)"

MatTheCat · MatTheCat · commit 5b05c106a149 · 2024-08-08T12:57:56.000+02:00
This reverts commit 69ef02e, reversing changes made to 04c7ccf.
diff --git a/src/Symfony/Bridge/Doctrine/SchemaListener/LockStoreSchemaListener.php b/src/Symfony/Bridge/Doctrine/SchemaListener/LockStoreSchemaListener.php
@@ -12,7 +12,6 @@
 namespace Symfony\Bridge\Doctrine\SchemaListener;
 
 use Doctrine\ORM\Tools\Event\GenerateSchemaEventArgs;
-use Symfony\Component\Lock\Exception\InvalidArgumentException;
 use Symfony\Component\Lock\PersistingStoreInterface;
 use Symfony\Component\Lock\Store\DoctrineDbalStore;
 
@@ -30,20 +29,12 @@ public function postGenerateSchema(GenerateSchemaEventArgs $event): void
     {
         $connection = $event->getEntityManager()->getConnection();
 
-        $storesIterator = new \ArrayIterator($this->stores);
-        while ($storesIterator->valid()) {
-            try {
-                $store = $storesIterator->current();
-                if (!$store instanceof DoctrineDbalStore) {
-                    continue;
-                }
-
-                $store->configureSchema($event->getSchema(), $this->getIsSameDatabaseChecker($connection));
-            } catch (InvalidArgumentException) {
-                // no-op
+        foreach ($this->stores as $store) {
+            if (!$store instanceof DoctrineDbalStore) {
+                continue;
             }
 
-            $storesIterator->next();
+            $store->configureSchema($event->getSchema(), $this->getIsSameDatabaseChecker($connection));
         }
     }
 }
diff --git a/src/Symfony/Bridge/Doctrine/Tests/SchemaListener/LockStoreSchemaListenerTest.php b/src/Symfony/Bridge/Doctrine/Tests/SchemaListener/LockStoreSchemaListenerTest.php
@@ -17,7 +17,6 @@
 use Doctrine\ORM\Tools\Event\GenerateSchemaEventArgs;
 use PHPUnit\Framework\TestCase;
 use Symfony\Bridge\Doctrine\SchemaListener\LockStoreSchemaListener;
-use Symfony\Component\Lock\Exception\InvalidArgumentException;
 use Symfony\Component\Lock\Store\DoctrineDbalStore;
 
 class LockStoreSchemaListenerTest extends TestCase
@@ -40,20 +39,4 @@ public function testPostGenerateSchemaLockPdo()
         $subscriber = new LockStoreSchemaListener((static fn () => yield $lockStore)());
         $subscriber->postGenerateSchema($event);
     }
-
-    public function testPostGenerateSchemaWithInvalidLockStore()
-    {
-        $entityManager = $this->createMock(EntityManagerInterface::class);
-        $entityManager->expects($this->once())
-            ->method('getConnection')
-            ->willReturn($this->createMock(Connection::class));
-        $event = new GenerateSchemaEventArgs($entityManager, new Schema());
-
-        $subscriber = new LockStoreSchemaListener((static function (): \Generator {
-            yield $this->createMock(DoctrineDbalStore::class);
-
-            throw new InvalidArgumentException('Unsupported Connection');
-        })());
-        $subscriber->postGenerateSchema($event);
-    }
 }

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@`
`12`	`12`	`namespace Symfony\Bridge\Doctrine\SchemaListener;`
`13`	`13`
`14`	`14`	`use Doctrine\ORM\Tools\Event\GenerateSchemaEventArgs;`
`15`		`-use Symfony\Component\Lock\Exception\InvalidArgumentException;`
`16`	`15`	`use Symfony\Component\Lock\PersistingStoreInterface;`
`17`	`16`	`use Symfony\Component\Lock\Store\DoctrineDbalStore;`
`18`	`17`
`@@ -30,20 +29,12 @@ public function postGenerateSchema(GenerateSchemaEventArgs $event): void`
`30`	`29`	`{`
`31`	`30`	`$connection = $event->getEntityManager()->getConnection();`
`32`	`31`
`33`		`- $storesIterator = new \ArrayIterator($this->stores);`
`34`		`- while ($storesIterator->valid()) {`
`35`		`- try {`
`36`		`- $store = $storesIterator->current();`
`37`		`- if (!$store instanceof DoctrineDbalStore) {`
`38`		`- continue;`
`39`		`- }`
`40`		`-`
`41`		`- $store->configureSchema($event->getSchema(), $this->getIsSameDatabaseChecker($connection));`
`42`		`- } catch (InvalidArgumentException) {`
`43`		`- // no-op`
	`32`	`+ foreach ($this->stores as $store) {`
	`33`	`+ if (!$store instanceof DoctrineDbalStore) {`
	`34`	`+ continue;`
`44`	`35`	`}`
`45`	`36`
`46`		`- $storesIterator->next();`
	`37`	`+ $store->configureSchema($event->getSchema(), $this->getIsSameDatabaseChecker($connection));`
`47`	`38`	`}`
`48`	`39`	`}`
`49`	`40`	`}`