Fix ColumnDefinitionParser

CHANGELOG.md

@@ -15,7 +15,7 @@
 - Enh #279: Separate column type constants (@Tigrov)
 - New #280, #291: Realize `ColumnBuilder` class (@Tigrov)
 - Enh #281: Update according changes in `ColumnSchemaInterface` (@Tigrov)
-- New #282, #291: Add `ColumnDefinitionBuilder` class (@Tigrov)
+- New #282, #291, #299: Add `ColumnDefinitionBuilder` class (@Tigrov)
 - Bug #285: Fix `DMLQueryBuilder::insertBatch()` method (@Tigrov)
 - Enh #283: Refactor `Dsn` class (@Tigrov)
 - Enh #286: Use constructor to create columns and initialize properties (@Tigrov)
@@ -27,6 +27,7 @@
 - Enh #296: Remove `ColumnInterface` (@Tigrov)
 - Enh #298: Rename `ColumnSchemaInterface` to `ColumnInterface` (@Tigrov)
 - Enh #298: Refactor `DMLQueryBuilder::prepareInsertValues()` method (@Tigrov)
+- Enh #299: Add `ColumnDefinitionParser` class (@Tigrov)
 
 ## 1.3.0 March 21, 2024
 

src/Column/ColumnDefinitionBuilder.php

@@ -26,7 +26,7 @@ final class ColumnDefinitionBuilder extends AbstractColumnDefinitionBuilder
         'number',
         'float',
         'timestamp',
-        'interval day(0) to second',
+        'interval day to second',
         'raw',
         'urowid',
     ];
