Propagate TypeRegistry through SchemaConfig to Table

SchemaConfig now carries a TypeRegistry that is populated by
AbstractSchemaManager::createSchemaConfig() from the connection
configuration. Schema::createTable() forwards it to each Table so
that Table::addColumn() resolves types from the per-connection
TypeRegistry instead of falling back to the global static registry.

Table's constructor parameter is changed from ?Configuration to
?TypeRegistry directly, since Configuration was only needed to reach
its TypeRegistry.

A TODO comment is added to ColumnDiff::hasTypeChanged() noting that
the current class-based comparison is insufficient now that types are
services: same-class aliases (json / json_object) produce false
negatives, and distinct service instances of the same class would not
be detected. The fix (identity comparison) is left for a follow-up.

Author: GromNaN

src/Schema/AbstractSchemaManager.php

@@ -1456,6 +1456,7 @@ public function createSchemaConfig(): SchemaConfig
         }
 
         $schemaConfig->setDefaultTableOptions($params['defaultTableOptions']);
+        $schemaConfig->setTypeRegistry($this->connection->getConfiguration()->getTypeRegistry());
 
         return $schemaConfig;
     }

src/Schema/ColumnDiff.php

@@ -54,6 +54,13 @@ public function hasNameChanged(): bool
 
     public function hasTypeChanged(): bool
     {
+        // TODO: This comparison by class is insufficient now that types can be distinct services sharing the same
+        // class (e.g. two differently configured instances of the same Type subclass). It also produces false
+        // negatives for built-in aliases that share a class: json and json_object both map to JsonType::class,
+        // so switching between them is not detected as a change.
+        // The fix is to compare Type instances by identity (===) once both the introspected schema and the
+        // target schema are guaranteed to resolve types from the same TypeRegistry, so the flyweight invariant
+        // (one instance per registered name) holds across both sides of the diff.
         return $this->newColumn->getType()::class !== $this->oldColumn->getType()::class;
     }
 

src/Schema/Schema.php

@@ -72,6 +72,7 @@ class Schema extends AbstractAsset
 
     protected SchemaConfig $_schemaConfig;
 
+
     /**
      * Indicates whether the schema uses unqualified names for its objects. Once this flag is set to true, it won't be
      * unset even after the objects with unqualified names have been dropped from the schema.
@@ -391,7 +392,7 @@ public function createNamespace(string $name): self
      */
     public function createTable(string $name): Table
     {
-        $table = new Table($name, [], [], [], [], [], $this->_schemaConfig->toTableConfiguration());
+        $table = new Table($name, [], [], [], [], [], $this->_schemaConfig->toTableConfiguration(), null, $this->_schemaConfig->getTypeRegistry());
         $this->_addTable($table);
 
         foreach ($this->_schemaConfig->getDefaultTableOptions() as $option => $value) {

src/Schema/SchemaConfig.php

@@ -4,11 +4,14 @@
 
 namespace Doctrine\DBAL\Schema;
 
+use Doctrine\DBAL\Types\TypeRegistry;
+
 /**
  * Configuration for a Schema.
  */
 class SchemaConfig
 {
+    private ?TypeRegistry $typeRegistry = null;
     /** @var positive-int */
     protected int $maxIdentifierLength = 63;
 
@@ -67,6 +70,16 @@ public function setDefaultTableOptions(array $defaultTableOptions): void
         $this->defaultTableOptions = $defaultTableOptions;
     }
 
+    public function getTypeRegistry(): ?TypeRegistry
+    {
+        return $this->typeRegistry;
+    }
+
+    public function setTypeRegistry(TypeRegistry $typeRegistry): void
+    {
+        $this->typeRegistry = $typeRegistry;
+    }
+
     public function toTableConfiguration(): TableConfiguration
     {
         return new TableConfiguration($this->maxIdentifierLength);

src/Schema/Table.php

@@ -4,7 +4,6 @@
 
 namespace Doctrine\DBAL\Schema;
 
-use Doctrine\DBAL\Configuration;
 use Doctrine\DBAL\Schema\Exception\ColumnAlreadyExists;
 use Doctrine\DBAL\Schema\Exception\ColumnDoesNotExist;
 use Doctrine\DBAL\Schema\Exception\ForeignKeyDoesNotExist;
@@ -21,6 +20,7 @@
 use Doctrine\DBAL\Schema\Name\UnqualifiedName;
 use Doctrine\DBAL\Types\Exception\TypesException;
 use Doctrine\DBAL\Types\Type;
+use Doctrine\DBAL\Types\TypeRegistry;
 use Doctrine\Deprecations\Deprecation;
 use LogicException;
 
@@ -101,7 +101,7 @@ public function __construct(
         array $options = [],
         ?TableConfiguration $configuration = null,
         ?PrimaryKeyConstraint $primaryKeyConstraint = null,
-        private ?Configuration $dbalConfiguration = null,
+        private ?TypeRegistry $typeRegistry = null,
     ) {
         if ($name === '') {
             throw InvalidTableName::new($name);
@@ -388,7 +388,7 @@ public function columnsAreIndexed(array $columnNames): bool
      */
     public function addColumn(string $name, string $typeName, array $options = []): Column
     {
-        $type = $this->dbalConfiguration?->getTypeRegistry()->get($typeName) ?? Type::getType($typeName);
+        $type = $this->typeRegistry !== null ? $this->typeRegistry->get($typeName) : Type::getType($typeName);
 
         $column = new Column($name, $type, $options);