[DoctrineBridge] drop binary variants of UID types

Author: nicolas-grekas

CHANGELOG.md

@@ -4,7 +4,7 @@ CHANGELOG
 5.2.0
 -----
 
- * added support for symfony/uid as `UlidType`, `UuidType`, `UlidBinaryType` and `UuidBinaryType` as Doctrine types
+ * added support for symfony/uid as `UlidType` and `UuidType` as Doctrine types
  * added `UlidGenerator`, `UUidV1Generator`, `UuidV4Generator` and `UuidV6Generator`
 
 5.0.0

DependencyInjection/CompilerPass/RegisterUidTypePass.php

@@ -11,9 +11,7 @@
 
 namespace Symfony\Bridge\Doctrine\DependencyInjection\CompilerPass;
 
-use Symfony\Bridge\Doctrine\Types\UlidBinaryType;
 use Symfony\Bridge\Doctrine\Types\UlidType;
-use Symfony\Bridge\Doctrine\Types\UuidBinaryType;
 use Symfony\Bridge\Doctrine\Types\UuidType;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -40,14 +38,6 @@ public function process(ContainerBuilder $container)
             $typeDefinition['ulid'] = ['class' => UlidType::class];
         }
 
-        if (!isset($typeDefinition['uuid_binary'])) {
-            $typeDefinition['uuid_binary'] = ['class' => UuidBinaryType::class];
-        }
-
-        if (!isset($typeDefinition['ulid_binary'])) {
-            $typeDefinition['ulid_binary'] = ['class' => UlidBinaryType::class];
-        }
-
         $container->setParameter('doctrine.dbal.connection_factory.types', $typeDefinition);
     }
 }

Tests/Types/UlidBinaryTypeTest.php

@@ -37,6 +37,9 @@ public static function setUpBeforeClass(): void
     protected function setUp(): void
     {
         $this->platform = $this->createMock(AbstractPlatform::class);
+        $this->platform
+            ->method('hasNativeGuidType')
+            ->willReturn(true);
         $this->platform
             ->method('getGuidTypeDeclarationSQL')
             ->willReturn('DUMMYVARCHAR()');

Tests/Types/UlidTypeTest.php

@@ -37,6 +37,9 @@ public static function setUpBeforeClass(): void
     protected function setUp(): void
     {
         $this->platform = $this->createMock(AbstractPlatform::class);
+        $this->platform
+            ->method('hasNativeGuidType')
+            ->willReturn(true);
         $this->platform
             ->method('getGuidTypeDeclarationSQL')
             ->willReturn('DUMMYVARCHAR()');