Change, fix tests

briannesbitt · Jan 21, 2022 · 942ba23 · 942ba23
1 parent 19a0881
commit 942ba23
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/src/Carbon/Doctrine/CarbonTypeConverter.php b/src/Carbon/Doctrine/CarbonTypeConverter.php
@@ -36,9 +36,16 @@ protected function getCarbonClassName(): string
      */
     public function getSQLDeclaration(array $fieldDeclaration, AbstractPlatform $platform)
     {
-        $precision = ($fieldDeclaration['precision'] ?? 10) === 10
-            ? DateTimeDefaultPrecision::get()
-            : $fieldDeclaration['precision'];
+        $precision = $fieldDeclaration['precision'] ?: 10;
+
+        if ($fieldDeclaration['secondPrecision'] ?? false) {
+            $precision = 0;
+        }
+
+        if ($precision === 10) {
+            $precision = DateTimeDefaultPrecision::get();
+        }
+
         $type = parent::getSQLDeclaration($fieldDeclaration, $platform);
 
         if (!$precision) {

diff --git a/tests/Doctrine/CarbonTypesTest.php b/tests/Doctrine/CarbonTypesTest.php
@@ -65,15 +65,17 @@ public function testGetSQLDeclaration(string $name)
         $this->assertSame(6, $precision);
 
         $this->assertSame('DATETIME', $type->getSQLDeclaration([
-            'precision' => 0,
+            'precision' => null,
+            'secondPrecision' => true,
         ], new MySQL57Platform()));
 
         $this->assertSame('DATETIME(3)', $type->getSQLDeclaration([
             'precision' => 3,
         ], new MySQL57Platform()));
 
         $this->assertSame('TIMESTAMP(0)', $type->getSQLDeclaration([
-            'precision' => 0,
+            'precision' => null,
+            'secondPrecision' => true,
         ], new DB2Platform()));
 
         $this->assertSame('TIMESTAMP(6)', $type->getSQLDeclaration([