Skip to content

Commit

Permalink
feature #40029 [DoctineBridge] Remove UuidV*Generator classes (nicola…
Browse files Browse the repository at this point in the history
…s-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
  • Loading branch information
nicolas-grekas committed Jan 29, 2021
2 parents 0f4c905 + 3c296bd commit 4537f85
Show file tree
Hide file tree
Showing 9 changed files with 7 additions and 184 deletions.
5 changes: 5 additions & 0 deletions UPGRADE-5.3.md
Expand Up @@ -6,6 +6,11 @@ Asset

* Deprecated `RemoteJsonManifestVersionStrategy`, use `JsonManifestVersionStrategy` instead

DoctrineBridge
--------------

* Remove `UuidV*Generator` classes

DomCrawler
----------

Expand Down
3 changes: 2 additions & 1 deletion src/Symfony/Bridge/Doctrine/CHANGELOG.md
Expand Up @@ -4,7 +4,8 @@ CHANGELOG
5.3
---

* deprecated `DoctrineTestHelper` and `TestRepositoryFactory`
* Deprecate `DoctrineTestHelper` and `TestRepositoryFactory`
* [BC BREAK] Remove `UuidV*Generator` classes

5.2.0
-----
Expand Down
3 changes: 0 additions & 3 deletions src/Symfony/Bridge/Doctrine/IdGenerator/UlidGenerator.php
Expand Up @@ -15,9 +15,6 @@
use Doctrine\ORM\Id\AbstractIdGenerator;
use Symfony\Component\Uid\Ulid;

/**
* @experimental in 5.2
*/
final class UlidGenerator extends AbstractIdGenerator
{
public function generate(EntityManager $em, $entity): Ulid
Expand Down
27 changes: 0 additions & 27 deletions src/Symfony/Bridge/Doctrine/IdGenerator/UuidV1Generator.php

This file was deleted.

27 changes: 0 additions & 27 deletions src/Symfony/Bridge/Doctrine/IdGenerator/UuidV4Generator.php

This file was deleted.

27 changes: 0 additions & 27 deletions src/Symfony/Bridge/Doctrine/IdGenerator/UuidV6Generator.php

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit 4537f85

Please sign in to comment.