@@ -64,37 +64,46 @@ protected function buildOnUpdate(string $onUpdate): string
 
     protected function getDbType(ColumnInterface $column): string
     {
+        $dbType = $column->getDbType();
         $size = $column->getSize();
+        $scale = $column->getScale();
 
         /** @psalm-suppress DocblockTypeContradiction */
-        return $column->getDbType() ?? match ($column->getType()) {
-            ColumnType::BOOLEAN => 'number(1)',
-            ColumnType::BIT => match (true) {
-                $size === null => 'number(38)',
-                $size <= 126 => 'number(' . ceil(log10(2 ** $size)) . ')',
-                default => 'raw(' . ceil($size / 8) . ')',
+        return match ($dbType) {
+            default => $dbType,
+            null => match ($column->getType()) {
+                ColumnType::BOOLEAN => 'number(1)',
+                ColumnType::BIT => match (true) {
+                    $size === null => 'number(38)',
+                    $size <= 126 => 'number(' . ceil(log10(2 ** $size)) . ')',
+                    default => 'raw(' . ceil($size / 8) . ')',
+                },
+                ColumnType::TINYINT => 'number(' . ($size ?? 3) . ')',
+                ColumnType::SMALLINT => 'number(' . ($size ?? 5) . ')',
+                ColumnType::INTEGER => 'number(' . ($size ?? 10) . ')',
+                ColumnType::BIGINT => 'number(' . ($size ?? 20) . ')',
+                ColumnType::FLOAT => 'binary_float',
+                ColumnType::DOUBLE => 'binary_double',
+                ColumnType::DECIMAL => 'number(' . ($size ?? 10) . ',' . ($scale ?? 0) . ')',
+                ColumnType::MONEY => 'number(' . ($size ?? 19) . ',' . ($scale ?? 4) . ')',
+                ColumnType::CHAR => 'char',
+                ColumnType::STRING => 'varchar2(' . ($size ?? 255) . ')',
+                ColumnType::TEXT => 'clob',
+                ColumnType::BINARY => 'blob',
+                ColumnType::UUID => 'raw(16)',
+                ColumnType::DATETIME => 'timestamp',
+                ColumnType::TIMESTAMP => 'timestamp',
+                ColumnType::DATE => 'date',
+                ColumnType::TIME => 'interval day(0) to second' . ($size !== null ? "($size)" : ''),
+                ColumnType::ARRAY => 'clob',
+                ColumnType::STRUCTURED => 'clob',
+                ColumnType::JSON => 'clob',
+                default => 'varchar2',
             },
-            ColumnType::TINYINT => 'number(' . ($size ?? 3) . ')',
-            ColumnType::SMALLINT => 'number(' . ($size ?? 5) . ')',
-            ColumnType::INTEGER => 'number(' . ($size ?? 10) . ')',
-            ColumnType::BIGINT => 'number(' . ($size ?? 20) . ')',
-            ColumnType::FLOAT => 'binary_float',
-            ColumnType::DOUBLE => 'binary_double',
-            ColumnType::DECIMAL => 'number(' . ($size ?? 10) . ',' . ($column->getScale() ?? 0) . ')',
-            ColumnType::MONEY => 'number(' . ($size ?? 19) . ',' . ($column->getScale() ?? 4) . ')',
-            ColumnType::CHAR => 'char',
-            ColumnType::STRING => 'varchar2(' . ($size ?? 255) . ')',
-            ColumnType::TEXT => 'clob',
-            ColumnType::BINARY => 'blob',
-            ColumnType::UUID => 'raw(16)',
-            ColumnType::DATETIME => 'timestamp',
-            ColumnType::TIMESTAMP => 'timestamp',
-            ColumnType::DATE => 'date',
-            ColumnType::TIME => 'interval day(0) to second',
-            ColumnType::ARRAY => 'clob',
-            ColumnType::STRUCTURED => 'clob',
-            ColumnType::JSON => 'clob',
-            default => 'varchar2',
+            'timestamp with time zone' => 'timestamp' . ($size !== null ? "($size)" : '') . ' with time zone',
+            'timestamp with local time zone' => 'timestamp' . ($size !== null ? "($size)" : '') . ' with local time zone',
+            'interval day to second' => 'interval day' . ($scale !== null ? "($scale)" : '') . ' to second' . ($size !== null ? "($size)" : ''),
+            'interval year to month' => 'interval year' . ($scale !== null ? "($scale)" : '') . ' to month',
         };
     }
 

src/Column/ColumnDefinitionParser.php

@@ -0,0 +1,55 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Db\Oracle\Column;
+
+use function preg_match;
+use function preg_replace;
+use function strlen;
+use function strtolower;
+use function substr;
+
+/**
+ * Parses column definition string. For example, `string(255)` or `int unsigned`.
+ */
+final class ColumnDefinitionParser extends \Yiisoft\Db\Syntax\ColumnDefinitionParser
+{
+    private const TYPE_PATTERN = '/^('
+        . 'timestamp\s*(?:\((\d+)\))? with(?: local)? time zone'
+        . '|interval year\s*(?:\((\d+)\))? to month'
+        . ')|('
+        . 'interval day\s*(?:\((\d+)\))? to second'
+        . '|long raw'
+        . '|\w*'
+        . ')\s*(?:\(([^)]+)\))?\s*'
+        . '/i';
+
+    public function parse(string $definition): array
+    {
+        preg_match(self::TYPE_PATTERN, $definition, $matches);
+
+        $type = strtolower(preg_replace('/\s*\(\d+\)/', '', $matches[4] ?? $matches[1]));
+        $info = ['type' => $type];
+
+        $typeDetails = $matches[6] ?? $matches[2] ?? '';
+
+        if ($typeDetails !== '') {
+            if ($type === 'enum') {
+                $info += $this->enumInfo($typeDetails);
+            } else {
+                $info += $this->sizeInfo($typeDetails);
+            }
+        }
+
+        $scale = $matches[5] ?? $matches[3] ?? '';
+
+        if ($scale !== '') {
+            $info += ['scale' => (int) $scale];
+        }
+
+        $extra = substr($definition, strlen($matches[0]));
+
+        return $info + $this->extraInfo($extra);
+    }
+}

src/Column/ColumnFactory.php

@@ -8,6 +8,7 @@
 use Yiisoft\Db\Schema\Column\AbstractColumnFactory;
 use Yiisoft\Db\Schema\Column\ColumnInterface;
 
+use function in_array;
 use function preg_replace;
 use function rtrim;
 use function strtolower;
@@ -49,6 +50,11 @@ final class ColumnFactory extends AbstractColumnFactory
         'long' => ColumnType::TEXT,
     ];
 
+    protected function columnDefinitionParser(): ColumnDefinitionParser
+    {
+        return new ColumnDefinitionParser();
+    }
+
     protected function getType(string $dbType, array $info = []): string
     {
         $dbType = strtolower($dbType);
@@ -63,8 +69,18 @@ protected function getType(string $dbType, array $info = []): string
 
         $dbType = preg_replace('/\([^)]+\)/', '', $dbType);
 
-        if ($dbType === 'interval day to second' && isset($info['size']) && $info['size'] > 0) {
-            return ColumnType::STRING;
+        if (in_array($dbType, [
+            'timestamp',
+            'timestamp with time zone',
+            'timestamp with local time zone',
+            'interval day to second',
+            'interval year to month',
+        ], true)) {
+            [$info['size'], $info['scale']] = [$info['scale'] ?? null, $info['size'] ?? null];
+
+            if ($dbType === 'interval day to second' && $info['scale'] > 0) {
+                return ColumnType::STRING;
+            }
         }
 
         return parent::getType($dbType, $info);

src/Command.php

@@ -5,7 +5,7 @@
 namespace Yiisoft\Db\Oracle;
 
 use PDO;
-use Yiisoft\Db\Command\DataType;
+use Yiisoft\Db\Constant\DataType;
 use Yiisoft\Db\Constant\PhpType;
 use Yiisoft\Db\Driver\Pdo\AbstractPdoCommand;
 use Yiisoft\Db\QueryBuilder\AbstractQueryBuilder;

tests/ColumnDefinitionParserTest.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Db\Oracle\Tests;
+
+use Yiisoft\Db\Oracle\Column\ColumnDefinitionParser;
+use Yiisoft\Db\Tests\AbstractColumnDefinitionParserTest;
+
+/**
+ * @group oracle
+ */
+final class ColumnDefinitionParserTest extends AbstractColumnDefinitionParserTest
+{
+    protected function createColumnDefinitionParser(): ColumnDefinitionParser
+    {
+        return new ColumnDefinitionParser();
+    }
+
+    /**
+     * @dataProvider \Yiisoft\Db\Oracle\Tests\Provider\ColumnDefinitionParserProvider::parse
+     */
+    public function testParse(string $definition, array $expected): void
+    {
+        parent::testParse($definition, $expected);
+    }
+}

tests/Provider/ColumnDefinitionParserProvider.php

@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Yiisoft\Db\Oracle\Tests\Provider;
+
+class ColumnDefinitionParserProvider extends \Yiisoft\Db\Tests\Provider\ColumnDefinitionParserProvider
+{
+    public static function parse(): array
+    {
+        return [
+            ...parent::parse(),
+            ['long raw', ['type' => 'long raw']],
+            ['interval day to second', ['type' => 'interval day to second']],
+            ['interval day to second (2)', ['type' => 'interval day to second', 'size' => 2]],
+            ['interval day(0) to second(2)', ['type' => 'interval day to second', 'size' => 2, 'scale' => 0]],
+            ['timestamp with time zone', ['type' => 'timestamp with time zone']],
+            ['timestamp (3) with time zone', ['type' => 'timestamp with time zone', 'size' => 3]],
+            ['timestamp(3) with local time zone', ['type' => 'timestamp with local time zone', 'size' => 3]],
+            ['interval year to month', ['type' => 'interval year to month']],
+            ['interval year (3) to month', ['type' => 'interval year to month', 'scale' => 3]],
+        ];
+    }
+}

tests/Provider/QueryBuilderProvider.php

@@ -344,6 +344,10 @@ public static function buildColumnDefinition(): array
 
             ['number(10) REFERENCES "ref_table" ("id")', ColumnBuilder::integer()->reference($referenceRestrict)],
             ['number(10) REFERENCES "ref_table" ("id") ON DELETE SET NULL', ColumnBuilder::integer()->reference($referenceSetNull)],
+            ['timestamp(3) with time zone', 'timestamp (3) with time zone'],
+            ['timestamp with local time zone', 'timestamp with local time zone'],
+            ['interval day(5) to second(6)', 'interval day(5) to second (6)'],
+            ['interval year(8) to month', 'interval year(8) to month'],
         ];
     }