added option convertDecimal & converts to int/float [Closes #257]

src/Database/Connection.php

@@ -28,7 +28,7 @@ class Connection
 		'pdo-sqlite' => Drivers\PDO\SQLite\Driver::class,
 		'pdo-sqlsrv' => Drivers\PDO\SQLSrv\Driver::class,
 	];
-	private const TypeConverterOptions = ['convertBoolean', 'convertDateTime', 'newDateTime'];
+	private const TypeConverterOptions = ['convertBoolean', 'convertDateTime', 'convertDecimal', 'newDateTime'];
 
 	/** @var array<callable(self): void>  Occurs after connection is established */
 	public array $onConnect = [];

src/Database/Drivers/Engines/MySQLEngine.php

@@ -178,9 +178,6 @@ public function getForeignKeys(string $table): array
 	public function convertToPhp(mixed $value, array $meta, TypeConverter $converter): mixed
 	{
 		return match ($meta['nativeType']) {
-			'NEWDECIMAL' => $meta['scale'] === 0
-				? $converter->toInt($value)
-				: $converter->toFloat($value),
 			'TINY' => $meta['size'] === 1 && $converter->convertBoolean
 				? $converter->toBool($value)
 				: $converter->toInt($value),

src/Database/TypeConverter.php

@@ -38,6 +38,7 @@ final class TypeConverter
 
 	public bool $convertBoolean = true;
 	public bool $convertDateTime = true;
+	public bool $convertDecimal = true;
 	public bool $newDateTime = true;
 
 
@@ -64,8 +65,10 @@ public function convertToPhp(mixed $value, array $meta): mixed
 	{
 		return match ($this->detectType($meta['nativeType'])) {
 			self::Integer => $this->toInt($value),
-			self::Float,
-			self::Decimal => $this->toFloat($value),
+			self::Float => $this->toFloat($value),
+			self::Decimal => $this->convertDecimal
+				? ($meta['scale'] === 0 ? $this->toInt($value) : $this->toFloat($value))
+				: $value,
 			self::Boolean => $this->convertBoolean ? $this->toBool($value) : $value,
 			self::DateTime, self::Date => $this->convertDateTime ? $this->toDateTime($value) : $value,
 			self::Time => $this->convertDateTime ? $this->toTime($value) : $value,
@@ -75,7 +78,7 @@ public function convertToPhp(mixed $value, array $meta): mixed
 	}
 
 
-	public function toInt(int|string $value): int|float
+	public function toInt(int|float|string $value): int|float|string
 	{
 		return is_float($tmp = $value * 1) ? $value : $tmp;
 	}

tests/Database/ResultSet.normalizeRow.sqlsrv.phpt

@@ -26,7 +26,7 @@ Assert::equal([
 	'date' => new DateTime('2012-10-13 00:00:00'),
 	'datetime' => new DateTime('2012-10-13 10:10:10'),
 	'datetime2' => new DateTime('2012-10-13 10:10:10'),
-	'decimal' => 1.0,
+	'decimal' => 1,
 	'float' => 1.1,
 	'geography' => "\xe6\x10\x00\x00\x01\x14\x87\x16\xd9\xce\xf7\xd3G@\xd7\xa3p=\n\x97^\xc0\x87\x16\xd9\xce\xf7\xd3G@\xcb\xa1E\xb6\xf3\x95^\xc0",
 	'geometry' => "\x00\x00\x00\x00\x01\x04\x03\x00\x00\x00\x00\x00\x00\x00\x00\x00Y@\x00\x00\x00\x00\x00\x00Y@\x00\x00\x00\x00\x00\x004@\x00\x00\x00\x00\x00\x80f@\x00\x00\x00\x00\x00\x80f@\x00\x00\x00\x00\x00\x80f@\x01\x00\x00\x00\x01\x00\x00\x00\x00\x01\x00\x00\x00\xff\xff\xff\xff\x00\x00\x00\x00\x02",
@@ -35,7 +35,7 @@ Assert::equal([
 	'money' => 1111.1,
 	'nchar' => 'a',
 	'ntext' => 'a',
-	'numeric_10_0' => 1.0,
+	'numeric_10_0' => 1,
 	'numeric_10_2' => 1.1,
 	'nvarchar' => 'a',
 	'real' => 1.1,
@@ -59,7 +59,7 @@ Assert::equal([
 	'date' => new DateTime('0001-01-01 00:00:00'),
 	'datetime' => new DateTime('1753-01-01 00:00:00'),
 	'datetime2' => new DateTime('0001-01-01 00:00:00'),
-	'decimal' => 0.0,
+	'decimal' => 0,
 	'float' => 0.5,
 	'geography' => null,
 	'geometry' => null,
@@ -68,7 +68,7 @@ Assert::equal([
 	'money' => 0.0,
 	'nchar' => ' ',
 	'ntext' => '',
-	'numeric_10_0' => 0.0,
+	'numeric_10_0' => 0,
 	'numeric_10_2' => 0.5,
 	'nvarchar' => '',
 	'real' => 0.0,

tests/Database/connection.options.mysql.phpt

@@ -91,3 +91,37 @@ test('convertDateTime = true', function () {
 	$field = $connection->fetchField('SELECT NOW()');
 	Assert::type(Nette\Database\DateTime::class, $field);
 });
+
+
+test('default convertDecimal', function () {
+	$connection = connectToDB(['convertDecimal' => null])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/mysql-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same(1, $row->decimal);
+	Assert::same(1.1, $row->decimal2);
+
+	$fields = $connection->fetchFields('SELECT 10, 10.5');
+	Assert::same([10, 10.5], $fields);
+});
+
+test('convertDecimal = false', function () {
+	$connection = connectToDB(['convertDecimal' => false])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/mysql-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same('1', $row->decimal);
+	Assert::same('1.10', $row->decimal2);
+
+	$fields = $connection->fetchFields('SELECT 10, 10.5');
+	Assert::same([10, '10.5'], $fields);
+});
+
+test('convertDecimal = true', function () {
+	$connection = connectToDB(['convertDecimal' => true])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/mysql-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same(1, $row->decimal);
+	Assert::same(1.1, $row->decimal2);
+
+	$fields = $connection->fetchFields('SELECT 10, 10.5');
+	Assert::same([10, 10.5], $fields);
+});

tests/Database/connection.options.sqlsrv.phpt

@@ -0,0 +1,40 @@
+<?php
+
+/**
+ * Test: Nette\Database\Connection sqlsrv options.
+ * @dataProvider? databases.ini  sqlsrv
+ */
+
+declare(strict_types=1);
+
+use Tester\Assert;
+
+require __DIR__ . '/../bootstrap.php';
+
+
+test('default convertDecimal', function () {
+	$connection = connectToDB(['convertDecimal' => null])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/sqlsrv-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same(1, $row->decimal);
+	Assert::same(1, $row->numeric_10_0);
+	Assert::same(1.1, $row->numeric_10_2);
+});
+
+test('convertDecimal = true', function () {
+	$connection = connectToDB(['convertDecimal' => true])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/sqlsrv-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same(1, $row->decimal);
+	Assert::same(1, $row->numeric_10_0);
+	Assert::same(1.1, $row->numeric_10_2);
+});
+
+test('convertDecimal = false', function () {
+	$connection = connectToDB(['convertDecimal' => false])->getConnection();
+	Nette\Database\Helpers::loadFromFile($connection, __DIR__ . '/files/sqlsrv-nette_test3.sql');
+	$row = $connection->fetch('SELECT * FROM types');
+	Assert::same('1', $row->decimal);
+	Assert::same('1', $row->numeric_10_0);
+	Assert::same('1.10', $row->numeric_10_2);
+});