Skip to content

Commit ecc3273

Browse files
derrabusderrabus
committed
Kill DBAL 2 support
1 parent aadd474 commit ecc3273

File tree

43 files changed

+255
-1187
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

43 files changed

+255
-1187
lines changed

‎.github/expected-missing-return-types.diff

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,6 @@ diff --git a/src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php
1111
index ada5fcbd49..51af652f08 100644
1212
--- a/src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php
1313
+++ b/src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php
14-
@@ -51,5 +51,5 @@ class DoctrineDataCollector extends DataCollector
15-
* @return void
16-
*/
17-
- public function addLogger(string $name, DebugStack $logger)
18-
+ public function addLogger(string $name, DebugStack $logger): void
19-
{
20-
$this->loggers[$name] = $logger;
2114
@@ -59,5 +59,5 @@ class DoctrineDataCollector extends DataCollector
2215
* @return void
2316
*/
@@ -201,17 +194,6 @@ index c096b558db..8d584900a9 100644
201194
+ public function configureOptions(OptionsResolver $resolver): void
202195
{
203196
parent::configureOptions($resolver);
204-
diff --git a/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php b/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php
205-
index b2369e95d6..c33484608e 100644
206-
--- a/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php
207-
+++ b/src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php
208-
@@ -52,5 +52,5 @@ class DbalLogger implements SQLLogger
209-
* @return void
210-
*/
211-
- protected function log(string $message, array $params)
212-
+ protected function log(string $message, array $params): void
213-
{
214-
$this->logger->debug($message, $params);
215197
diff --git a/src/Symfony/Bridge/Doctrine/Messenger/DoctrineClearEntityManagerWorkerSubscriber.php b/src/Symfony/Bridge/Doctrine/Messenger/DoctrineClearEntityManagerWorkerSubscriber.php
216198
index 38618fc15e..eb599eb0b4 100644
217199
--- a/src/Symfony/Bridge/Doctrine/Messenger/DoctrineClearEntityManagerWorkerSubscriber.php

‎UPGRADE-7.0.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,13 @@ Symfony 6.4 and Symfony 7.0 will be released simultaneously at the end of Novemb
55
release process, both versions will have the same features, but Symfony 7.0 won't include any deprecated features.
66
To upgrade, make sure to resolve all deprecation notices.
77

8+
DoctrineBridge
9+
--------------
10+
11+
* Remove `DoctrineDbalCacheAdapterSchemaSubscriber`, use `DoctrineDbalCacheAdapterSchemaListener` instead
12+
* Remove `MessengerTransportDoctrineSchemaSubscriber`, use `MessengerTransportDoctrineSchemaListener` instead
13+
* Remove `RememberMeTokenProviderDoctrineSchemaSubscriber`, use `RememberMeTokenProviderDoctrineSchemaListener` instead
14+
815
Serializer
916
----------
1017

‎composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@
131131
"doctrine/annotations": "^1.13.1|^2",
132132
"doctrine/collections": "^1.0|^2.0",
133133
"doctrine/data-fixtures": "^1.1",
134-
"doctrine/dbal": "^2.13.1|^3.0",
134+
"doctrine/dbal": "^3.6",
135135
"doctrine/orm": "^2.15",
136136
"dragonmantank/cron-expression": "^3",
137137
"egulias/email-validator": "^2.1.10|^3.1|^4",
@@ -163,7 +163,7 @@
163163
"ext-psr": "<1.1|>=2",
164164
"async-aws/core": "<1.5",
165165
"doctrine/annotations": "<1.13.1",
166-
"doctrine/dbal": "<2.13.1",
166+
"doctrine/dbal": "<3.6",
167167
"doctrine/orm": "<2.15",
168168
"egulias/email-validator": "~3.0.0",
169169
"masterminds/html5": "<2.6",

‎src/Symfony/Bridge/Doctrine/DataCollector/DoctrineDataCollector.php

Lines changed: 4 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111

1212
namespace Symfony\Bridge\Doctrine\DataCollector;
1313

14-
use Doctrine\DBAL\Logging\DebugStack;
1514
use Doctrine\DBAL\Types\ConversionException;
1615
use Doctrine\DBAL\Types\Type;
1716
use Doctrine\Persistence\ManagerRegistry;
@@ -32,35 +31,12 @@ class DoctrineDataCollector extends DataCollector
3231
private array $connections;
3332
private array $managers;
3433

35-
/**
36-
* @var array<string, DebugStack>
37-
*/
38-
private array $loggers = [];
39-
4034
public function __construct(
4135
private ManagerRegistry $registry,
42-
private ?DebugDataHolder $debugDataHolder = null,
36+
private DebugDataHolder $debugDataHolder,
4337
) {
4438
$this->connections = $registry->getConnectionNames();
4539
$this->managers = $registry->getManagerNames();
46-
47-
if (null === $debugDataHolder) {
48-
trigger_deprecation('symfony/doctrine-bridge', '6.4', 'Not passing an instance of "%s" as "$debugDataHolder" to "%s()" is deprecated.', DebugDataHolder::class, __METHOD__);
49-
}
50-
}
51-
52-
/**
53-
* Adds the stack logger for a connection.
54-
*
55-
* @return void
56-
*
57-
* @deprecated since Symfony 6.4, use a DebugDataHolder instead.
58-
*/
59-
public function addLogger(string $name, DebugStack $logger)
60-
{
61-
trigger_deprecation('symfony/doctrine-bridge', '6.4', '"%s()" is deprecated. Pass an instance of "%s" to the constructor instead.', __METHOD__, DebugDataHolder::class);
62-
63-
$this->loggers[$name] = $logger;
6440
}
6541

6642
/**
@@ -79,16 +55,8 @@ private function collectQueries(): array
7955
{
8056
$queries = [];
8157

82-
if (null !== $this->debugDataHolder) {
83-
foreach ($this->debugDataHolder->getData() as $name => $data) {
84-
$queries[$name] = $this->sanitizeQueries($name, $data);
85-
}
86-
87-
return $queries;
88-
}
89-
90-
foreach ($this->loggers as $name => $logger) {
91-
$queries[$name] = $this->sanitizeQueries($name, $logger->queries);
58+
foreach ($this->debugDataHolder->getData() as $name => $data) {
59+
$queries[$name] = $this->sanitizeQueries($name, $data);
9260
}
9361

9462
return $queries;
@@ -100,17 +68,7 @@ private function collectQueries(): array
10068
public function reset()
10169
{
10270
$this->data = [];
103-
104-
if (null !== $this->debugDataHolder) {
105-
$this->debugDataHolder->reset();
106-
107-
return;
108-
}
109-
110-
foreach ($this->loggers as $logger) {
111-
$logger->queries = [];
112-
$logger->currentQuery = 0;
113-
}
71+
$this->debugDataHolder->reset();
11472
}
11573

11674
public function getManagers()

‎src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
namespace Symfony\Bridge\Doctrine\Form\ChoiceList;
1313

1414
use Doctrine\DBAL\ArrayParameterType;
15-
use Doctrine\DBAL\Connection;
1615
use Doctrine\DBAL\Types\ConversionException;
1716
use Doctrine\DBAL\Types\Type;
1817
use Doctrine\ORM\QueryBuilder;
@@ -71,13 +70,13 @@ public function getEntitiesByIds(string $identifier, array $values): array
7170
$entity = current($qb->getRootEntities());
7271
$metadata = $qb->getEntityManager()->getClassMetadata($entity);
7372
if (\in_array($type = $metadata->getTypeOfField($identifier), ['integer', 'bigint', 'smallint'])) {
74-
$parameterType = class_exists(ArrayParameterType::class) ? ArrayParameterType::INTEGER : Connection::PARAM_INT_ARRAY;
73+
$parameterType = ArrayParameterType::INTEGER;
7574

7675
// Filter out non-integer values (e.g. ""). If we don't, some
7776
// databases such as PostgreSQL fail.
7877
$values = array_values(array_filter($values, fn ($v) => (string) $v === (string) (int) $v || ctype_digit($v)));
7978
} elseif (\in_array($type, ['ulid', 'uuid', 'guid'])) {
80-
$parameterType = class_exists(ArrayParameterType::class) ? ArrayParameterType::STRING : Connection::PARAM_STR_ARRAY;
79+
$parameterType = ArrayParameterType::STRING;
8180

8281
// Like above, but we just filter out empty strings.
8382
$values = array_values(array_filter($values, fn ($v) => '' !== (string) $v));
@@ -96,7 +95,7 @@ public function getEntitiesByIds(string $identifier, array $values): array
9695
unset($value);
9796
}
9897
} else {
99-
$parameterType = class_exists(ArrayParameterType::class) ? ArrayParameterType::STRING : Connection::PARAM_STR_ARRAY;
98+
$parameterType = ArrayParameterType::STRING;
10099
}
101100
if (!$values) {
102101
return [];

‎src/Symfony/Bridge/Doctrine/Logger/DbalLogger.php

Lines changed: 0 additions & 91 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/SchemaListener/DoctrineDbalCacheAdapterSchemaSubscriber.php

Lines changed: 0 additions & 39 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/SchemaListener/MessengerTransportDoctrineSchemaSubscriber.php

Lines changed: 0 additions & 43 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/SchemaListener/RememberMeTokenProviderDoctrineSchemaSubscriber.php

Lines changed: 0 additions & 39 deletions
This file was deleted.

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy