Fix handling timestamp with time zone in Oracle

Author: phansys

src/Driver/OCI8/Middleware/InitializeSession.php

@@ -28,7 +28,7 @@ public function connect(
                         . " NLS_TIME_FORMAT = 'HH24:MI:SS'"
                         . " NLS_DATE_FORMAT = 'YYYY-MM-DD HH24:MI:SS'"
                         . " NLS_TIMESTAMP_FORMAT = 'YYYY-MM-DD HH24:MI:SS'"
-                        . " NLS_TIMESTAMP_TZ_FORMAT = 'YYYY-MM-DD HH24:MI:SS TZH:TZM'"
+                        . " NLS_TIMESTAMP_TZ_FORMAT = 'YYYY-MM-DD HH24:MI:SSTZH:TZM'"
                         . " NLS_NUMERIC_CHARACTERS = '.,'",
                 );
 

src/Platforms/OraclePlatform.php

@@ -82,6 +82,8 @@ public function getNowExpression($type = 'timestamp')
         );
 
         switch ($type) {
+            case 'timestamptz':
+                return 'TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SSTZH:TZM\')';
             case 'date':
             case 'time':
             case 'timestamp':

tests/Functional/Types/DateTimeTzImmutableTypeTest.php

@@ -0,0 +1,84 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Doctrine\DBAL\Tests\Functional\Types;
+
+use DateTimeImmutable;
+use DateTimeZone;
+use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Tests\FunctionalTestCase;
+use Doctrine\DBAL\Tests\TestUtil;
+use Doctrine\DBAL\Types\Type;
+use Doctrine\DBAL\Types\Types;
+
+use function sprintf;
+
+class DateTimeTzImmutableTypeTest extends FunctionalTestCase
+{
+    private const TEST_TABLE = 'datetimetz_test';
+
+    protected function setUp(): void
+    {
+        if (! TestUtil::isDriverOneOf('pdo_oci', 'oci8')) {
+            self::markTestSkipped('This test is only allowed for "pdo_oci" and "oci8" by now.');
+        }
+
+        $table = new Table(self::TEST_TABLE);
+        $table->addColumn('id', Types::INTEGER);
+
+        $table->addColumn('val', Types::DATETIMETZ_IMMUTABLE);
+        $table->setPrimaryKey(['id']);
+
+        $this->dropAndCreateTable($table);
+    }
+
+    public function testInsertAndSelect(): void
+    {
+        $platform                = $this->connection->getDatabasePlatform();
+        $dateTimeTzImmutableType = Type::getType(Types::DATETIMETZ_IMMUTABLE);
+
+        $id1    = 1;
+        $value1 = new DateTimeImmutable('1986-03-22 19:45:30', new DateTimeZone('America/Argentina/Buenos_Aires'));
+
+        $this->insert($id1, $value1);
+
+        $res1 = $this->select($id1);
+
+        $resultDateTimeTzValue = $dateTimeTzImmutableType
+            ->convertToPHPValue($res1, $platform)
+            ->setTimezone(new DateTimeZone('UTC'));
+
+        self::assertInstanceOf(DateTimeImmutable::class, $resultDateTimeTzValue);
+        self::assertSame($value1->getTimestamp(), $resultDateTimeTzValue->getTimestamp());
+        self::assertSame($value1->getTimestamp(), $resultDateTimeTzValue->getTimestamp());
+        self::assertSame('UTC', $resultDateTimeTzValue->format('T'));
+        self::assertSame('1986-03-22T22:45:30+00:00', $resultDateTimeTzValue->format(DateTimeImmutable::ATOM));
+    }
+
+    private function insert(int $id, DateTimeImmutable $value): void
+    {
+        $result = $this->connection->insert(self::TEST_TABLE, [
+            'id'  => $id,
+            'val' => $value,
+        ], [
+            Types::INTEGER,
+            Type::getType(Types::DATETIMETZ_IMMUTABLE),
+        ]);
+
+        self::assertSame(1, $result);
+    }
+
+    private function select(int $id): string
+    {
+        $value = $this->connection->fetchOne(
+            sprintf('SELECT val FROM %s WHERE id = ?', self::TEST_TABLE),
+            [$id],
+            [Types::INTEGER],
+        );
+
+        self::assertIsString($value);
+
+        return $value;
+    }
+}

tests/Platforms/OraclePlatformTest.php

@@ -926,6 +926,23 @@ public function asciiStringSqlDeclarationDataProvider(): array
         ];
     }
 
+    /** @psalm-return iterable<int, array{string, string}> */
+    public function getNowExpressionCases(): iterable
+    {
+        yield ['TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SSTZH:TZM\')', 'timestamptz'];
+        yield ['TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SS\')', 'date'];
+        yield ['TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SS\')', 'time'];
+        yield ['TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SS\')', 'timestamp'];
+        yield ['TO_CHAR(CURRENT_TIMESTAMP, \'YYYY-MM-DD HH24:MI:SS\')', 'unknown_unsupported_type'];
+    }
+
+    /** @dataProvider getNowExpressionCases */
+    public function testGetNowExpression(string $expected, string $type): void
+    {
+        /** @psalm-suppress DeprecatedMethod */
+        self::assertSame($expected, $this->platform->getNowExpression($type));
+    }
+
     protected function getLimitOffsetCastToIntExpectedQuery(): string
     {
         return 'SELECT * FROM (SELECT a.*, ROWNUM AS doctrine_rownum FROM (SELECT * FROM user) a WHERE ROWNUM <= 3)'