Drop Flyweight pattern

There is a growing need for types with parameters, and that pattern was
probably used out of concerns that are no longer relevant anyway.

src/Exception.php

@@ -137,11 +137,4 @@ public static function typeNotRegistered(Type $type): self
             sprintf('Type of the class %s@%s is not registered.', get_class($type), spl_object_hash($type))
         );
     }
-
-    public static function typeAlreadyRegistered(Type $type): self
-    {
-        return new self(
-            sprintf('Type of the class %s@%s is already registered.', get_class($type), spl_object_hash($type))
-        );
-    }
 }

src/Types/Type.php

@@ -130,7 +130,6 @@ private static function createTypeRegistry(): TypeRegistry
 
     /**
      * Factory method to create type instances.
-     * Type instances are implemented as flyweights.
      *
      * @param string $name The name of the type (as returned by getName()).
      *

src/Types/TypeRegistry.php

@@ -7,15 +7,13 @@
 use Doctrine\DBAL\Exception;
 
 use function array_search;
-use function in_array;
 
 /**
  * The type registry is responsible for holding a map of all known DBAL types.
- * The types are stored using the flyweight pattern so that one type only exists as exactly one instance.
  */
 final class TypeRegistry
 {
-    /** @var array<string, Type> Map of type names and their corresponding flyweight objects. */
+    /** @var array<string, Type> Map of type names and their corresponding objects. */
     private $instances;
 
     /**
@@ -47,9 +45,9 @@ public function get(string $name): Type
      */
     public function lookupName(Type $type): string
     {
-        $name = $this->findTypeName($type);
+        $name = array_search($type, $this->instances, true);
 
-        if ($name === null) {
+        if ($name === false) {
             throw Exception::typeNotRegistered($type);
         }
 
@@ -75,10 +73,6 @@ public function register(string $name, Type $type): void
             throw Exception::typeExists($name);
         }
 
-        if ($this->findTypeName($type) !== null) {
-            throw Exception::typeAlreadyRegistered($type);
-        }
-
         $this->instances[$name] = $type;
     }
 
@@ -93,10 +87,6 @@ public function override(string $name, Type $type): void
             throw Exception::typeNotFound($name);
         }
 
-        if (! in_array($this->findTypeName($type), [$name, null], true)) {
-            throw Exception::typeAlreadyRegistered($type);
-        }
-
         $this->instances[$name] = $type;
     }
 
@@ -111,15 +101,4 @@ public function getMap(): array
     {
         return $this->instances;
     }
-
-    private function findTypeName(Type $type): ?string
-    {
-        $name = array_search($type, $this->instances, true);
-
-        if ($name === false) {
-            return null;
-        }
-
-        return $name;
-    }
 }

tests/Types/TypeRegistryTest.php

@@ -86,7 +86,7 @@ public function testRegister(): void
         self::assertSame($newType, $this->registry->get('some'));
     }
 
-    public function testRegisterWithAlradyRegisteredName(): void
+    public function testRegisterWithAlreadyRegisteredName(): void
     {
         $this->registry->register('some', new TextType());
 
@@ -99,9 +99,11 @@ public function testRegisterWithAlreadyRegisteredInstance(): void
         $newType = new TextType();
 
         $this->registry->register('some', $newType);
-
-        $this->expectException(Exception::class);
-        $this->registry->register('other', $newType);
+        $this->registry->register('sameButDifferent', $newType);
+        self::assertSame(
+            $this->registry->get('some'),
+            $this->registry->get('sameButDifferent')
+        );
     }
 
     public function testOverride(): void
@@ -125,17 +127,6 @@ public function testOverrideAllowsExistingInstance(): void
         self::assertSame($type, $this->registry->get('some'));
     }
 
-    public function testOverrideWithAlreadyRegisteredInstance(): void
-    {
-        $newType = new TextType();
-
-        $this->registry->register('first', $newType);
-        $this->registry->register('second', new StringType());
-
-        $this->expectException(Exception::class);
-        $this->registry->override('second', $newType);
-    }
-
     public function testOverrideWithUnknownType(): void
     {
         $this->expectException(Exception::class);