Fix BigDecimal type

GrandMedia · Jun 26, 2020 · 517739b · 517739b
1 parent 6ed53c2
commit 517739b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Money/BigDecimalType.php b/src/Money/BigDecimalType.php
@@ -15,6 +15,11 @@ final class BigDecimalType extends \Doctrine\DBAL\Types\Type
 	 */
 	public function getSQLDeclaration(array $fieldDeclaration, AbstractPlatform $platform): string
 	{
+		$fieldDeclaration['precision'] = !isset($fieldDeclaration['precision']) || empty($fieldDeclaration['precision'])
+			? 20 : $fieldDeclaration['precision'];
+		$fieldDeclaration['scale'] = !isset($fieldDeclaration['scale']) || empty($fieldDeclaration['scale'])
+			? 4 : $fieldDeclaration['scale'];
+
 		return $platform->getDecimalTypeDeclarationSQL($fieldDeclaration);
 	}
 

diff --git a/tests/DoctrineTypes/Money/BigDecimalTypeTest.phpt b/tests/DoctrineTypes/Money/BigDecimalTypeTest.phpt
@@ -16,13 +16,13 @@ require_once __DIR__ . '/../../bootstrap.php';
 final class BigDecimalTypeTest extends \Tester\TestCase
 {
 
-	private const NUMBER = '123456789';
+	private const NUMBER = '123456789.95';
 
 	public function testGetSQLDeclaration(): void
 	{
 		$type = Type::getType(BigDecimalType::NAME);
 
-		Assert::same('NUMERIC(10, 0)', $type->getSQLDeclaration([], new Platform()));
+		Assert::same('NUMERIC(20, 4)', $type->getSQLDeclaration([], new Platform()));
 	}
 
 	public function testConvertToPhpValue(): void