Merge branch '3.8.x' into 4.0.x

* 3.8.x:
  Generalize CustomIntrospectionTest (doctrine#6210)
  Run tests on MySQL 8.2 (doctrine#6207)
  MySQLSchemaManager. Check expected database type for json columns only. (doctrine#6189)
  Make the type annotation for CompositeExpression::count more specific (doctrine#6188)

Author: derrabus

.github/workflows/continuous-integration.yml

@@ -349,7 +349,7 @@ jobs:
         mysql-version:
           - "5.7"
           - "8.0"
-          - "8.1"
+          - "8.2"
         extension:
           - "mysqli"
           - "pdo_mysql"

src/Query/Expression/CompositeExpression.php

@@ -68,6 +68,8 @@ public function with(self|string $part, self|string ...$parts): self
 
     /**
      * Retrieves the amount of expressions on composite expression.
+     *
+     * @psalm-return int<0, max>
      */
     public function count(): int
     {

tests/Functional/Schema/CustomIntrospectionTest.php

@@ -0,0 +1,68 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Functional\Schema;
+
+use Doctrine\DBAL\Schema\ColumnDiff;
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\DBAL\Tests\Functional\Schema\Types\MoneyType;
+use Doctrine\DBAL\Tests\FunctionalTestCase;
+use Doctrine\DBAL\Tests\TestUtil;
+use Doctrine\DBAL\Types\Type;
+
+use function array_map;
+use function implode;
+use function sprintf;
+
+/**
+ * Tests introspection of a custom column type with an underlying decimal column
+ * on MySQL and MariaDb platforms.
+ *
+ * See bug #6185
+ */
+class CustomIntrospectionTest extends FunctionalTestCase
+{
+    public static function setUpBeforeClass(): void
+    {
+        if (TestUtil::isDriverOneOf('oci8', 'pdo_oci')) {
+            self::markTestSkipped('Skip on Oracle');
+        }
+
+        Type::addType(MoneyType::NAME, MoneyType::class);
+    }
+
+    public function testCustomColumnIntrospection(): void
+    {
+        $tableName     = 'test_custom_column_introspection';
+        $schemaManager = $this->connection->createSchemaManager();
+        $schema        = new Schema([], [], $schemaManager->createSchemaConfig());
+        $table         = $schema->createTable($tableName);
+
+        $table->addColumn('id', 'integer');
+        $table->addColumn('quantity', 'decimal', [
+            'notnull' => false,
+            'scale' => 2,
+            'precision' => 10,
+        ]);
+        $table->addColumn('amount', 'money', [
+            'notnull' => false,
+            'scale' => 2,
+            'precision' => 10,
+        ]);
+
+        $this->dropAndCreateTable($table);
+
+        $onlineTable = $schemaManager->introspectTable($tableName);
+        $diff        = $schemaManager->createComparator()->compareTables($onlineTable, $table);
+        $changedCols = array_map(
+            static fn (ColumnDiff $columnDiff): string => $columnDiff->getOldColumn()->getName(),
+            $diff->getModifiedColumns(),
+        );
+
+        self::assertTrue($diff->isEmpty(), sprintf(
+            'Tables should be identical. Differences detected in %s.',
+            implode(', ', $changedCols),
+        ));
+    }
+}

tests/Functional/Schema/Types/Money.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Functional\Schema\Types;
+
+final class Money
+{
+    public function __construct(
+        private readonly string $value,
+    ) {
+    }
+
+    public function __toString(): string
+    {
+        return $this->value;
+    }
+}

tests/Functional/Schema/Types/MoneyType.php

@@ -0,0 +1,56 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Functional\Schema\Types;
+
+use Doctrine\DBAL\Platforms\AbstractPlatform;
+use Doctrine\DBAL\Types\Exception\InvalidFormat;
+use Doctrine\DBAL\Types\Exception\InvalidType;
+use Doctrine\DBAL\Types\Type;
+use InvalidArgumentException;
+
+use function is_string;
+
+class MoneyType extends Type
+{
+    public const NAME = 'money';
+
+    /**
+     * {@inheritDoc}
+     */
+    public function getSQLDeclaration(array $column, AbstractPlatform $platform): string
+    {
+        return $platform->getDecimalTypeDeclarationSQL($column);
+    }
+
+    public function convertToDatabaseValue(mixed $value, AbstractPlatform $platform): ?string
+    {
+        if ($value === null) {
+            return null;
+        }
+
+        if ($value instanceof Money) {
+            return (string) $value;
+        }
+
+        throw InvalidType::new($value, self::NAME, ['null', Money::class]);
+    }
+
+    public function convertToPHPValue(mixed $value, AbstractPlatform $platform): ?Money
+    {
+        if ($value === null || $value instanceof Money) {
+            return $value;
+        }
+
+        if (! is_string($value)) {
+            throw InvalidType::new($value, self::NAME, ['null', 'string']);
+        }
+
+        try {
+            return new Money($value);
+        } catch (InvalidArgumentException $e) {
+            throw InvalidFormat::new($value, self::NAME, Money::class, $e);
+        }
+    }
+}