Skip to content

Commit c158a6a

Browse files
committed
Merge branch '5.4' into 6.0
* 5.4: [Messenger] Fix calls to deprecated DBAL methods Check for null instead of type
2 parents 8ed99e1 + d72da69 commit c158a6a

File tree

2 files changed

+19
-4
lines changed

2 files changed

+19
-4
lines changed

src/Symfony/Component/Messenger/Bridge/Doctrine/Transport/Connection.php

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
use Doctrine\DBAL\Schema\AbstractSchemaManager;
2525
use Doctrine\DBAL\Schema\Comparator;
2626
use Doctrine\DBAL\Schema\Schema;
27+
use Doctrine\DBAL\Schema\SchemaDiff;
2728
use Doctrine\DBAL\Schema\Synchronizer\SchemaSynchronizer;
2829
use Doctrine\DBAL\Schema\Table;
2930
use Doctrine\DBAL\Types\Types;
@@ -467,8 +468,9 @@ private function updateSchema(): void
467468
return;
468469
}
469470

470-
$comparator = new Comparator();
471-
$schemaDiff = $comparator->compare($this->createSchemaManager()->createSchema(), $this->getSchema());
471+
$schemaManager = $this->createSchemaManager();
472+
$comparator = $this->createComparator($schemaManager);
473+
$schemaDiff = $this->compareSchemas($comparator, $schemaManager->createSchema(), $this->getSchema());
472474

473475
foreach ($schemaDiff->toSaveSql($this->driverConnection->getDatabasePlatform()) as $sql) {
474476
if (method_exists($this->driverConnection, 'executeStatement')) {
@@ -485,6 +487,20 @@ private function createSchemaManager(): AbstractSchemaManager
485487
? $this->driverConnection->createSchemaManager()
486488
: $this->driverConnection->getSchemaManager();
487489
}
490+
491+
private function createComparator(AbstractSchemaManager $schemaManager): Comparator
492+
{
493+
return method_exists($schemaManager, 'createComparator')
494+
? $schemaManager->createComparator()
495+
: new Comparator();
496+
}
497+
498+
private function compareSchemas(Comparator $comparator, Schema $from, Schema $to): SchemaDiff
499+
{
500+
return method_exists($comparator, 'compareSchemas')
501+
? $comparator->compareSchemas($from, $to)
502+
: $comparator->compare($from, $to);
503+
}
488504
}
489505

490506
if (!class_exists(\Symfony\Component\Messenger\Transport\Doctrine\Connection::class, false)) {

src/Symfony/Component/Security/Http/Firewall/LogoutListener.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,7 @@ public function authenticate(RequestEvent $event)
8686
$logoutEvent = new LogoutEvent($request, $this->tokenStorage->getToken());
8787
$this->eventDispatcher->dispatch($logoutEvent);
8888

89-
$response = $logoutEvent->getResponse();
90-
if (!$response instanceof Response) {
89+
if (!$response = $logoutEvent->getResponse()) {
9190
throw new \RuntimeException('No logout listener set the Response, make sure at least the DefaultLogoutListener is registered.');
9291
}
9392

0 commit comments

Comments
 (0)