Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 4537f85

Browse filesBrowse files
feature #40029 [DoctineBridge] Remove UuidV*Generator classes (nicolas-grekas)
This PR was merged into the 5.3-dev branch. Discussion ---------- [DoctineBridge] Remove UuidV*Generator classes | Q | A | ------------- | --- | Branch? | 5.x | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - There is no benefit in using these classes over creating the UUIDs inline. Let's keep things simple. For `UlidGenerator`, I'm keeping it as it will provide something useful once #39507 and doctrine/DoctrineBundle#1284 are finished. Commits ------- 3c296bd [DoctineBridge] Remove UuidV*Generator
2 parents 0f4c905 + 3c296bd commit 4537f85
Copy full SHA for 4537f85

9 files changed

+7
-184
lines changed

‎UPGRADE-5.3.md

Copy file name to clipboardExpand all lines: UPGRADE-5.3.md
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,11 @@ Asset
66

77
* Deprecated `RemoteJsonManifestVersionStrategy`, use `JsonManifestVersionStrategy` instead
88

9+
DoctrineBridge
10+
--------------
11+
12+
* Remove `UuidV*Generator` classes
13+
914
DomCrawler
1015
----------
1116

‎src/Symfony/Bridge/Doctrine/CHANGELOG.md

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/CHANGELOG.md
+2-1Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ CHANGELOG
44
5.3
55
---
66

7-
* deprecated `DoctrineTestHelper` and `TestRepositoryFactory`
7+
* Deprecate `DoctrineTestHelper` and `TestRepositoryFactory`
8+
* [BC BREAK] Remove `UuidV*Generator` classes
89

910
5.2.0
1011
-----

‎src/Symfony/Bridge/Doctrine/IdGenerator/UlidGenerator.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/IdGenerator/UlidGenerator.php
-3Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,6 @@
1515
use Doctrine\ORM\Id\AbstractIdGenerator;
1616
use Symfony\Component\Uid\Ulid;
1717

18-
/**
19-
* @experimental in 5.2
20-
*/
2118
final class UlidGenerator extends AbstractIdGenerator
2219
{
2320
public function generate(EntityManager $em, $entity): Ulid

‎src/Symfony/Bridge/Doctrine/IdGenerator/UuidV1Generator.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/IdGenerator/UuidV1Generator.php
-27Lines changed: 0 additions & 27 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/IdGenerator/UuidV4Generator.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/IdGenerator/UuidV4Generator.php
-27Lines changed: 0 additions & 27 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/IdGenerator/UuidV6Generator.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/IdGenerator/UuidV6Generator.php
-27Lines changed: 0 additions & 27 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV1GeneratorTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV1GeneratorTest.php
-33Lines changed: 0 additions & 33 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV4GeneratorTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV4GeneratorTest.php
-33Lines changed: 0 additions & 33 deletions
This file was deleted.

‎src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV6GeneratorTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Doctrine/Tests/IdGenerator/UuidV6GeneratorTest.php
-33Lines changed: 0 additions & 33 deletions
This file was deleted.

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.