Deprecate passing timezone information to methods where it is ignored

Author: phansys

src/Types/DateImmutableType.php

@@ -35,6 +35,22 @@ public function convertToDatabaseValue($value, AbstractPlatform $platform)
         }
 
         if ($value instanceof DateTimeImmutable) {
+            $offset        = $value->format('O');
+            $defaultOffset = (new DateTimeImmutable())->format('O');
+
+            if ($offset !== $defaultOffset) {
+                Deprecation::triggerIfCalledFromOutside(
+                    'doctrine/dbal',
+                    'https://github.com/doctrine/dbal/pull/6020',
+                    'Passing a timezone offset (%s) different than the default one (%s) is deprecated'
+                    . ' as it will be lost, use %s::%s() instead.',
+                    $offset,
+                    $defaultOffset,
+                    DateTimeTzImmutableType::class,
+                    __FUNCTION__,
+                );
+            }
+
             return $value->format($platform->getDateFormatString());
         }
 
@@ -56,7 +72,27 @@ public function convertToDatabaseValue($value, AbstractPlatform $platform)
      */
     public function convertToPHPValue($value, AbstractPlatform $platform)
     {
-        if ($value === null || $value instanceof DateTimeImmutable) {
+        if ($value === null) {
+            return null;
+        }
+
+        if ($value instanceof DateTimeImmutable) {
+            $offset        = $value->format('O');
+            $defaultOffset = (new DateTimeImmutable())->format('O');
+
+            if ($offset !== $defaultOffset) {
+                Deprecation::triggerIfCalledFromOutside(
+                    'doctrine/dbal',
+                    'https://github.com/doctrine/dbal/pull/6020',
+                    'Passing a timezone offset (%s) different than the default one (%s) is deprecated'
+                    . ' as it may be lost, use %s::%s() instead.',
+                    $offset,
+                    $defaultOffset,
+                    DateTimeTzImmutableType::class,
+                    __FUNCTION__,
+                );
+            }
+
             return $value;
         }
 

tests/Types/DateImmutableTypeTest.php

@@ -4,17 +4,22 @@
 
 use DateTime;
 use DateTimeImmutable;
+use DateTimeZone;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Platforms\AbstractPlatform;
 use Doctrine\DBAL\Types\ConversionException;
 use Doctrine\DBAL\Types\DateImmutableType;
+use Doctrine\Deprecations\PHPUnit\VerifyDeprecations;
+use InvalidArgumentException;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 
 use function get_class;
 
 class DateImmutableTypeTest extends TestCase
 {
+    use VerifyDeprecations;
+
     /** @var AbstractPlatform&MockObject */
     private AbstractPlatform $platform;
 
@@ -48,10 +53,18 @@ public function testConvertsDateTimeImmutableInstanceToDatabaseValue(): void
         $this->platform->expects(self::once())
             ->method('getDateFormatString')
             ->willReturn('Y-m-d');
-        $date->expects(self::once())
+        $date->expects(self::exactly(2))
             ->method('format')
-            ->with('Y-m-d')
-            ->willReturn('2016-01-01');
+            ->willReturnCallback(static function (string $format): string {
+                switch ($format) {
+                    case 'O':
+                        return 'UTC';
+                    case 'Y-m-d':
+                        return '2016-01-01';
+                    default:
+                        throw new InvalidArgumentException();
+                }
+            });
 
         self::assertSame(
             '2016-01-01',
@@ -117,4 +130,42 @@ public function testRequiresSQLCommentHint(): void
     {
         self::assertTrue($this->type->requiresSQLCommentHint($this->platform));
     }
+
+    /** @dataProvider provideDateTimeInstancesWithTimezone */
+    public function testTimezoneDeprecationFromConvertsToDatabaseValue(DateTimeImmutable $date): void
+    {
+        $defaultOffset = (new DateTimeImmutable())->format('O');
+
+        self::assertFalse($defaultOffset === $date->format('O'));
+
+        $this->platform->expects(self::once())
+            ->method('getDateFormatString')
+            ->willReturn('Y-m-d');
+
+        $this->expectDeprecationWithIdentifier('https://github.com/doctrine/dbal/pull/6020');
+
+        $this->type->convertToDatabaseValue($date, $this->platform);
+    }
+
+    /** @psalm-return iterable<string, array<0: DateTimeImmutable>> */
+    public function provideDateTimeInstancesWithTimezone(): iterable
+    {
+        yield 'timezone' => [
+            (new DateTimeImmutable())->setTimezone(new DateTimeZone('America/Argentina/Buenos_Aires')),
+        ];
+        yield 'offset' => [
+            (new DateTimeImmutable())->setTimezone(new DateTimeZone('-0300')),
+        ];
+    }
+
+    public function testNoTimezoneFromConvertsToDatabaseValue(): void
+    {
+        $this->platform->expects(self::once())
+            ->method('getDateFormatString')
+            ->willReturn('Y-m-d');
+
+        $this->expectNoDeprecationWithIdentifier('https://github.com/doctrine/dbal/pull/6020');
+
+        $this->type->convertToDatabaseValue(new DateTimeImmutable(), $this->platform);
+    }
 }